Last Comment Bug 743941 - B2G Wifi: More small cleanup
: B2G Wifi: More small cleanup
Product: Core
Classification: Components
Component: DOM: Device Interfaces (show other bugs)
: Trunk
: ARM Gonk (Firefox OS)
-- normal (vote)
: mozilla14
Assigned To: Blake Kaplan (:mrbkap)
: Andrew Overholt [:overholt]
Depends on:
Blocks: b2g-wifi
  Show dependency treegraph
Reported: 2012-04-10 02:51 PDT by Blake Kaplan (:mrbkap)
Modified: 2012-04-13 01:38 PDT (History)
3 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Proposed fix (5.18 KB, patch)
2012-04-10 02:55 PDT, Blake Kaplan (:mrbkap)
cjones.bugs: review+
Details | Diff | Splinter Review

Description User image Blake Kaplan (:mrbkap) 2012-04-10 02:51:50 PDT
I have a small patch here that does the following:
- Always updates dhcpInfo so that it's more likely to be correct.
- Updates to be correct in more places.
- Moves connection logic out of the wifi manager and into the code that does the rest of the connection logic stuff. The idea is that the WifiManager is responsible for sending notifications and holds a bit of state while the actual logic for controlling the supplicant lives in the WifiController. I fear that this is always going to be a leaky abstraction, but at least it'll help group the related code together.
Comment 1 User image Blake Kaplan (:mrbkap) 2012-04-10 02:55:13 PDT
Created attachment 613531 [details] [diff] [review]
Proposed fix
Comment 2 User image Blake Kaplan (:mrbkap) 2012-04-13 01:38:29 PDT

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