Closed Bug 1322606 Opened 3 years ago Closed 3 years ago
Error::message attribute available on all version
58 bytes, text/x-review-board-request
HTML5 Specs have been amended to include the MediaError::message attribute. We want to expose it on all versions, not just aurora and nightly... https://github.com/whatwg/html/issues/2085 https://github.com/whatwg/html/pull/2086
Comment on attachment 8817540 [details] Bug 1322606: Don't hide MediaError::message behind a pref. https://reviewboard.mozilla.org/r/97800/#review98674 r+ with two small comments: * Can you please also remove http://searchfox.org/mozilla-central/rev/594937fec2e2fc45fa9308ba2fb964816631f017/dom/media/test/test_decode_error.html#58? * Can you please send a quick intend to ship for this, mostly to inform folks. Thank you!
Attachment #8817540 - Flags: review?(ehsan) → review+
Attachment #8817540 - Flags: review?(bzbarsky)
(In reply to :Ehsan Akhgari from comment #2) > Comment on attachment 8817540 [details] > Bug 1322606: Don't hide MediaError::message behind a pref. > > https://reviewboard.mozilla.org/r/97800/#review98674 > > r+ with two small comments: > > * Can you please also remove > http://searchfox.org/mozilla-central/rev/ > 594937fec2e2fc45fa9308ba2fb964816631f017/dom/media/test/test_decode_error. > html#58? > * Can you please send a quick intend to ship for this, mostly to inform > folks. this was done yesterday... going to wait a few days for it to settle.
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/3bbd52424e36 Don't hide MediaError::message behind a pref. r=Ehsan
Too late for 51. Mark 51 won't fix. Hi :jya, Is this worth uplifting to Aurora52?
Comment on attachment 8817540 [details] Bug 1322606: Don't hide MediaError::message behind a pref. Approval Request Comment [Feature/Bug causing the regression]: new feature [User impact if declined]: not compliant with new W3C spec [Is this code covered by automated tests?]: yes [Has the fix been verified in Nightly?]: yes [Needs manual test from QE? If yes, steps to reproduce]: no [List of other uplifts needed for the feature/fix]: no [Is the change risky?]: no [Why is the change risky/not risky?]: adding a new attribute [String changes made/needed]: none
Attachment #8817540 - Flags: approval-mozilla-aurora?
Comment on attachment 8817540 [details] Bug 1322606: Don't hide MediaError::message behind a pref. expose MediaError::message, aurora52+
Attachment #8817540 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
For documentation purposes: if I read this correctly, this is basically about making this property available by default on beta and release builds starting with Firefox 52. I assume it shipped enabled by default in Firefox 52 despite being labeled as “Fixed” in 53. Please advise if I’m wrong.
Yes, that's why "firefox52: fixed" in the tracking flags.
(In reply to Boris Zbarsky [:bz] (still a bit busy) from comment #12) > Yes, that's why "firefox52: fixed" in the tracking flags. Yeah, these accurate about 99.999% of the time, but I've been bitten a couple of times, so if I have any doubt at all, I double-check.
Documentation added: https://developer.mozilla.org/en-US/docs/Web/API/MediaError/message https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement/onerror Documentation updated: https://developer.mozilla.org/en-US/docs/Web/API/MediaError https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement/error https://developer.mozilla.org/en-US/docs/Web/Events/error Listed on Firefox 52 for developers: https://developer.mozilla.org/en-US/Firefox/Releases/53#AudioVideoMedia Example added to Github: https://github.com/mdn/dom-examples/tree/master/media/mediaerror This is now complete.
You need to log in before you can comment on or make changes to this bug.