Closed
Bug 712629
Opened 12 years ago
Closed 12 years ago
Make Wifi work under gonk
Categories
(Core :: DOM: Device Interfaces, defect)
Tracking
()
RESOLVED
FIXED
mozilla12
People
(Reporter: mrbkap, Assigned: mrbkap)
References
Details
Attachments
(1 file, 1 obsolete file)
56.45 KB,
patch
|
gal
:
review+
mrbkap
:
checkin+
|
Details | Diff | Splinter Review |
Here's a patch to Gecko that seems to work under gonk. There's still some unexplained business on my phone where I'm not sure why wifi seemed to magically start working. I'm going to see if this works on other phones and if not, I'll debug further.
Attachment #583477 -
Flags: review?(gal)
Assignee | ||
Comment 1•12 years ago
|
||
With this patch we connect to a hardcoded network and properly bring up dhcp and dns. There's pretty clearly a bunch of refactoring/cleaning up to do, but this actually appears to work.
Attachment #583477 -
Attachment is obsolete: true
Attachment #583477 -
Flags: review?(gal)
Attachment #584443 -
Flags: review?(gal)
Comment 2•12 years ago
|
||
Comment on attachment 584443 [details] [diff] [review] wip patch Review of attachment 584443 [details] [diff] [review]: ----------------------------------------------------------------- Looks very nice. Lets land this and we can work on the high-level wifi control next (look for configured available networks, associated in order, see if we have a connection, if not move to the next, keep pinging network to make sure its available, if not reconfigure, etc). We should file a bug for that. We also need a DOM API for editing networks and wifi on/off (bug for that too?). ::: dom/system/b2g/RadioManager.cpp @@ +253,5 @@ > NS_ASSERTION(NS_IsMainThread(), "Wrong thread!"); > > StopRil(); > mTelephone = nsnull; > + mWifi = nsnull; RadioManager isn't the best name for this class any more. We should rename it down the road.
Attachment #584443 -
Flags: review?(gal) → review+
Assignee | ||
Comment 3•12 years ago
|
||
Comment on attachment 584443 [details] [diff] [review] wip patch https://hg.mozilla.org/integration/mozilla-inbound/rev/8a498f3bae5c Bug 711671 covers the renaming you talked about.
Attachment #584443 -
Flags: checkin+
Comment 4•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/8a498f3bae5c
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
Where's the followup work for this being tracked? Part of the "network manager", bug 713199 and bug 697355, or something else?
Updated•12 years ago
|
Blocks: b2g-network-manager
Updated•12 years ago
|
No longer blocks: b2g-network-manager
You need to log in
before you can comment on or make changes to this bug.
Description
•