XBL Replacement Newsletter #5

[Brian Grinstead]

This is the fifth edition of the XBL Replacement Newsletter. Since the last update we’ve moved below 200 bindings, so we’ve removed just over one third of the bindings. We’ve also been making progress on in-content bindings and doing some cleanup that’s been enabled by the work so far. The easiest way to follow along with this work is to watch the main meta bug.

In Content XBL

Stopping use of XBL in the content process will be a big milestone for the project, and it’ll let us remove a lot of complexity that exists to make these elements secure. We’ve come up with an outline for how to replace in-content bindings that require DOM/JS in the content process: the basic idea is to use a “User Agent” Shadow DOM that’s only accessible to chrome code and to run the JS in a new limited privileged scope. You can see a more detailed proposal on the bug.

In addition to this, we’ve been making progress on removing and simplifying the current in-content bindings:

• We no longer use XBL for scrollbars! Tim removed the scrollbar binding, and we instead build up native anonymous content to render them. This unblocks further improvements such as using a single element instead of a full DOM tree for scrollbars.
• The “resizer” is the little grippy icon that shows up on the corner of textareas to let you resize them. Tim removed the resizer binding and we now use native anonymous content instead.
• touchControls and suppressChangeEvent were separate bindings for mobile that extended the videocontrols binding. Tim removed them and now we now share a single binding with different styling for mobile.
• XML pretty printing used to require JS to expand and collapse tags. Ian Moody changed this by rendering the tree with HTML <details> and <summary> tags instead. We still use a XBL binding to mount the content into the page, but there’s a bug on file to stop doing that.

Cleanup

• There was a feature called NODE_FORCE_XBL_BINDINGS that would eagerly attach XBL to a cloned node even before it was added to the DOM. We no longer rely on that feature, so it was removed.
• We no longer load xul.css in content documents. This used to get loaded when we rendered touchControls, among other things. We still load minimal-xul.css in all content documents, but there’s a new metabug tracking removal of in-content xul to ratchet this down.
• We’ve been removing nsIDOMXUL* stuff, which also aligns with :bz’s work to remove nsIDOM* stuff. We removed nsIDOMXULTreeElement, nsIDOMXULTextBoxElement, and everything but one attribute from nsIDOMXULCheckboxElement.
• Neil simplified the XBL implementation of popups by allowing nsXULElement to be subclassed and then using a subclass for menupopup, popup, panel, and tooltip elements.
• There have been some nice cleanups to the gBrowser object since migrating away from XBL. Dão did a bunch of refactoring, which can be seen in bugs inside the ‘Blocks’ field of the JS migration bug. We also saw a tabpaint perf win when deleting the tabbrowser binding.

Binding Removals

There are 196 more bindings left, compared to 215 from the last update and 300 from the start of the project. Here’s a list of changes:

• Dão removed the menubar binding and cleaned up some redundant styling in the meantime.
• Dão also removed the statuspanel binding, by moving the implementation into a plain JS object inside tabbrowser.js.
• Edouard Oger removed the menulist-base binding, by loading the CSS file inside of components.css instead of with XBL <resources>.
• I removed the sorters binding used in about:addons in order to unblock simplification of nsIDOMXULCheckboxElement.
• I also removed the container binding from about:preferences, using a normal <richlistitem> instead.
• I also removed the offlineapp binding, which was unused.
• Paolo removed the filefield binding by using a normal <textbox> with custom styling.
• Paolo also removed the button-repeat binding, which was unused.
• Paolo also removed the toolbarbutton-menu-button binding, which was unused.
• Potch removed the splitter binding, by loading the CSS file inside of components.css instead of with XBL <resources>.
• Tim removed a bunch of in-content bindings: scrollbar, suppressChangeEvent, touchControls, and resizer.
• Tim also removed the grippy binding, which was unused.
• Tim also removed the groupbox-base binding by loading the CSS inside of components.css instead of with XBL <resources>.