Closed
Bug 961866
Opened 11 years ago
Closed 6 years ago
Fix race condition that prevents using OS.File in the startup path in Webapps.jsm
Categories
(Core Graveyard :: DOM: Apps, defect)
Core Graveyard
DOM: Apps
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: marco, Unassigned)
Details
There's a race condition that prevents using OS.File in the startup path in Webapps.jsm.
I think the problem is that we should wait for the registry to be loaded before using it.
In the webapp runtime we're waiting: http://mxr.mozilla.org/mozilla-central/source/webapprt/Startup.jsm#104
So I think the problem is restricted to b2g.
(You can see the problem in this try run: https://tbpl.mozilla.org/?tree=Try&rev=7e86c439da11. I've just replaced a couple of nsIFile::exists with OS.File.exists)
Registry loading is asynchronous anyway, so this problem could also arise now, without OS.File (I guess with OS.File it's more noticeable because OS.File may be slower).
Comment 1•11 years ago
|
||
Performance note: we are just realizing that OS.File is *very* slow to load (actually, ChromeWorkers are all very slow to load). Be careful if you use it during critical startup path.
Reporter | ||
Comment 2•11 years ago
|
||
(In reply to David Rajchenbach Teller [:Yoric] (please use "needinfo?") from comment #1)
> Performance note: we are just realizing that OS.File is *very* slow to load
> (actually, ChromeWorkers are all very slow to load). Be careful if you use
> it during critical startup path.
Is there an alternative? Are there any plans to improve this? (bug #?)
The race condition is worth fixing anyway I guess. Right now it shows up only with OS.File, but it could show up when a lot of applications are installed (because at startup we read all their manifests).
Updated•7 years ago
|
Product: Core → Core Graveyard
Comment 3•6 years ago
|
||
Core Graveyard / DOM: Apps is inactive. Closing all bugs in this component.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•