Bug 105481

Summary: Implement ID selector matching for the WebVTT ::cue pseudo element
Product: WebKit Reporter: Dima Gorbik <dgorbik>
Component: MediaAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: eric.carlson, feature-media-reviews, koivisto, ojan.autocc, silviapf, webkit-bug-importer, webkit.review.bot
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 105480    
Bug Blocks: 43668, 106762    
Attachments:
Description Flags
Proposed fix 0.1
none
Proposed fix 0.2
eric.carlson: review+
Proposed fix 0.2 none

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.