RESOLVED FIXED 127483
Merge Chromium TextTrack cleanups 
https://bugs.webkit.org/show_bug.cgi?id=127483
Summary Merge Chromium TextTrack cleanups
Eric Carlson
Reported 2014-01-23 08:37:13 PST
Merge some of the Chromium TextTrack cleanups that have happened since the fork.
Attachments
Proposed patch (13.95 KB, text/plain)
2014-01-23 08:50 PST, Eric Carlson 		darin: review+
Details
View All Add attachment proposed patch, testcase, etc.
Eric Carlson
Comment 1 2014-01-23 08:50:00 PST
Created attachment 221986 [details] Proposed patch
Darin Adler
Comment 2 2014-01-23 09:46:53 PST
Comment on attachment 221986 [details] Proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=221986&action=review > Source/WebCore/html/HTMLTrackElement.h:91 > + bool canLoadUrl(const URL&); Should be named canLoadURL. > Source/WebCore/loader/TextTrackLoader.cpp:79 > + m_resource = 0; nullptr
Eric Carlson
Comment 3 2014-01-27 07:03:09 PST
Committed r162836: http://trac.webkit.org/changeset/162836
Note You need to log in before you can comment on or make changes to this bug.