889597 - Hide the panelUI panel and overflow panel initially to improve performance

Status: RESOLVED FIXED

(Firefox :: Toolbars and Customization, defect)

Description

[:Gijs (he/him)]

Attachment: Patch

As in summary.

Comment 1

[:Gijs (he/him)]

https://hg.mozilla.org/projects/ux/rev/0b667de52fa4

Comment 2

[:Gijs (he/him)]

Bizarrely, this actually seemed to have regressed OS X 10.7 (but nothing else?) when committed. Trying to find out if that's actually the case:

https://tbpl.mozilla.org/?tree=Try&rev=3c6299b175d7

Comment 3

[:Gijs (he/him)]

(In reply to :Gijs Kruitbosch from comment #2)
> Bizarrely, this actually seemed to have regressed OS X 10.7 (but nothing
> else?) when committed. Trying to find out if that's actually the case:
> 
> https://tbpl.mozilla.org/?tree=Try&rev=3c6299b175d7

Broken try syntax that causes no builds doesn't seem to trigger warnings, but I think after 40 minutes it's safe to say, that didn't do anything. Hopefully this will: https://tbpl.mozilla.org/?tree=Try&rev=c693bc117c07

Comment 4

[:Gijs (he/him)]

(In reply to :Gijs Kruitbosch from comment #3)
> (In reply to :Gijs Kruitbosch from comment #2)
> > Bizarrely, this actually seemed to have regressed OS X 10.7 (but nothing
> > else?) when committed. Trying to find out if that's actually the case:
> > 
> > https://tbpl.mozilla.org/?tree=Try&rev=3c6299b175d7
> 
> Broken try syntax that causes no builds doesn't seem to trigger warnings,
> but I think after 40 minutes it's safe to say, that didn't do anything.
> Hopefully this will: https://tbpl.mozilla.org/?tree=Try&rev=c693bc117c07

So... maybe... not? It would be great if these numbers were a bit more stable. Anyway, (partially) reverting this doesn't seem very promising, so abandoning this idea, at least for now.

Comment 5

[:Gijs (he/him)]

https://hg.mozilla.org/mozilla-central/rev/0b667de52fa4