Closed
Bug 892862
Opened 11 years ago
Closed 11 years ago
[b2g-bluetooth] Failed to receive system message of Sco connection status changed
Categories
(Firefox OS Graveyard :: Bluetooth, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: gyeh, Assigned: gyeh)
References
Details
(Whiteboard: [fixed-in-birch])
Attachments
(2 files, 1 obsolete file)
705 bytes,
patch
|
Details | Diff | Splinter Review | |
1.10 KB,
patch
|
echou
:
review+
|
Details | Diff | Splinter Review |
Per Arthur, when Sco connection is established/destroyed, we didn't notice by system message with type "bluetooth-sco-status-changed". The root cause here is that we should add it in dom/messages/SystemMessagePermissionsChecker.jsm for permission check.
Assignee | ||
Comment 1•11 years ago
|
||
Eric, please help to review this patch. Thanks.
Assignee: nobody → gyeh
Attachment #774474 -
Flags: review?(echou)
Comment 2•11 years ago
|
||
Comment on attachment 774474 [details] [diff] [review] Patch 1(v1): Add SystemMessagePermissionCheck Review of attachment 774474 [details] [diff] [review]: ----------------------------------------------------------------- r=me with nit addressed. Please make sure the debug message won't be enabled before you check it in. Thank you. ::: dom/messages/SystemMessageInternal.js @@ +43,5 @@ > "SystemMessageManager:HandleMessagesDone", > "child-process-shutdown"] > > function debug(aMsg) { > + dump("-- SystemMessageInternal " + Date.now() + " : " + aMsg + "\n"); We should not let this be compiled. Please readd the comment sign.
Attachment #774474 -
Flags: review?(echou) → review+
Assignee | ||
Comment 3•11 years ago
|
||
(In reply to Eric Chou [:ericchou] [:echou] from comment #2) > ::: dom/messages/SystemMessageInternal.js > > + dump("-- SystemMessageInternal " + Date.now() + " : " + aMsg + "\n"); > > We should not let this be compiled. Please readd the comment sign. Agree. I accidentally added thie line into patch. :(
Assignee | ||
Comment 4•11 years ago
|
||
try: https://tbpl.mozilla.org/?tree=Try&rev=2c8541038a2e
Assignee | ||
Comment 5•11 years ago
|
||
Attachment #774474 -
Attachment is obsolete: true
Assignee | ||
Comment 6•11 years ago
|
||
http://hg.mozilla.org/projects/birch/rev/8e88c01ceb0a
Whiteboard: [fixed-in-birch]
Assignee | ||
Comment 7•11 years ago
|
||
Please help to check this patch. Thanks.
Attachment #775589 -
Flags: review?(echou)
Updated•11 years ago
|
Attachment #775589 -
Flags: review?(echou) → review+
Assignee | ||
Updated•11 years ago
|
Attachment #775589 -
Attachment description: Patch 2(v1): Bluetooth-related system messages cleanup → Final patch, part 2, r=echou
Assignee | ||
Comment 8•11 years ago
|
||
Push patch 2 to birch: http://hg.mozilla.org/projects/birch/rev/403531e4ebdf
Comment 9•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/8e88c01ceb0a
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•