Closed Bug 742537 Opened 8 years ago Closed 7 years ago

Remove HTMLMediaElement.initialTime

Categories

(Core :: Audio/Video, defect)

11 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla23

People

(Reporter: cpearce, Assigned: drexler)

Details

(Keywords: dev-doc-complete, site-compat)

Attachments

(1 file)

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.
Attached patch patchSplinter Review
Assignee: nobody → andrew.quartey
Attachment #731550 - Flags: review?(cpearce)
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. ;)
Attachment #731550 - Flags: review?(cpearce)
Attachment #731550 - Flags: review?(chris.double)
Attachment #731550 - Flags: review+
Comment on attachment 731550 [details] [diff] [review]
patch

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

Need to regenerate the uuid in the .idl file?
Attachment #731550 - Flags: review?(chris.double) → review+
https://hg.mozilla.org/mozilla-central/rev/88ff175d85fe
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in before you can comment on or make changes to this bug.