1350209 - Turn off Android OOP decoding on beta53

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: Audio/Video, defect)

Description

[John Lin [:jhlin][:jolin]]

Bug 1319987 creates a new version of RemoteDataDecoder that behaves considerably different from the one in beta53. Disable OOP decoding in beta53 so we don't have to worry about bugs that live for only 6 weeks.

Comment 1

[John Lin [:jhlin][:jolin]]

Attachment: Turn off remote decoder pref.

Comment 2

[Liz Henry (:lizzard) (relman/hg->git project)]

Tracking for 53 as this may be causing problems with mp4 playback - in bug 1350279
Please request uplift once this lands (I will check back Sunday night to get ready for Monday's beta 7 build)

Comment 3

[John Lin [:jhlin][:jolin]]

Comment on attachment 8850860 [details] [diff] [review]
Turn off remote decoder pref.

Approval Request Comment
[Feature/Bug causing the regression]: bug 1257777
[User impact if declined]: unstable video playback
[Is this code covered by automated tests?]:no
[Has the fix been verified in Nightly?]:no
[Needs manual test from QE? If yes, steps to reproduce]: no
[List of other uplifts needed for the feature/fix]: none
[Is the change risky?]: low risk
[Why is the change risky/not risky?]: disabling pref makes it behave the exactly same as 52.
[String changes made/needed]:none

Comment 4

[Gerry Chang [:gchang]]

Comment on attachment 8850860 [details] [diff] [review]
Turn off remote decoder pref.

Per comment #2, disable OOP in Beta53. Beta53+.

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-beta/rev/e7ec52251499

Comment 6

[Randell Jesup [:jesup] (needinfo me)]

since this landed on beta, this bug should be marked fixed

Comment 7

[Oana Horvath [:ohorvath]]

Verified as fixed on Beta 53.0b9 (2017-04-04). The media.android-remote-codec.enabled pref is turned off.
Device: Motorola Nexus 6 (Android 7.0)

Comment 8

[Liz Henry (:lizzard) (relman/hg->git project)]

We disabled this for 53. How about 54?

Comment 9

[John Lin [:jhlin][:jolin]]

It's enabled in 54.