Closed Bug 1193058 Opened 10 years ago Closed 10 years ago

loadQueuedTabs is called on the main thread

Categories

(Firefox for iOS :: General, defect)

All
iOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
fxios + ---

People

(Reporter: rnewman, Assigned: rnewman)

Details

Attachments

(1 file)

47 bytes, text/x-github-pull-request
bnicholson
: review+
Details | Review
1. That means we create the queue table, and the browser DB, on the main thread. 2. That might mean we're racing, causing crashes. 3. That also might mean we're harming startup performance on multi-core devices. tracking? for those reasons.
Attached file Pull req.
Let's see who gets to this first. Tested restore.
Attachment #8646010 - Flags: review?(sleroux)
Attachment #8646010 - Flags: review?(sarentz)
Attachment #8646010 - Flags: review?(bnicholson)
Comment on attachment 8646010 [details] [review] Pull req. LGTM
Attachment #8646010 - Flags: review?(bnicholson) → review+
Attachment #8646010 - Flags: review?(sleroux)
Attachment #8646010 - Flags: review?(sarentz)
a33d99c
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: