Bug 1280829 fixed video playback at http://my.xfinity.com/video/ but it regressed again between 2016-09-28 and 2016-09-29.
[Tracking Requested - why for this release]: This is an EME regression in Nightly 52 from bug 1280829.
status-firefox50: --- → unaffected
status-firefox51: --- → unaffected
status-firefox52: --- → affected
tracking-firefox52: --- → ?
This is a regression from Bug 1300654, that landed with an incorrect merge conflict resolution which effectively backed out bug 1280829: https://hg.mozilla.org/mozilla-central/rev/8eef5d2cc850#l8.108
https://hg.mozilla.org/integration/mozilla-inbound/rev/ff26deb4079871216636252d24fa0dceb6b67c98 Bug 1307595 - Fixup bad merge conflict resolution from Bug 1300654/8eef5d2cc850. r=partial-backout
(In reply to Chris Pearce (:cpearce) from comment #3) > https://hg.mozilla.org/integration/mozilla-inbound/rev/ > ff26deb4079871216636252d24fa0dceb6b67c98 > Bug 1307595 - Fixup bad merge conflict resolution from Bug > 1300654/8eef5d2cc850. r=partial-backout Partial backout of the incorrect merge conflict resolution.
Created attachment 8797850 [details] [diff] [review] Fixup bad merge conflict resolution from Bug 1300654/8eef5d2cc850 8eef5d2cc850 in Bug 1300654 incorrectly resolved a merge conflict and backed out part of the fix for Bug 1280829. Relanding. MozReview-Commit-ID: 6DIkUrc9R4t
Comment on attachment 8797850 [details] [diff] [review] Fixup bad merge conflict resolution from Bug 1300654/8eef5d2cc850 Approval Request Comment [Feature/regressing bug #]: The build was broken on Aurora with the landing of bug 1299694. The fix for that is in bug 1300654, however that had a mismerge, so we also need this patch to fix the mismerge. [User impact if declined]: Some videos which setup EME but don't actually use EME will not play. [Describe test coverage new/current, TreeHerder]: We have a specific test to test this case, it's currently on aurroa. [Risks and why]: Low, fixing a bad merge. [String/UUID change made/needed]: None.
Attachment #8797850 - Flags: approval-mozilla-aurora?
Comment on attachment 8797850 [details] [diff] [review] Fixup bad merge conflict resolution from Bug 1300654/8eef5d2cc850 Fix a bad merge conflict. Take it in 51 aurora.
Attachment #8797850 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
status-firefox51: affected → fixed
tracking-firefox52: ? → ---
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox52: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
status-firefox49: --- → unaffected
Version: unspecified → 51 Branch
This issue is verified fixed on latest Nightly/Aurora builds from 2016-10-10 on the following OSes: - Windows 7 x64 - Ubuntu 16.04 x64 LTS - Mac OS X 10.11.6
Status: RESOLVED → VERIFIED
status-firefox51: fixed → verified
status-firefox52: fixed → verified
You need to log in before you can comment on or make changes to this bug.