Chris Jones [:cjones] inactive; ni?/f?/r? if you need me Reporter
	Description • 13 years ago

Blake has a patch that works but needs a bit of cleanup.

	Blake Kaplan (:mrbkap) (inactive) Assignee
	Comment 1 • 13 years ago

This is mostly cleanup (and some preparation for the eventual API) but it's nice to have real objects so we can say "net.flags" instead of "net[1]".

	Blake Kaplan (:mrbkap) (inactive) Assignee
	Comment 2 • 13 years ago

This is what was missing last night: we don't want to force anything until we know that the network doesn't actually exist.

	Blake Kaplan (:mrbkap) (inactive) Assignee
	Comment 3 • 13 years ago

This is basically the patch from last night, but a little cleaned up.

	Blake Kaplan (:mrbkap) (inactive) Assignee
	Comment 4 • 13 years ago

It turns out the INACTIVE notification races with the scan results. This is a diff -w.

	Chris Jones [:cjones] inactive; ni?/f?/r? if you need me Reporter
	Comment 5 • 13 years ago

Comment on attachment 597797 [details] [diff] [review]
Make networks objects

>diff --git a/dom/wifi/nsIWifi.idl b/dom/wifi/nsIWifi.idl

>+[scriptable, uuid(14c815f0-e9db-41d4-a15e-f3e69140f83b)]
>+interface nsIWifiNetwork : nsISupports {
>+    readonly attribute DOMString ssid; // can be null
>+    readonly attribute DOMString bssid; // can be null
>+    readonly attribute DOMString flags; // TODO make this be real flags instead of a string
>+    readonly attribute long signal;
>+};
>+

Do these need to be exposed through XPCOM?  It doesn't seem to be used
in your patches.  Is there another reason for doing this?

r=me pending xpcom question

	Chris Jones [:cjones] inactive; ni?/f?/r? if you need me Reporter
	Comment 6 • 13 years ago

Comment on attachment 597802 [details] [diff] [review]
Force a connection to Mozilla Guest

>diff --git a/dom/wifi/nsWifiWorker.js b/dom/wifi/nsWifiWorker.js
>index e1d48d2..be83da2 100644

>+      var config = Object.create(null);

let?

	Blake Kaplan (:mrbkap) (inactive) Assignee
	Comment 7 • 13 years ago

(In reply to Chris Jones [:cjones] [:warhammer] from comment #5)
> Do these need to be exposed through XPCOM?  It doesn't seem to be used
> in your patches.  Is there another reason for doing this?

It's not used yet. However, when we want to expose Wifi state in order to write, e.g. the UI for selecting a network, we're going to need information like this in the DOM, which means exposing it through XPCOM.

	Blake Kaplan (:mrbkap) (inactive) Assignee
	Comment 8 • 13 years ago

(In reply to Chris Jones [:cjones] [:warhammer] from comment #6)
> >+      var config = Object.create(null);
> 
> let?

let at the top level is equivalent to var (in the parser, we actually change the token) and given that I've used var basically everywhere else in the file, I'd prefer to keep this as-is.

	Blake Kaplan (:mrbkap) (inactive) Assignee
	Comment 9 • 13 years ago

Comment on attachment 597797 [details] [diff] [review]
Make networks objects

https://hg.mozilla.org/integration/mozilla-inbound/rev/c9f5f7c81b48

	Blake Kaplan (:mrbkap) (inactive) Assignee
	Comment 10 • 13 years ago

Comment on attachment 597800 [details] [diff] [review]
Wait to connect until we have scan results

https://hg.mozilla.org/integration/mozilla-inbound/rev/337bf2beee17

	Blake Kaplan (:mrbkap) (inactive) Assignee
	Comment 11 • 13 years ago

Comment on attachment 597802 [details] [diff] [review]
Force a connection to Mozilla Guest

https://hg.mozilla.org/integration/mozilla-inbound/rev/a93f5dd0dd57

	Blake Kaplan (:mrbkap) (inactive) Assignee
	Comment 12 • 13 years ago

Comment on attachment 597903 [details] [diff] [review]
Fix another race

https://hg.mozilla.org/integration/mozilla-inbound/rev/7d8ae4b53ead

	Ed Morley [:emorley]
	Comment 13 • 13 years ago

https://hg.mozilla.org/mozilla-central/rev/c9f5f7c81b48
https://hg.mozilla.org/mozilla-central/rev/337bf2beee17
https://hg.mozilla.org/mozilla-central/rev/a93f5dd0dd57
https://hg.mozilla.org/mozilla-central/rev/7d8ae4b53ead