1437825 - browser_urlbar_keyed_search_reflows.js and browser_urlbar_search_reflows.js failing locally - unexpected uninterruptible reflow @_onLazyResize

Status: RESOLVED FIXED

(Firefox :: Address Bar, enhancement)

Description

[Florian Quèze [:florian]]

browser_urlbar_keyed_search_reflows.js and browser_urlbar_search_reflows.js are failing locally for me on Mac, and mconley confirmed he's experiencing the same behavior.

It turns out CustomizableUI.jsm runs some code doing sync reflows off of a 200ms timer after receiving a resize event. We should force this to happen immediately being sending events to the urlbar.

Comment 1

[Florian Quèze [:florian]]

Attachment: Patch

Comment 2

[Mike Conley (:mconley) (:⚙️)]

Comment on attachment 8950545 [details] [diff] [review]
Patch

Review of attachment 8950545 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks, florian! I have one suggestion, see below.

Also, I suspect the reflows you mentioned are tracked in bug 1358388.

::: browser/base/content/test/performance/browser_urlbar_keyed_search_reflows.js
@@ +122,5 @@
>   */
>  add_task(async function() {
>    let win = await BrowserTestUtils.openNewBrowserWindow();
>    await ensureNoPreloadedBrowser(win);
> +  ensureNoPendingLazyResize(win);

Perhaps we should combine this with ensureNoPreloadedBrowser into a single function, like prepareSettledWindow or something.

Comment 3

[Florian Quèze [:florian]]

(In reply to Mike Conley (:mconley) (:⚙️) (Totally backlogged on reviews and needinfos) from comment #2)

> Also, I suspect the reflows you mentioned are tracked in bug 1358388.

The reflows annoying me here are:
	"_onLazyResize@resource:///modules/CustomizableUI.jsm:4445:1",
	"_runTask@resource://gre/modules/DeferredTask.jsm:308:15",
	"async*_timerCallback/<@resource://gre/modules/DeferredTask.jsm:278:13",
	"async*_timerCallback@resource://gre/modules/DeferredTask.jsm:276:30",

That's https://searchfox.org/mozilla-central/rev/9d47b5fb14152865ad1375eb3ee2e571d81ecdb9/browser/components/customizableui/CustomizableUI.jsm#4445
    if (this._target.scrollLeftMin != this._target.scrollLeftMax) {
so not the _moveItemsBackToTheirOrigin function covered by bug 1358388.

Comment 4

[Florian Quèze [:florian]]

Attachment: Patch v2

Comment 5

[Mike Conley (:mconley) (:⚙️)]

Comment on attachment 8950795 [details] [diff] [review]
Patch v2

Review of attachment 8950795 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks!

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5654cb32980e
browser_urlbar_*search_reflows.js tests should force CustomizableUI.jsm's deferred resize code to run before the actual tests starts, r=mconley.

Comment 7

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/5654cb32980e