Open
Bug 1469862
Opened 7 years ago
Updated 3 years ago
Sync notifications should be main-thread only
Categories
(Firefox for iOS :: General, enhancement, P3)
Tracking
()
NEW
| Tracking | Status | |
|---|---|---|
| fxios | + | --- |
People
(Reporter: garvan, Unassigned)
Details
I saw this sync-related code sending an NSNotification on the background thread:
https://github.com/mozilla-mobile/firefox-ios/blob/master/Providers/Profile.swift#L659
NSNotifications should be main-thread only due their lack of containment/modularity, i.e. hard to safeguard their use to the correct thread only.
|
||
Updated•7 years ago
|
Priority: -- → P3
|
||
Updated•3 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•