Closed
Bug 1032708
Opened 10 years ago
Closed 10 years ago
Keep discovery alive when WiFi network changes
Categories
(DevTools Graveyard :: WebIDE, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 33
People
(Reporter: jryans, Assigned: jryans)
References
Details
Attachments
(1 file, 1 obsolete file)
2.21 KB,
patch
|
jryans
:
review+
|
Details | Diff | Splinter Review |
The discovery socket is dropped on b2g devices if the WiFi network changes, goes to sleep, etc.
Assignee | ||
Comment 1•10 years ago
|
||
We listen for a message on b2g whenever the network changes, and restart any sockets that were open previously. Try: https://tbpl.mozilla.org/?tree=Try&rev=e5e667d3cec1
Attachment #8449023 -
Flags: review?(paul)
Comment 2•10 years ago
|
||
Comment on attachment 8449023 [details] [diff] [review] Restart discovery when network changes + this.observe = this.observe.bind(this); I'm not sure this is necessary.
Attachment #8449023 -
Flags: review?(paul) → review+
Assignee | ||
Comment 3•10 years ago
|
||
(In reply to Paul Rouget [:paul] (slow to respond. Ping me on IRC) from comment #2) > Comment on attachment 8449023 [details] [diff] [review] > Restart discovery when network changes > > + this.observe = this.observe.bind(this); > > I'm not sure this is necessary. You're right, it's not. Removed! Also, I changed this slightly to only restart if the active network is WiFi. Try: https://tbpl.mozilla.org/?tree=Try&rev=4861df991499
Attachment #8449023 -
Attachment is obsolete: true
Attachment #8449674 -
Flags: review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 4•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/167888bcb787
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
Comment 5•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/167888bcb787
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 33
Updated•6 years ago
|
Product: Firefox → DevTools
Updated•4 years ago
|
Product: DevTools → DevTools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•