Last Comment Bug 743614 - B2G Wifi: Beef up error checking a little
: B2G Wifi: Beef up error checking a little
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: DOM: Device Interfaces (show other bugs)
: Trunk
: ARM Gonk (Firefox OS)
: -- normal (vote)
: mozilla14
Assigned To: Blake Kaplan (:mrbkap) (please use needinfo!)
:
Mentors:
Depends on:
Blocks: b2g-wifi
  Show dependency treegraph
 
Reported: 2012-04-08 23:38 PDT by Blake Kaplan (:mrbkap) (please use needinfo!)
Modified: 2012-04-13 01:39 PDT (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Proposed fix (2.24 KB, patch)
2012-04-08 23:38 PDT, Blake Kaplan (:mrbkap) (please use needinfo!)
cjones.bugs: review+
Details | Diff | Review

Description Blake Kaplan (:mrbkap) (please use needinfo!) 2012-04-08 23:38:47 PDT
Created attachment 613238 [details] [diff] [review]
Proposed fix

Right now, calling wifi functions on a disabled wifi manager will try to do something but fail in random places. We should fail earlier and give a better error message.

Relatedly we don't handle errors in the case of onscanresultsavailable (which can happen if we turn wifi off between the notification and when we call getScanResults).
Comment 1 Blake Kaplan (:mrbkap) (please use needinfo!) 2012-04-13 01:39:50 PDT
https://hg.mozilla.org/mozilla-central/rev/a3974cd83d6e

Note You need to log in before you can comment on or make changes to this bug.