This is specified here: http://dvcs.w3.org/hg/webcomponents/raw-file/tip/spec/shadow/index.html#lower-boundary-encapsulation The key is that if the element is inactive, it should act as any HTML element: be stylable and show its contents. A good example of why this is good is http://dglazkov.github.com/Web-Components-Polyfill/news.html, which is a component definition (used here: http://dglazkov.github.com/Web-Components-Polyfill/). I added a special debug.css, which helps understand the structure of the component. If <content> were visible, I could style it and put "CONTENTS GO HERE" label with a selector or something. That's instant debugging for a component developer :)
Looks like this came from http://crbug.com/114667
Created attachment 129169 [details] Patch
Created attachment 129200 [details] Patch
Created attachment 129207 [details] Catching up ToT
Comment on attachment 129207 [details] Catching up ToT View in context: https://bugs.webkit.org/attachment.cgi?id=129207&action=review > Source/WebCore/dom/ShadowRoot.h:104 > +// Put this TreeScope method here to inline it. Is it really so hot that we need to do this? > Source/WebCore/dom/ShadowRoot.h:105 > + Probably don't need this extra blank linke.
Comment on attachment 129207 [details] Catching up ToT View in context: https://bugs.webkit.org/attachment.cgi?id=129207&action=review Hi Dimitri, thanks for the review! I'll land this after fixing your point. >> Source/WebCore/dom/ShadowRoot.h:104 >> +// Put this TreeScope method here to inline it. > > Is it really so hot that we need to do this? Since this is in every attach path, I want to minimize the impact. >> Source/WebCore/dom/ShadowRoot.h:105 >> + > > Probably don't need this extra blank linke. Sure.
Committed r109179: <http://trac.webkit.org/changeset/109179>