Closed Bug 854174 Opened 12 years ago Closed 12 years ago

Chrome debugging should use its own port to avoid interfering with remote port changes

Categories

(DevTools :: Debugger, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 22

People

(Reporter: vporof, Assigned: vporof)

References

Details

Attachments

(2 files)

Since there's currently no way of passing settings around between firefox instances when initializing the browser debugger process, the server initialization port and the client connection port may end up being different across the two profiles. This is especially obvious when the Firefox OS simulator dashboard changes the port pref.
Attached patch v1 (deleted) — Splinter Review
Assignee: nobody → vporof
Status: NEW → ASSIGNED
Attachment #728710 - Flags: review?(past)
Blocks: 854185
Depends on: 853181
Comment on attachment 728710 [details] [diff] [review] v1 Review of attachment 728710 [details] [diff] [review]: ----------------------------------------------------------------- Just a couple of minor issues, but otherwise it looks good. ::: browser/app/profile/firefox.js @@ +1056,5 @@ > // Enable the Debugger > pref("devtools.debugger.enabled", true); > pref("devtools.debugger.chrome-enabled", true); > +pref("devtools.debugger.chrome-debugging-host", "localhost"); > +pref("devtools.debugger.chrome-debugging-port", 6123); Could we pick a port number that is unassigned this time? 6080 seems like a good one: https://www.iana.org/assignments/service-names-port-numbers/service-names-port-numbers.txt ::: browser/devtools/debugger/DebuggerUI.jsm @@ +10,5 @@ > const Cu = Components.utils; > > const DBG_XUL = "chrome://browser/content/debugger.xul"; > const DBG_STRINGS_URI = "chrome://browser/locale/devtools/debugger.properties"; > +const CHROME_DEBUGGER_PROFILE_NAME = "-chrome-debugger"; Could you avoid changing this? Every time we change the profile name, garbage gets accumulated in people's computers, because nobody besides us is going to remove the obsolete profiles.
Attachment #728710 - Flags: review?(past) → review+
(In reply to Panos Astithas [:past] from comment #2) > > Could you avoid changing this? Every time we change the profile name, > garbage gets accumulated in people's computers, because nobody besides us is > going to remove the obsolete profiles. Not really, I'm changing the profile name because otherwise the prefs won't be updated (the old profile won't know about the new chrome-debugging-* prefs).
(In reply to Panos Astithas [:past] from comment #2) > > Could we pick a port number that is unassigned this time? 6080 seems like a > good one: > > https://www.iana.org/assignments/service-names-port-numbers/service-names- > port-numbers.txt > Good idea. What do you think about making the default chrome debugging port to 6081 and change the remote debugging port to 6080? This will also make > user_pref("devtools.debugger.remote-port", 6023); in automation.py unnecessary (which I'd say is a good thing) and probably help us avoid any pain we may ever encounter. I know the simulator is being smart about this, but still..
(In reply to Victor Porof [:vp] from comment #3) > (In reply to Panos Astithas [:past] from comment #2) > > > > Could you avoid changing this? Every time we change the profile name, > > garbage gets accumulated in people's computers, because nobody besides us is > > going to remove the obsolete profiles. > > Not really, I'm changing the profile name because otherwise the prefs won't > be updated (the old profile won't know about the new chrome-debugging-* > prefs). Ugh, OK then. (In reply to Victor Porof [:vp] from comment #4) > (In reply to Panos Astithas [:past] from comment #2) > > > > Could we pick a port number that is unassigned this time? 6080 seems like a > > good one: > > > > https://www.iana.org/assignments/service-names-port-numbers/service-names- > > port-numbers.txt > > > > Good idea. What do you think about making the default chrome debugging port > to 6081 and change the remote debugging port to 6080? This will also make > > user_pref("devtools.debugger.remote-port", 6023); > in automation.py unnecessary (which I'd say is a good thing) and probably > help us avoid any pain we may ever encounter. > > I know the simulator is being smart about this, but still.. 6081 has been used by a trojan, so I'm worried that we might trigger warnings from anti-virus programs. I'd also rather not change the default remote port at this point, because it has been widely used and documented, and I don't think it makes sense to break working configurations for users without a significant gain.
Attached patch v1.1 (deleted) — Splinter Review
Changed default chrome debugging port to 6080.
Attachment #729594 - Flags: review+
Whiteboard: [land-in-fx-team]
Priority: -- → P3
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 22
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: