Closed Bug 1136165 Opened 7 years ago Closed 7 years ago

Uplift EME UI patches to 37 beta to increase testing population

Categories

(Firefox :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: Gijs, Unassigned)

References

Details

Attachments

(7 files, 1 obsolete file)

(I'm assuming this is confidential per previous email discussions, if not, please open up immediately)

So I was hoping to get patches ready, except it seems like most of the backend stuff (specifically at least bug 1124491 and bug 1111160) haven't landed either, and so neither the context menu nor the notifications would actually work... I tried generating uplift patches myself, but at least 1111160 doesn't apply at all cleanly on current 37 beta. Chris, can you look at this?
Flags: needinfo?(cpearce)
Release management want to see the launch partner's site working with Adobe EME working in Nightly before approving platform uplifts, and currently Adobe's CDM doesn't work and the launch partner have no CDM to test against so uplifting the platform bits in time for a week of runtime in a beta cycle before we have to pref it off seems overly optimistic. Adobe are at a summit this week, so nothing will happen on their end this week. We're meeting with Adobe and the launch partner on Monday PDT to discuss whether its possible. My guess is that we're not going to make the window for testing in beta 37.
Flags: needinfo?(cpearce)
To be clear: there is significant risk that we won't be able to get user testing in 37-Beta (due to partner work being behind schedule), but we _do_ want to proceed with getting patches uplifted to 37. So that work should proceed.

I assume 1111160 will apply to 37 one Chris finished preparing his ~50-patch EME/MSE uplift.
Yes. Bug 1111160 should apply, or at least be trivially unbitrotable once I'm done uplifting.
I am tracking uplift of platform pieces in bug 1137045.
Depends on: 1139022
Group: mozilla-employee-confidential
Attached file MozReview Request: bz://1136165/Gijs (obsolete) —
/r/4555 - Bug 1111146 - [EME] Implement master pref for playing DRM content, including pref. r=gijs,dao
/r/4557 - Bug 1127288 - add header for DRM checkbox, fix link alignment, r=dolske
/r/4559 - Bug 1111147 - update nsContextMenu for EME, r=florian
/r/4561 - Bug 1111148 - show doorhanger for EME being played back, r=florian
/r/4563 - Bug 1111153 - show error notifications for broken EME content (includes fix for bug 1139022), r=florian
/r/4565 - Bug 1131758 - indicate 64-bit windows or OSX/Linux incompatibilities for Adobe's CDM, r=dolske
/r/4567 - Bug 1135237 - update message for EME notification, r=dolske

Pull down these commits:

hg pull review -r 67cb9c08b70c0def672e11939dff0a17f86c6880
Comment on attachment 8572105 [details]
MozReview Request: bz://1136165/Gijs

Approval Request Comment
[Feature/regressing bug #]: EME
[User impact if declined]: No UI for EME when we try it out on beta
[Describe test coverage new/current, TreeHerder]: nope. :-(
[Risks and why]: reasonably low, disabled by default and will only be used for beta testing with partners
[String/UUID change made/needed]: moved all the strings to either be inline or in a content-side properties file, and verified with Axel that that's OK for l10n.
Attachment #8572105 - Flags: review?(dolske)
Attachment #8572105 - Flags: approval-mozilla-beta?
https://treeherder.mozilla.org/#/jobs?repo=try&revision=78f41e211302 has everything except the last commit, which I'd fix except try is closed...
Comment on attachment 8572105 [details]
MozReview Request: bz://1136165/Gijs

rs=me, I didn't look deeply at this since it's just hard-coding strings from previous work.
Attachment #8572105 - Flags: review?(dolske)
Attachment #8572105 - Flags: review+
Attachment #8572105 - Flags: approval-mozilla-beta?
Attachment #8572105 - Flags: approval-mozilla-beta+
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
\o/ Thanks.
Attachment #8572105 - Attachment is obsolete: true
Attachment #8619568 - Flags: review+
Attachment #8619569 - Flags: review+
Attachment #8619570 - Flags: review+
Attachment #8619571 - Flags: review+
Attachment #8619572 - Flags: review+
Attachment #8619573 - Flags: review+
Attachment #8619574 - Flags: review+
You need to log in before you can comment on or make changes to this bug.