Closed Bug 762403 Opened 10 years ago Closed 10 years ago

Disable the chrome debugger menu item until it's ready

Categories

(DevTools :: Debugger, defect, P2)

defect

Tracking

(firefox15 verified)

RESOLVED FIXED
Firefox 16
Tracking Status
firefox15 --- verified

People

(Reporter: past, Assigned: past)

References

Details

Attachments

(1 file)

It looks like getting all the pieces for chrome debugging will take a while. We should stop misleading users until that time, by hiding the "Browser Debugger" menu item.
Depends on: 761153
Attached patch PatchSplinter Review
Chrome debugging requires remote debugging by definition, so I added a dependency to the remote flag. When we have chrome debugging working, we can flip the chrome-enabled pref from then on and users can control it from the standard in our developer tools devtools.chrome.enabled pref.
Attachment #630902 - Flags: review?(dtownsend+bugmail)
Attachment #630902 - Flags: review?(dcamp)
Attachment #630902 - Flags: review?(dcamp) → review+
Attachment #630902 - Flags: review?(dtownsend+bugmail) → review+
https://hg.mozilla.org/mozilla-central/rev/f6df0a1573e5
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 16
Comment on attachment 630902 [details] [diff] [review]
Patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): New feature.
User impact if declined: Users will have access to a feature that is not ready yet.
Testing completed (on m-c, etc.): ON m-c.
Risk to taking this patch (and alternatives if risky): Riskier not to take it.
String or UUID changes made by this patch: none, but we did add a pref to firefox.js:

+pref("devtools.debugger.chrome-enabled", false);
Attachment #630902 - Flags: approval-mozilla-aurora?
Comment on attachment 630902 [details] [diff] [review]
Patch

[Triage Comment]
Approved for Aurora 15 - are you aware of anywhere this has been communicated and may need to be corrected? I know this wasn't noted for release notes.
Attachment #630902 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
(In reply to Alex Keybl [:akeybl] from comment #5)
> Approved for Aurora 15 - are you aware of anywhere this has been
> communicated and may need to be corrected? I know this wasn't noted for
> release notes.

I believe this has only been discussed internally so far (including work weeks and weekly calls).
(In reply to Panos Astithas [:past] from comment #7)
> (In reply to Alex Keybl [:akeybl] from comment #5)
> > Approved for Aurora 15 - are you aware of anywhere this has been
> > communicated and may need to be corrected? I know this wasn't noted for
> > release notes.
> 
> I believe this has only been discussed internally so far (including work
> weeks and weekly calls).

also, I don't think the addition of the Chrome Debugger was widely advertised to begin with. We plan on enabling it as a feature in a future version so it's probably not really relnote-worthy.

thanks Alex!
I am verifying this fix on Firefox 15.0 beta 3 but I need some clarifications: after setting devtools.chrome.enabled to true, should there be any visible effect on the debugger? How does the user start debugging chrome?
(In reply to Ioana Budnar [QA] from comment #9)
> I am verifying this fix on Firefox 15.0 beta 3 but I need some
> clarifications: after setting devtools.chrome.enabled to true, should there
> be any visible effect on the debugger?

Nope.

> How does the user start debugging chrome?

Chrome debugging is not implemented yet.
Verified as fixed on:
Mozilla/5.0 (X11; Linux i686; rv:15.0) Gecko/20100101 Firefox/15.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:15.0) Gecko/20100101 Firefox/15.0
Mozilla/5.0 (Windows NT 6.1; rv:15.0) Gecko/20100101 Firefox/15.0
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.