Closed Bug 1019372 Opened 6 years ago Closed 6 years ago

[bluetooth2] Avoid accessing static variables on different thread

Categories

(Firefox OS Graveyard :: Bluetooth, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ben.tian, Assigned: ben.tian)

References

Details

(Whiteboard: [webbt-api])

Attachments

(6 files)

Bug 1015826 avoids accessing static variables on different thread. The fix should be also applied on bluetooth2.
Whiteboard: [webbt-api]
Blocks: 1019376
Attachment #8433018 - Flags: review?(echou) → review+
Attachment #8433019 - Flags: review?(echou) → review+
Attachment #8433020 - Flags: review?(echou) → review+
Attachment #8433022 - Flags: review?(echou) → review+
Attachment #8433023 - Flags: review?(echou) → review+
Comment on attachment 8433026 [details] [diff] [review]
Patch 6/6 (v1): [bluetooth2] Dispatch part of DiscoveryChangedCallback to main thread

Review of attachment 8433026 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM.
Attachment #8433026 - Flags: review?(echou) → review+
Set 'checkin-needed' for patch 1 ~ 6.
Keywords: checkin-needed
Blocks: 1020300
Do you have a link to a recent Try push handy by chance? :)
Keywords: checkin-needed
The patches don't affect tpbl result since the affected files are not built yet. They are WIP files for new bluetooth API.
You need to log in before you can comment on or make changes to this bug.