Closed
Bug 917479
Opened 11 years ago
Closed 11 years ago
Starting a simulator changes host and port settings
Categories
(DevTools Graveyard :: WebIDE, defect, P1)
DevTools Graveyard
WebIDE
Tracking
(firefox26 verified, firefox27 verified, b2g-v1.2 fixed)
RESOLVED
FIXED
Firefox 27
People
(Reporter: jryans, Assigned: paul)
Details
(Keywords: verifyme)
Attachments
(1 file, 1 obsolete file)
1.79 KB,
patch
|
ochameau
:
review+
lsblakk
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
Right now, there's only one set of host and port settings in the connection footer. If you start a simulator, the port will likely change to a randomly chosen port at startup time. If you later want to connect to a device, you'll have to edit the port back to 6000 first, then connect. Generally, it feels like we may need to rethink the flow of the connection footer to make it simpler to preserve the device settings.
Assignee | ||
Comment 1•11 years ago
|
||
If you re-use the connection used by the Simulator, indeed. When the simulator closes, we should kill the connection as well, and create a new one.
Assignee | ||
Updated•11 years ago
|
Priority: -- → P1
Assignee | ||
Comment 3•11 years ago
|
||
Attachment #810563 -
Flags: review?(poirot.alex)
Assignee | ||
Updated•11 years ago
|
Whiteboard: [needed-in-aurora-26]
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → paul
Comment 4•11 years ago
|
||
Comment on attachment 810563 [details] [diff] [review] Patch v1 Needs rebase, connected listener in now a singleton.
Attachment #810563 -
Flags: review?(poirot.alex)
Comment 5•11 years ago
|
||
The conflicting patch is the one from bug 919681.
Assignee | ||
Comment 6•11 years ago
|
||
Attachment #810563 -
Attachment is obsolete: true
Attachment #811116 -
Flags: review?(poirot.alex)
Assignee | ||
Updated•11 years ago
|
Whiteboard: [needed-in-aurora-26] → [waiting for review]
Comment 7•11 years ago
|
||
Comment on attachment 811116 [details] [diff] [review] Patch V2 Review of attachment 811116 [details] [diff] [review]: ----------------------------------------------------------------- FYI: If you launch the simulator and immediatly click on cancel, it doesn't reset to 6000. Otherwise, the patch works fine in sane usages of the simulator.
Attachment #811116 -
Flags: review?(poirot.alex) → review+
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Whiteboard: [waiting for review] → [land-in-fx-team]
Comment 8•11 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/d079bd51114c
Keywords: checkin-needed
Whiteboard: [land-in-fx-team] → [fixed-in-fx-team]
Comment 9•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/d079bd51114c
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 27
Assignee | ||
Comment 10•11 years ago
|
||
Comment on attachment 811116 [details] [diff] [review] Patch V2 [Approval Request Comment] Bug caused by (feature/regressing bug #): new feature (app manager) User impact if declined: connection params messed up after simulator started Testing completed (on m-c, etc.): m-c Risk to taking this patch (and alternatives if risky): low String or IDL/UUID changes made by this patch: none
Attachment #811116 -
Flags: approval-mozilla-aurora?
Assignee | ||
Updated•11 years ago
|
Whiteboard: [waiting-for-aurora-approval]
Updated•11 years ago
|
Attachment #811116 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 11•11 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/90ad9f7c9795
status-b2g-v1.2:
--- → fixed
status-firefox26:
--- → fixed
status-firefox27:
--- → fixed
Whiteboard: [waiting-for-aurora-approval]
Comment 12•11 years ago
|
||
Verified as fixed with: latest Aurora 27.0a2, latest Nightly 28.0a1 and latest Beta 26 beta 4, on: Win 7 32-bit, Ubuntu 13.04 32-bit, Mac OS X 10.6.8
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
•