Summary: | Implement querySelector on ShadowRoot | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Hayato Ito <hayato> | ||||||||
Component: | DOM | Assignee: | Hayato Ito <hayato> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | dglazkov, dominicc, morrita, rolandsteiner, shinyak, webkit.review.bot | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 63601 | ||||||||||
Attachments: |
|
Description
Hayato Ito
2012-02-02 22:04:31 PST
Created attachment 125259 [details]
FAIL case
Created attachment 125594 [details]
implement querySelector
Comment on attachment 125594 [details] implement querySelector View in context: https://bugs.webkit.org/attachment.cgi?id=125594&action=review > Source/WebCore/dom/SelectorQuery.cpp:101 > +static inline bool nodeIsRootNodeOfTreeScope(Node* node) seems too long of a name. Maybe isTreeScopeRoot? Thank you for the review. (In reply to comment #3) > (From update of attachment 125594 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=125594&action=review > > > Source/WebCore/dom/SelectorQuery.cpp:101 > > +static inline bool nodeIsRootNodeOfTreeScope(Node* node) > > seems too long of a name. Maybe isTreeScopeRoot? Sounds good. Let me rename it and land this patch. Created attachment 125747 [details]
Patch for landing
Comment on attachment 125747 [details] Patch for landing Clearing flags on attachment: 125747 Committed r106902: <http://trac.webkit.org/changeset/106902> All reviewed patches have been landed. Closing bug. |