1335166 - [EME] Backout Bug 1324925

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, defect, P1)

Description

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

I misinterpreted the spec in Bug 1324925, and need to back that out from 53, i.e. Aurora.

Comment 1

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

Attachment: Backed out changeset 229e1278d976i (Bug 1335166)

MozReview-Commit-ID: NnHo7YXFls

Comment 2

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

Comment on attachment 8831799 [details] [diff] [review]
Backed out changeset 229e1278d976i (Bug 1335166)

Requesting approval to back out this change from 53, i.e. Aurora.

Approval Request Comment
[Feature/Bug causing the regression]: Bug 1324925.
[User impact if declined]: Some EME videos will incorrectly report their expiration time to JavaScript. This could potentially cause EME video playback to fail.
[Is this code covered by automated tests?]: We don't have tests covering EME expiration as we don't have our own DRM server.
[Has the fix been verified in Nightly?]: Yes, I have verified the backout is as expected in Nightly.
[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?]: No.
[Why is the change risky/not risky?]: It's basically removing a divide by 1000.
[String changes made/needed]: None.

Comment 3

[Liz Henry (:lizzard)]

Comment on attachment 8831799 [details] [diff] [review]
Backed out changeset 229e1278d976i (Bug 1335166)

Regression from 53, let's back this out from aurora.

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/086d2492f20b

Comment 5

[Carsten Book [:Tomcat]]

landed this backout in aurora