Closed
Bug 786283
Opened 12 years ago
Closed 12 years ago
Wifi: Report when a scan attempt fails.
Categories
(Core :: DOM: Device Interfaces, defect, P1)
Tracking
()
RESOLVED
FIXED
People
(Reporter: mrbkap, Assigned: mrbkap)
References
Details
Attachments
(2 files)
(deleted),
patch
|
vchang
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
vchang
:
review+
|
Details | Diff | Splinter Review |
Right now, if scanning fails (e.g. because the user requested a scan while we were associating) the wifi manager doesn't inform the DOM that the scan request didn't work, leading to a situation where we're waiting for a scan that wasn't scheduled. We could reschedule the scan request in the backend, but kaze said that he'd prefer doing so in the frontend, which makes sense to me.
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #656004 -
Flags: review?(vchang)
Assignee | ||
Comment 2•12 years ago
|
||
Oops, there were a couple of dumb bugs in the previous patch.
Attachment #656162 -
Flags: review?(vchang)
Comment 3•12 years ago
|
||
Comment on attachment 656162 [details] [diff] [review]
Additonal fixes on otp of attachment 656004 [details] [diff] [review]
Review of attachment 656162 [details] [diff] [review]:
-----------------------------------------------------------------
It looks really good.
Attachment #656162 -
Flags: review?(vchang) → review+
Updated•12 years ago
|
Attachment #656004 -
Flags: review?(vchang) → review+
Assignee | ||
Comment 4•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/cd7ccc95cefa
https://hg.mozilla.org/mozilla-central/rev/61faabceedd4
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 5•12 years ago
|
||
Actually, wpa_supplicant is totally broken -- it doesn't respond at all to scan requests.
Resolution: FIXED → INVALID
Assignee | ||
Comment 6•12 years ago
|
||
Oops, so, wpa_supplicant isn't totally broken, the old version was, but we don't use it anymore. I was seeing a gaia bug.
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Assignee | ||
Comment 7•12 years ago
|
||
This needs to wait until https://github.com/mozilla-b2g/gaia/pull/4081 lands.
Assignee | ||
Comment 8•12 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 12 years ago → 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•