Bug 99680

Summary: Elements assinged to <shadow> should not be reprojected.
Product: WebKit Reporter: Shinya Kawanaka <shinyak>
Component: DOMAssignee: Shinya Kawanaka <shinyak>
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov, webcomponents-bugzilla, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 96986    
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch none

Shinya Kawanaka
Reported 2012-10-17 21:53:34 PDT
In the current spec, we don't have shadow reprojection. Let's have a test case for it.
Attachments
Patch (6.44 KB, patch)
2012-10-17 22:20 PDT, Shinya Kawanaka 		no flags
DetailsFormatted DiffDiff
Patch (11.65 KB, patch)
2012-10-18 21:41 PDT, Shinya Kawanaka 		no flags
DetailsFormatted DiffDiff
Patch (9.65 KB, patch)
2012-10-18 22:20 PDT, Shinya Kawanaka 		no flags
DetailsFormatted DiffDiff
Patch (8.53 KB, patch)
2012-10-18 22:21 PDT, Shinya Kawanaka 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Shinya Kawanaka
Comment 1 2012-10-17 22:03:10 PDT
Ah we don't prohibit shadow reprojection now in the current implementation...
Shinya Kawanaka
Comment 2 2012-10-17 22:20:50 PDT
Created attachment 169345 [details] Patch
WebKit Review Bot
Comment 3 2012-10-18 00:18:21 PDT
Comment on attachment 169345 [details] Patch Attachment 169345 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/14387901 New failing tests: editing/shadow/breaking-editing-boundaries.html
Dimitri Glazkov (Google)
Comment 4 2012-10-18 08:57:16 PDT
Comment on attachment 169345 [details] Patch what's that test failure all about?
Shinya Kawanaka
Comment 5 2012-10-18 18:32:41 PDT
Hmm... I'll check it soon.
Shinya Kawanaka
Comment 6 2012-10-18 18:49:04 PDT
Oh... The test hits my bug. We cannot use multiple <shadow> in this implementation... I have to fix it.
Shinya Kawanaka
Comment 7 2012-10-18 21:41:53 PDT
Created attachment 169545 [details] Patch
Shinya Kawanaka
Comment 8 2012-10-18 22:12:07 PDT
Ah... This is still wrong in the current spec (W3C Editor's Draft 19 October 2012). In the current spec, <content> can select <shadow> itself, but it cannot decompose the elements assigned to <shadow>.
Shinya Kawanaka
Comment 9 2012-10-18 22:20:42 PDT
Created attachment 169546 [details] Patch
Shinya Kawanaka
Comment 10 2012-10-18 22:21:28 PDT
Created attachment 169547 [details] Patch
WebKit Review Bot
Comment 11 2012-10-19 09:56:30 PDT
Comment on attachment 169547 [details] Patch Clearing flags on attachment: 169547 Committed r131910: <http://trac.webkit.org/changeset/131910>
WebKit Review Bot
Comment 12 2012-10-19 09:56:34 PDT
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.