Summary: | Remove Node::isContentElement and Node::isShadowElement | ||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Shinya Kawanaka <shinyak> | ||||||||||||||||
Component: | DOM | Assignee: | Shinya Kawanaka <shinyak> | ||||||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||||||
Severity: | Normal | CC: | dglazkov, dominicc, gustavo, hayato, morrita, rolandsteiner, shinyak, webkit.review.bot, xan.lopez | ||||||||||||||||
Priority: | P2 | ||||||||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||||||||
Hardware: | Unspecified | ||||||||||||||||||
OS: | Unspecified | ||||||||||||||||||
Bug Depends on: | |||||||||||||||||||
Bug Blocks: | 78596 | ||||||||||||||||||
Attachments: |
|
Description
Shinya Kawanaka
2012-03-07 00:59:24 PST
Created attachment 130585 [details]
Patch
Comment on attachment 130585 [details] Patch Attachment 130585 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/11833954 Ah, symbols are necessary? I'll add them after EWS building windows port. Comment on attachment 130585 [details] Patch Attachment 130585 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/11850020 New failing tests: fast/dom/shadow/content-element-user-agent-shadow.html Comment on attachment 130585 [details] Patch Attachment 130585 [details] did not pass gtk-ews (gtk): Output: http://queues.webkit.org/results/11839973 Comment on attachment 130585 [details]
Patch
Looks great! please fix the build failures before landing.
Created attachment 130747 [details]
Build test
I found this approach is very difficult now. We have RuntimeEnabledFeatures::shadowDOMEnabled() now, and the tagName of <content> is changed by the flag... (In reply to comment #8) > I found this approach is very difficult now. > We have RuntimeEnabledFeatures::shadowDOMEnabled() now, and the tagName of <content> is changed by the flag... I found I was wrong... The content elements in <details> and <summary> should always have HTMLNames::webkitShadowContentTag. Created attachment 130766 [details]
Patch
Comment on attachment 130766 [details] Patch Attachment 130766 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/11863164 Created attachment 130773 [details]
Patch
Created attachment 130776 [details]
Patch
Created attachment 130777 [details]
Patch
Comment on attachment 130777 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=130777&action=review > Source/WebCore/html/shadow/InsertionPoint.h:54 > + virtual bool isInsertionPoint() const { return true; } OVERRIDE here. > Source/WebCore/html/shadow/InsertionPoint.h:77 > + // https://bugs.webkit.org/show_bug.cgi?id=78596 It looks we don't need something special for <shadow>. Created attachment 130780 [details]
Patch
(In reply to comment #16) > Created an attachment (id=130780) [details] > Patch (In reply to comment #15) > (From update of attachment 130777 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=130777&action=review > > > Source/WebCore/html/shadow/InsertionPoint.h:54 > > + virtual bool isInsertionPoint() const { return true; } > > OVERRIDE here. Done. > > > Source/WebCore/html/shadow/InsertionPoint.h:77 > > + // https://bugs.webkit.org/show_bug.cgi?id=78596 > > It looks we don't need something special for <shadow>. Done. Comment on attachment 130780 [details]
Patch
Looks OK.
Committed r110151: <http://trac.webkit.org/changeset/110151> |