Closed
Bug 818234
Opened 12 years ago
Closed 12 years ago
Quickly turning bluetooth off then on results in no device scan
Categories
(Firefox OS Graveyard :: Gaia::Settings, defect, P1)
Tracking
(blocking-basecamp:+)
People
(Reporter: ggrisco, Assigned: jj.evelyn)
Details
(Whiteboard: [LOE=S])
Attachments
(1 file, 1 obsolete file)
1. On test device, turn bluetooth off then back on, quickly
expectation: if there are other BT devices in the field then we should see them displayed in the list
reality: the device list is empty. Even paired devices do not show up.
Updated•12 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 1•12 years ago
|
||
Confirmed Unagi 12/4 build. I was able to reproduce this, as well as freeze the entire phone while toggling Bluetooth.
Assignee: nobody → echou
blocking-basecamp: ? → +
Priority: -- → P1
Target Milestone: --- → B2G C2 (20nov-10dec)
Comment 2•12 years ago
|
||
Seems like a platform issue. Moving the component.
Component: Gaia::Settings → DOM: Device Interfaces
Product: Boot2Gecko → Core
Comment 3•12 years ago
|
||
After the user turn Bluetooth on / off, low-level Bluetooth implementation needs few seconds to actually set up Bluetooth chip, and events will be fired once the process is done. So, if the user turn on / off back and forth very quickly, then the UI and the real state of Bluetooth may be out of sync.
The solution for this case is to disable the Bluetooth switch button until receiving signals sent from Gecko. After discussion with Evelyn, she will take over this issue.
Eric
Updated•12 years ago
|
Assignee: echou → ehung
Comment 4•12 years ago
|
||
Which means it's back in Gaia, switching.
Component: DOM: Device Interfaces → Gaia::Settings
Product: Core → Boot2Gecko
Assignee | ||
Updated•12 years ago
|
Whiteboard: [LOE=S]
Assignee | ||
Comment 5•12 years ago
|
||
working on it, will provide a patch today.
Assignee | ||
Comment 6•12 years ago
|
||
prevent quickly tapping on bluetooth and wifi settings
Attachment #689583 -
Flags: review?(timdream+bugs)
Assignee | ||
Comment 7•12 years ago
|
||
In my patch above, I didn't spend too much time in CSS tweak, Rudy will help me on the switch style, and submit a follow-up patch here. Thanks.
Comment 8•12 years ago
|
||
Comment on attachment 689583 [details]
see https://github.com/mozilla-b2g/gaia/pull/6875
Please update the fix and re-request for review again. Thanks.
Attachment #689583 -
Flags: review?(timdream+bugs)
Comment 9•12 years ago
|
||
This is a C2 bug, which is due to Monday.
I am told by Evelyn that Rudy is helping on the CSS part. Can we meet C2 deadline on that? If not, we would have to land without that part. I still hope both can land at the same time on the same bug though.
Flags: needinfo?(rlu)
Comment 11•12 years ago
|
||
Comment on attachment 690084 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/6905
This is a follow-up patch for defining disabled style for switches in building blocks.
Thanks.
Attachment #690084 -
Flags: review?(timdream+bugs)
Assignee | ||
Comment 12•12 years ago
|
||
Comment on attachment 689583 [details]
see https://github.com/mozilla-b2g/gaia/pull/6875
add Rudy's commit in, please take a look again. Thanks!
Attachment #689583 -
Flags: review?(timdream+bugs)
Assignee | ||
Comment 13•12 years ago
|
||
Comment on attachment 690084 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/6905
cancel this review request because it has been included above.
Attachment #690084 -
Attachment is obsolete: true
Attachment #690084 -
Flags: review?(timdream+bugs)
Comment 14•12 years ago
|
||
Thank you both! Let's verify the visual first thing Monday morning :)
Comment 15•12 years ago
|
||
Comment on attachment 689583 [details]
see https://github.com/mozilla-b2g/gaia/pull/6875
r=me, thanks Rudy and Evelyn.
Attachment #689583 -
Flags: review?(timdream+bugs) → review+
Assignee | ||
Comment 16•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
QA Contact: wachen
Comment 17•12 years ago
|
||
Fixed in https://releases.mozilla.com/b2g/ 20121212 build
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•