RESOLVED FIXED 82429
ShadowRoot should have selection attribute. 
https://bugs.webkit.org/show_bug.cgi?id=82429
Summary ShadowRoot should have selection attribute.
Shinya Kawanaka
Reported 2012-03-27 23:56:11 PDT
ShadowRoot currently does not have selection attribute. Spec is here. https://dvcs.w3.org/hg/webcomponents/raw-file/tip/spec/shadow/index.html#api-shadow-root-selection
Attachments
Patch (6.86 KB, patch)
2012-03-30 00:55 PDT, Shinya Kawanaka 		no flags
DetailsFormatted DiffDiff
Patch (7.02 KB, patch)
2012-04-09 01:03 PDT, Shinya Kawanaka 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Shinya Kawanaka
Comment 1 2012-03-29 06:24:28 PDT
Since we don't want to maintain multiple Selection objects, it would be good if we have a proxy object which implements Selection interface to return elements in shadow subtrees.
Shinya Kawanaka
Comment 2 2012-03-29 23:45:38 PDT
Let us just add a selection attribute for ShadowDOM. It's OK to return the same thing in window for now.
Shinya Kawanaka
Comment 3 2012-03-30 00:55:12 PDT
Created attachment 134741 [details] Patch
Hajime Morrita
Comment 4 2012-04-02 21:47:57 PDT
Could you elaborate whole plan to have valid SadowRoot.selection? Starting from just adding attribute is fine but we need a big picture to justify it.
Shinya Kawanaka
Comment 5 2012-04-09 00:51:07 PDT
Sorry for late response... I'm panning to fix in the following way. (1) Appending ShadowRoot.selection (2) Extending DOMSelection to have TreeScope. Each TreeScope will have its own DOMSelection as a cache. This will be tracked in Bug 82698.
Shinya Kawanaka
Comment 6 2012-04-09 00:51:40 PDT
I'll update the ChangeLog to have the above explanation...
Shinya Kawanaka
Comment 7 2012-04-09 01:03:56 PDT
Created attachment 136196 [details] Patch
WebKit Review Bot
Comment 8 2012-04-09 20:05:43 PDT
Comment on attachment 136196 [details] Patch Clearing flags on attachment: 136196 Committed r113667: <http://trac.webkit.org/changeset/113667>
WebKit Review Bot
Comment 9 2012-04-09 20:05:48 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.