1534407 - Enable browser.xhtml by default

Status: RESOLVED FIXED

(Firefox :: General, enhancement, P3)

Description

[Brian Grinstead [:bgrins]]

After everything is green, we'll need to:

1. Provide a default value as true for the MOZ_BROWSER_XHTML define (presumably still allowing it to be explicitly set to false for testing purposes): https://searchfox.org/mozilla-central/rev/8ff2cd0a27e3764d9540abdc5a66b2fb1e4e9644/browser/base/moz.build#64
2. Write a XULStore migration from browser.xul to browser.xhtml: https://searchfox.org/mozilla-central/rev/8ff2cd0a27e3764d9540abdc5a66b2fb1e4e9644/browser/components/BrowserGlue.jsm#2266. That'd be something like the xulStore.getValue(BROWSER_DOCURL, ...) calls already in there, but we'll want to hardcode "chrome://browser/content/browser.xul" as the source and "chrome://browser/content/browser.xhtml" as the target of the copying. I guess we'll want to use getAttributeEnumerator and loop over browser.xul in the store and write them into browser.xhtml. Maybe we should expose a XULStore function to wrap up this "document migration" case since we could want it for future xul->xhtml migrations.

Comment 1

[:Gijs (he/him)]

Do we intend to do something like this prior to being convinced we can make it ride that train? I'd be pretty worried about making nightly quite that different from what we end up shipping...

Comment 2

[Brian Grinstead [:bgrins]]

(In reply to :Gijs (he/him) from comment #1)

Do we intend to do something like this prior to being convinced we can make it ride that train? I'd be pretty worried about making nightly quite that different from what we end up shipping...

Agreed, and this would happen only after we are confident it will ship. Right now we are using try and talos as the goalposts for this, so if they are both green after the merge I will propose we do it. Do you know of other criteria we should be tracking or sign-off we should be seeking before we land?

Comment 3

[:Gijs (he/him)]

I haven't been following this in detail, so I'm not really the right person to ask. There's some open deps on bug 1533881 that I guess we should ensure we feel are either not blockers to ship or we're very confident are going to be resolved post-merge? The a11y angle seems like a blocker; we can't ship a Firefox that a good portion of our users can't use anymore...

Comment 4

[Brian Grinstead [:bgrins]]

(In reply to :Gijs (he/him) from comment #3)

I haven't been following this in detail, so I'm not really the right person to ask. There's some open deps on bug 1533881 that I guess we should ensure we feel are either not blockers to ship or we're very confident are going to be resolved post-merge? The a11y angle seems like a blocker; we can't ship a Firefox that a good portion of our users can't use anymore...

Of course. I thought that was obvious - sorry that it wasn't clear. We aren't planning to land this before we get the sign off from accessibility team in Bug 1488918, and have resolutions for other important blockers. Let me reorganize this - changing this from blocking the meta to depending on the meta.

Comment 5

[Brian Grinstead [:bgrins]]

Did a talos push and seeing a few medium confidence ts_paint regressions along with one strong on startup_about_home_paint opt e10s stylo (although that one is QR and I don't know how stable those results are now): https://treeherder.mozilla.org/perf.html#/compare?originalProject=try&originalRevision=015844ab39ebe7f836c6647ee9392592a49cedb4&newProject=try&newRevision=590b11ff486fd121052604db2d5f6e15eccc1a6e&framework=1.

So there may still be a bit more to do here on perf, but it's way closer than it was pre Bug 1527977.

Comment 6

[Brian Grinstead [:bgrins]]

Attachment: Bug 1534407 - Enable browser.xhtml by default

Comment 8

[Brian Grinstead [:bgrins]]

Bug 1552058 is spinning out some of the patch here for landing separately, so I'll rebase around that once it lands.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ad38bcbe7879
Enable browser.xhtml by default r=mossop

Comment 10

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/ad38bcbe7879