786283 - Wifi: Report when a scan attempt fails.

Status: RESOLVED FIXED

(Core :: DOM: Device Interfaces, defect, P1)

Description

[Blake Kaplan (:mrbkap) (inactive)]

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.

Comment 1

[Blake Kaplan (:mrbkap) (inactive)]

Attachment: Fix

Comment 2

[Blake Kaplan (:mrbkap) (inactive)]

Attachment: Additonal fixes on otp of attachment 656004

Oops, there were a couple of dumb bugs in the previous patch.

Comment 3

[Vincent Chang[:vchang][changyihsin]]

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.

Comment 4

[Blake Kaplan (:mrbkap) (inactive)]

https://hg.mozilla.org/mozilla-central/rev/cd7ccc95cefa
https://hg.mozilla.org/mozilla-central/rev/61faabceedd4

Comment 5

[Blake Kaplan (:mrbkap) (inactive)]

Actually, wpa_supplicant is totally broken -- it doesn't respond at all to scan requests.

Comment 6

[Blake Kaplan (:mrbkap) (inactive)]

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.

Comment 7

[Blake Kaplan (:mrbkap) (inactive)]

This needs to wait until https://github.com/mozilla-b2g/gaia/pull/4081 lands.

Comment 8

[Blake Kaplan (:mrbkap) (inactive)]

https://hg.mozilla.org/mozilla-central/rev/b6a1295da486