Open Bug 920083 Opened 11 years ago Updated 8 months ago

nsAppStartup's crash tracking code causes major UI janks

Categories

(Toolkit :: Startup and Profile System, defect)

17 Branch
defect

Tracking

()

People

(Reporter: mayhemer, Unassigned)

References

Details

(Keywords: perf)

nsAppStartup::TrackStartupCrashBegin and nsAppStartup::TrackStartupCrashEnd calls nsIPrefService->SavePrefFile() on the main thread.  This is causing huge visible janks since it often happens during a page load.

No idea for a simple fix, but apparently pref service needs to be more asynchronous when saving the pref file.
I filed bug 888373 to provide an asynchronous alternative.
Could someone take a look at the proposed API and tell me whether this API could provide a suitable replacement?
(In reply to David Rajchenbach Teller [:Yoric] from comment #1)
> I filed bug 888373 to provide an asynchronous alternative.
> Could someone take a look at the proposed API and tell me whether this API
> could provide a suitable replacement?

IIUC, the proposal there doesn't start early enough (see bug 888373 comment 10 and bug 888373 comment 30).

In the process of migrating remaining bugs to the new severity system, the severity for this bug cannot be automatically determined. Please retriage this bug using the new severity system.

Severity: major → --

The severity field is not set for this bug.
:mossop, could you have a look please?

For more information, please visit BugBot documentation.

Flags: needinfo?(dtownsend)
Severity: -- → S3
Flags: needinfo?(dtownsend)
You need to log in before you can comment on or make changes to this bug.