1137045 - (eme-platform-uplift) [EME] Uplift Gecko EME changesets to Firefox 37 beta

Status: RESOLVED WORKSFORME

(Core :: Audio/Video, defect)

Description

[Chris Pearce [:cpearce (Not reading bugmail)]]

The decision has been made to uplift EME to Firefox 37. This bug covers the platform pieces.

Comment 1

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=42ee69e9b693

Comment 2

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=493df075a05a

Comment 3

[Chris Pearce [:cpearce (Not reading bugmail)]]

Patch queue at:

https://bitbucket.org/chrisryanpearce/eme-37-uplift/

Comment 4

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=89b76b7218f8

Comment 5

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=514abf393915

Comment 6

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=a7e1da46bc65

Comment 7

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=1454c17fc929

Comment 8

[Chris Pearce [:cpearce (Not reading bugmail)]]

Landed my queue:
https://treeherder.mozilla.org/#/jobs?repo=mozilla-beta&revision=8abdbdecd2d6

I'll still need to uplift the stuff that landed in Nightly after the last uplift.

Comment 9

[Chris Pearce [:cpearce (Not reading bugmail)]]

Round #2:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=152cc504aca0

Comment 10

[Chris Pearce [:cpearce (Not reading bugmail)]]

Beta push with these changesets:

eec321940021 Bug 1138771 - Ensure we don't deref a null pointer in assertion in GMPVideoDecoderParent::Shutdown(). r=edwin
ae1bfc458851 Bug 1131392 - Restrict EME to MSE only - r=cpearce
c7f1c4d0e605 Bug 1131392 - Remove unused test code after disabling non-MSE EME - r=cpearce
c3732d9a9abe Bug 1131392 - Fix EME tests after disable non-MSE EME - r=cpearce
2f369b1abdc2 Bug 1139316 - Add pref to enable EME outside of MSE. r=edwin

https://treeherder.mozilla.org/#/jobs?repo=try&revision=6322f4d700c2

I didn't include Bug 1131798, Bug 1123990, or Bug 1134913 as I recall Bug 1131798 broke Nightly builds without Bug 1134913, and Bug 1134913 requires Bug 1123990 which depends on bug 1138535 being fixed.

Comment 11

[Chris Pearce [:cpearce (Not reading bugmail)]]

Aurora push same as comment 10:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=f65141864dd9

Comment 12

[Chris Pearce [:cpearce (Not reading bugmail)]]

(In reply to Chris Pearce (:cpearce) from comment #11)
> Aurora push same as comment 10:
> 
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=f65141864dd9

Somehow, dom/media/test/short-cenc.mp4 ended up on Beta but not aurora...

Fixed:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=413f713d0ada

Comment 13

[Chris Pearce [:cpearce (Not reading bugmail)]]

Aurora try, with bug 1138535:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=9ac9690083eb

Comment 14

[Chris Pearce [:cpearce (Not reading bugmail)]]

Beta, with bug 1138535 and bug 1133572:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=42973f4c59d6

Comment 15

[Chris Pearce [:cpearce (Not reading bugmail)]]

Also with Bug 1138253:

Beta:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=81728c87a6d9

Aurora:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=8e8b118bb880

Comment 16

[Chris Pearce [:cpearce (Not reading bugmail)]]

(In reply to Chris Pearce (:cpearce) from comment #15)
> Also with Bug 1138253:
> 
> Beta:
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=81728c87a6d9

This push is missing Bug 1138535, so re-trying:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=9295d895f9d5

> Aurora:
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=8e8b118bb880

Aurora here had Bug 1138535 in the push. So this is good to land.

Comment 17

[Chris Pearce [:cpearce (Not reading bugmail)]]

Aurora, landed yesterday:
https://treeherder.mozilla.org/#/jobs?repo=mozilla-aurora&revision=39379b6b90dc

Beta, landed just now:
https://hg.mozilla.org/releases/mozilla-beta/pushloghtml?changeset=68707623b5a3

That should be everything except any fixes we find in the next few days...

Comment 18

[Chris Pearce [:cpearce (Not reading bugmail)]]

Try push of preffing on:

Aurora:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=8739d57dd26f

Beta:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=69f969ab6cd1

Comment 19

[Chris Pearce [:cpearce (Not reading bugmail)]]

Gonna close this... Uplift to 37 is halted.

Comment 20

[Chris Pearce [:cpearce (Not reading bugmail)]]

Try push for some more stragglers:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=97d3f40341c3