Bug 62890

Summary: Drop MutableTextTrack and make TextTrack always mutable
Product: WebKit Reporter: Anna Cavender <annacc>
Component: MediaAssignee: Eric Carlson <eric.carlson>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, gustavo.noronha, gustavo, ojan, philipj, webkit-bug-importer, webkit.review.bot, xan.lopez
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 43668    
Attachments:
Description Flags
Proposed patch
none
Updated patch
gustavo.noronha: commit-queue-
Updated patch sam: review+

Anna Cavender
Reported 2011-06-17 11:05:24 PDT
This change adds MutableTextTrack capabilities (addTextTrack() in HTMLMediaElement, and addCue() and removeCue() in MutableTextTrack, plus required functionality).
Attachments
Proposed patch (32.24 KB, text/plain)
2011-10-20 14:58 PDT, Eric Carlson
no flags
Updated patch (34.99 KB, patch)
2011-10-21 09:06 PDT, Eric Carlson
gustavo.noronha: commit-queue-
Updated patch (36.52 KB, patch)
2011-10-21 15:03 PDT, Eric Carlson
sam: review+
Anna Cavender
Comment 1 2011-10-19 21:09:18 PDT
MutableTextTrack was recently removed from the whatwg spec: http://html5.org/tools/web-apps-tracker?from=6709&to=6710
Eric Carlson
Comment 2 2011-10-20 14:58:22 PDT
Created attachment 111853 [details] Proposed patch Pull MutableTextTrack functionality into TextTrack.
Gustavo Noronha (kov)
Comment 3 2011-10-20 19:18:37 PDT
Comment on attachment 111853 [details] Proposed patch Attachment 111853 [details] did not pass gtk-ews (gtk): Output: http://queues.webkit.org/results/10177708
Radar WebKit Bug Importer
Comment 4 2011-10-21 09:05:28 PDT
Eric Carlson
Comment 5 2011-10-21 09:06:02 PDT
Created attachment 111973 [details] Updated patch Remove MutableTextTrack and its friends from a few more build files.
Collabora GTK+ EWS bot
Comment 6 2011-10-21 09:29:16 PDT
Comment on attachment 111973 [details] Updated patch Attachment 111973 [details] did not pass gtk-ews (gtk): Output: http://queues.webkit.org/results/10181909
Sam Weinig
Comment 7 2011-10-21 11:29:36 PDT
Comment on attachment 111973 [details] Updated patch View in context: https://bugs.webkit.org/attachment.cgi?id=111973&action=review This change is testable, since there shouldn't be a window.MutableTextTrack property anymore. Was there no test for that before? Is it not exposed on DOMWindow.idl at all yet? > Source/WebCore/html/HTMLMediaElement.cpp:1986 > + RefPtr<TextTrack> track = TextTrack::create(this, kind, label, language); > + return track.release(); This RefPtr is not necessary.
Eric Carlson
Comment 8 2011-10-21 14:58:44 PDT
(In reply to comment #7) > (From update of attachment 111973 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=111973&action=review > > This change is testable, since there shouldn't be a window.MutableTextTrack property anymore. Was there no test for that before? Is it not exposed on DOMWindow.idl at all yet? > No, not all of the cue classes are in DOMWindow.idl yet. I will file a bug about this. > > Source/WebCore/html/HTMLMediaElement.cpp:1986 > > + RefPtr<TextTrack> track = TextTrack::create(this, kind, label, language); > > + return track.release(); > > This RefPtr is not necessary. Indeed, thanks!
Eric Carlson
Comment 9 2011-10-21 15:03:35 PDT
Created attachment 112028 [details] Updated patch
Eric Carlson
Comment 10 2011-10-24 07:41:18 PDT
Note You need to log in before you can comment on or make changes to this bug.