Bug 20172 – [XBL] XBL elements should not be rendered

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Julien Chaffraix]

Currently XBL elements are shown on the screen and they should not.

[Dave Hyatt]

You can just add style rules for xbl to html4.css.  Should only take two...

xbl|* { display: none }
xbl|div { display: block }

[Julien Chaffraix]

Created attachment 22477 [details]
Proposed fix: create the render but remove the root from the hierarchy

This gist of this patch is to let attach() create the renderers for us so that we can use them to generate our shadow tree later. In order for them not to be displayed, we remove the renderer at the root of the hierarchy (which is the <xbl> element renderer).

[Dave Hyatt]

Comment on attachment 22477 [details]
Proposed fix: create the render but remove the root from the hierarchy

Renderers for shadow content are going to be created when you clone the shadow content templates.  The original tree is not going to have any renderers for the shadow content template.  Only the clones will have renderers.

In order to just make sure the main XBL document tree has no renderers, you just need a couple of style rules added to html4.css.

These style rules are described in the spec:

http://www.w3.org/TR/xbl/#xbl-elements

@namespace xbl url(http://www.w3.org/ns/xbl);
xbl|* { display: none; }
xbl|div { display: block; }

[Julien Chaffraix]

Created attachment 22483 [details]
Updated version: using html4.css

[Dave Hyatt]

Comment on attachment 22483 [details]
Updated version: using html4.css

r=me

[Julien Chaffraix]

Landed in r35372 on XBL2 branch.