Closed
Bug 791392
Opened 12 years ago
Closed 12 years ago
mozWifiManager: expose a list of known networks
Categories
(Core :: DOM: Device Interfaces, enhancement)
Tracking
()
People
(Reporter: kaze, Assigned: mrbkap)
References
Details
(Keywords: uiwanted, Whiteboard: [LOE:S])
Attachments
(1 file, 1 obsolete file)
It would be nice to expose a list of known networks — not only the currently available ones. The point would be to edit/remove networks without having to be connected to them.
Assignee | ||
Updated•12 years ago
|
blocking-basecamp: --- → ?
Whiteboard: [LOE:S]
Comment 1•12 years ago
|
||
We discussed this at triage today. While we agree this would be a great feature, we don't think it's a blocker. Of course, feel free to re-nom if you disagree.
blocking-basecamp: ? → -
Keywords: uiwanted
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → mrbkap
Assignee | ||
Comment 2•12 years ago
|
||
Still to-do: * Clean up/factor out common code in places. * Make explicit the relationship between a Scan Result and a Network. This does seem to work, though!
Assignee | ||
Comment 3•12 years ago
|
||
Attachment #662377 -
Attachment is obsolete: true
Attachment #662742 -
Flags: review?(vchang)
Attachment #662742 -
Flags: feedback?(kaze)
Comment 4•12 years ago
|
||
Comment on attachment 662742 [details] [diff] [review] patch v1 Review of attachment 662742 [details] [diff] [review]: ----------------------------------------------------------------- ::: dom/wifi/WifiWorker.js @@ +2098,5 @@ > + var networks = {}; > + for (let ssid in this.configuredNetworks) { > + networks[ssid] = netToDOM(this.configuredNetworks[ssid]); > + } > + Do we need to send error message when ssid is not found ?
Attachment #662742 -
Flags: review?(vchang) → review+
Assignee | ||
Comment 5•12 years ago
|
||
(In reply to Vincent Chang from comment #4) > Do we need to send error message when ssid is not found ? I don't understand what you mean. When which ssid isn't found where? Do you mean if there's a configured network without an ssid? That's taken care of (with an XXX comment) in _reloadConfiguredNetworks.
Reporter | ||
Comment 6•12 years ago
|
||
Works fine in Gaia: https://bugzilla.mozilla.org/show_bug.cgi?id=791392
Status: NEW → ASSIGNED
Reporter | ||
Updated•12 years ago
|
Attachment #662742 -
Flags: feedback?(kaze) → feedback+
Reporter | ||
Updated•12 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 7•12 years ago
|
||
Comment on attachment 662742 [details] [diff] [review] patch v1 https://hg.mozilla.org/integration/mozilla-inbound/rev/bbfb9d9241e4
Attachment #662742 -
Flags: checkin+
Assignee | ||
Updated•12 years ago
|
Blocks: b2g-wifi
Keywords: checkin-needed
Comment 8•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/bbfb9d9241e4
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
You need to log in
before you can comment on or make changes to this bug.
Description
•