Closed Bug 712629 Opened 10 years ago Closed 10 years ago

Make Wifi work under gonk

Categories

(Core :: DOM: Device Interfaces, defect)

ARM
Android
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla12

People

(Reporter: mrbkap, Assigned: mrbkap)

References

Details

Attachments

(1 file, 1 obsolete file)

Attached patch wip patch (obsolete) — 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)
Attached patch wip patchSplinter Review
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 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+
https://hg.mozilla.org/mozilla-central/rev/8a498f3bae5c
Status: ASSIGNED → RESOLVED
Closed: 10 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?
No longer blocks: 717122
Blocks: 729877
No longer blocks: 729877
You need to log in before you can comment on or make changes to this bug.