RESOLVED FIXED 105481
Implement ID selector matching for the WebVTT ::cue pseudo element
https://bugs.webkit.org/show_bug.cgi?id=105481
Summary Implement ID selector matching for the WebVTT ::cue pseudo element
Dima Gorbik
Reported 2012-12-19 16:54:13 PST
Attachments
Proposed fix 0.1 (1.95 KB, patch)
2012-12-20 19:30 PST, Dima Gorbik
no flags
Proposed fix 0.2 (6.80 KB, patch)
2013-01-13 23:14 PST, Dima Gorbik
eric.carlson: review+
Proposed fix 0.2 (7.24 KB, patch)
2013-01-14 18:28 PST, Dima Gorbik
no flags
Radar WebKit Bug Importer
Comment 1 2012-12-19 16:55:09 PST
Dima Gorbik
Comment 2 2012-12-20 19:30:10 PST
Created attachment 180459 [details] Proposed fix 0.1 This patch adds support for cue ID matching and also voice attribute matching for the WebVTT CSS extensions. I will add layout tests after the previous patch is being reviewed (https://bugs.webkit.org/show_bug.cgi?id=105473) because they will be based on the existing layout tests from that patch.
Dima Gorbik
Comment 3 2013-01-13 23:14:13 PST
Created attachment 182515 [details] Proposed fix 0.2
Eric Carlson
Comment 4 2013-01-14 08:41:04 PST
Comment on attachment 182515 [details] Proposed fix 0.2 View in context: https://bugs.webkit.org/attachment.cgi?id=182515&action=review > LayoutTests/media/track/track-css-matching.html:54 > consoleWrite(""); > consoleWrite(""); Nit: two blank lines can be added with consoleWrite("<br>");
Dima Gorbik
Comment 5 2013-01-14 18:28:06 PST
Created attachment 182675 [details] Proposed fix 0.2
WebKit Review Bot
Comment 6 2013-01-14 21:24:09 PST
Comment on attachment 182675 [details] Proposed fix 0.2 Clearing flags on attachment: 182675 Committed r139714: <http://trac.webkit.org/changeset/139714>
WebKit Review Bot
Comment 7 2013-01-14 21:24:13 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.