1183495 - Remove mozSrcObject alias to srcObject soon

Status: RESOLVED FIXED

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

Description

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

A couple of releases after Bug 1175523 we'll want to remove the prefixed backwards-compatible alias for HTMLMediaElement.srcObject.

Comment 1

[Kohei Yoshino]

Posted the site compatibility doc: https://www.fxsitecompat.com/en-US/docs/2015/htmlmediaelement-mozsrcobject-will-be-removed/

Comment 2

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

(In reply to Jan-Ivar Bruaroey [:jib] from comment #0)
> A couple of releases after Bug 1175523 we'll want to remove the prefixed
> backwards-compatible alias for HTMLMediaElement.srcObject.

After a dozen releases, feel like doing this?

Comment 3

[Bulk Bug Changes for mreavy's org]

Mass change P1->P2 to align with new Mozilla triage process

Comment 4

[Blake Wu [:bwu][:blakewu]]

Benjamin,
Can you put this on your plate?

Comment 5

[Benjamin Chen [:bechen]]

Attachment: Bug 1183495 - part1: Remove mozSrcObject in HtmlMediaElement.ipdl .

Review commit: https://reviewboard.mozilla.org/r/191446/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/191446/

Comment 6

[Benjamin Chen [:bechen]]

Attachment: Bug 1183495 - part2: Remove mozSrcObject in MediaElement.

Review commit: https://reviewboard.mozilla.org/r/191448/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/191448/

Comment 7

[Benjamin Chen [:bechen]]

Attachment: Bug 1183495 - part3: Remove mozSrcObject usage.

Review commit: https://reviewboard.mozilla.org/r/191450/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/191450/

Comment 8

[JW Wang [:jwwang] [:jw_wang]]

Comment on attachment 8920474 [details]
Bug 1183495 - part2: Remove mozSrcObject in MediaElement.

https://reviewboard.mozilla.org/r/191448/#review196662

Comment 9

[JW Wang [:jwwang] [:jw_wang]]

Comment on attachment 8920475 [details]
Bug 1183495 - part3: Remove mozSrcObject usage.

https://reviewboard.mozilla.org/r/191450/#review196664

Comment 10

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8920473 [details]
Bug 1183495 - part1: Remove mozSrcObject in HtmlMediaElement.ipdl .

https://reviewboard.mozilla.org/r/191446/#review196712

Hopefully this isn't used too often.

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6ab062f29097
part1: Remove mozSrcObject in HtmlMediaElement.ipdl . r=smaug
https://hg.mozilla.org/integration/autoland/rev/74602ce4e372
part2: Remove mozSrcObject in MediaElement. r=jwwang
https://hg.mozilla.org/integration/autoland/rev/21c88386cd1a
part3: Remove mozSrcObject usage. r=jwwang

Comment 12

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/6ab062f29097
https://hg.mozilla.org/mozilla-central/rev/74602ce4e372
https://hg.mozilla.org/mozilla-central/rev/21c88386cd1a

Comment 13

[Kohei Yoshino]

Posted the site compatibility note: https://www.fxsitecompat.com/en-CA/docs/2017/htmlmediaelement-mozsrcobject-has-been-removed/

Comment 14

[Chris Mills [:cmills]]

I've documented this on MDN, updating the relevant browser compat data:

https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement#Browser_compatibility
https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement/srcObject#Browser_compatibility

And adding a note the Fx58 rel notes:

https://developer.mozilla.org/en-US/Firefox/Releases/58#APIs_2