742537 - Remove HTMLMediaElement.initialTime

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

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

HTMLMediaElement.initialTime has been removed from the WhatWG HTML spec "since it doesn't have convincing use cases" so we should remove it too I guess.

Comment 1

[Andrew Quartey [:drexler]]

Attachment: patch

Comment 2

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

Comment on attachment 731550 [details] [diff] [review]
patch

Review of attachment 731550 [details] [diff] [review]:
-----------------------------------------------------------------

Can you also remove "TODO" the comments referring to initialTime in MediaDecoder.cpp and test_seekable.html please? Thanks.

Requesting additional review from doublec, so that this doesn't look like such an inside job. ;)

Comment 3

[cajbir (:cajbir)]

Comment on attachment 731550 [details] [diff] [review]
patch

Review of attachment 731550 [details] [diff] [review]:
-----------------------------------------------------------------

Need to regenerate the uuid in the .idl file?

Comment 4

[Andrew Quartey [:drexler]]

fixed nits and pushed to inbound: http://hg.mozilla.org/integration/mozilla-inbound/rev/88ff175d85fe

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/88ff175d85fe

Comment 6

[Kohei Yoshino [:kohei]]

https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement
https://developer.mozilla.org/en-US/docs/Web/API/HTMLAudioElement
https://developer.mozilla.org/en-US/docs/Site_Compatibility_for_Firefox_23