1319165 - RemotePageManager showing up in tab spinner hangs when doing .replace on URLs

Status: NEW

(Toolkit :: General, defect)

Description

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

Check out the tab switch spinner stacks I just posted to bug 1310255 comment 20 (organized by total hang time).

The top entry has this stack:

"Startup::XRE_InitChildProcess",
"PBrowser::Msg_LoadRemoteScript",
"global/content/browser-child.js:1",
"gre/modules/RemotePageManager.jsm:506"

That corresponds to this line, I believe:

http://searchfox.org/mozilla-central/rev/904bf9addd03b03d4cad11b82f19f43d875b7f27/toolkit/modules/RemotePageManager.jsm#506

Not sure if it's just bad luck that we're happening to register a hang here. It's possible that for some types of URLs, this type of replace is quite slow (perhaps data URIs for example?).

We can probably optimize RemotePageManager.jsm a bit here.

Comment 1

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

Attachment: Bug 1319165 - Make RemotePageManager only work for about: URLs.

Review commit: https://reviewboard.mozilla.org/r/94774/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/94774/