1523997 - Don't get computed styles in Element::WrapObject in the content process (no need to check for XBL binding)

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, task)

Description

[Brian Grinstead [:bgrins]]

We could either early return in WrapObject (https://searchfox.org/mozilla-central/rev/78cd247b5d7a08832f87d786541d3e2204842e8e/dom/base/Element.cpp#554) or return false in MayNeedToLoadXBLBinding (https://searchfox.org/mozilla-central/rev/78cd247b5d7a08832f87d786541d3e2204842e8e/dom/base/Element.cpp#508) when in the content process (XRE_IsContentProcess()).

The effect would be we don't enter this block which needs to read computed style to check for -moz-binding: https://searchfox.org/mozilla-central/rev/78cd247b5d7a08832f87d786541d3e2204842e8e/dom/base/Element.cpp#531-536.

Comment 1

[Brian Grinstead [:bgrins]]

I'll do a try push, but I think this will break various tests in automation that load XUL/XBL in the content process. Similar to how https://phabricator.services.mozilla.com/D18027 (bug 1523712) needs to allow -moz-binding in tests via a pref.

Comment 2

[Brian Grinstead [:bgrins]]

FWIW in Bug 1465457 we're planning to load chrome/XUL specific reftests in the parent process, but I'm guessing this is an issue across more than just reftests.

Comment 3

[Brian Grinstead [:bgrins]]

(In reply to Brian Grinstead [:bgrins] from comment #1)

I'll do a try push, but I think this will break various tests in automation that load XUL/XBL in the content process. Similar to how https://phabricator.services.mozilla.com/D18027 (bug 1523712) needs to allow -moz-binding in tests via a pref.

Hm, this is surprisingly green: https://treeherder.mozilla.org/#/jobs?repo=try&revision=4f3234dc68433caf2091375c7747fbc70c0d7cfd.

Comment 4

[Brian Grinstead [:bgrins]]

Attachment: Bug 1523997 - Skip checking for -moz-binding in Element::WrapObject when in the content process

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/28cd229556d1
Skip checking for -moz-binding in Element::WrapObject when in the content process r=bzbarsky

Comment 6

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/28cd229556d1