Closed Bug 768150 Opened 13 years ago Closed 13 years ago

The developer toolbar should be preffed on by default [devtb][GCLI]

Categories

(DevTools :: Console, defect, P1)

Product:
DevTools
Component:
Console
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
Firefox 16

People

(Reporter: jwalker, Assigned: jwalker)

References

Details

Attachments

(1 file, 3 obsolete files)

Upload 4
2.66 KB, patch
ttaubert
: review+
Details | Diff | Splinter Review
No description provided.
Component: Developer Tools → Developer Tools: Console
QA Contact: developer.tools → developer.tools.console
Attached patch Upload 1 (obsolete) — — Splinter Review
Assignee: nobody → jwalker
Status: NEW → ASSIGNED
Attachment #636423 - Flags: review?(dcamp)
Attachment #636423 - Flags: review?(dcamp) → review+
Blocks: 769186
Attached patch Upload 2 (obsolete) — — Splinter Review
Will need r from a browser peer. This removes the key sequence so we can land. Bug 769186 puts it back in again, however we will need to fix the conflict with the editor before we can do that.
Attachment #636423 - Attachment is obsolete: true
Attachment #637428 - Flags: review?(dcamp)
Comment on attachment 637428 [details] [diff] [review] Upload 2 Hey Tim - trivial patch that prefs the toolbar on by default, and removes the key sequence as a result of a conflict with the editor.
Attachment #637428 - Flags: review?(dcamp) → review?(ttaubert)
Comment on attachment 637428 [details] [diff] [review] Upload 2 Review of attachment 637428 [details] [diff] [review]: ----------------------------------------------------------------- Do you plan to leave the developer toolbar preffed on for Aurora as well after the migration or will this be enabled for a couple of trains for Nightly only? If it's the latter then it might make sense to move this to browser/branding/nightly/pref/firefox-branding.js so you don't have to revert it when merged to Aurora.
I think we're feeling more confident about the commands that we've got now and our progress, the intention is to pref this on, and leave it on. Clearly if things don't go according to plan, we can pref it off again. Title updated to reflect this.
Summary: The developer toolbar should be preffed on by default for testing on nightly only [devtb][GCLI] → The developer toolbar should be preffed on by default [devtb][GCLI]
(In reply to Tim Taubert [:ttaubert] from comment #5) > Do you plan to leave the developer toolbar preffed on for Aurora as well > after the migration or will this be enabled for a couple of trains for > Nightly only? If it's the latter then it might make sense to move this to > browser/branding/nightly/pref/firefox-branding.js so you don't have to > revert it when merged to Aurora. Gavin had reservations about using the branding pref file for this purpose, see bug 749628 comment 39.
(In reply to Panos Astithas [:past] from comment #7) > Gavin had reservations about using the branding pref file for this purpose, > see bug 749628 comment 39. Aha! Thanks for the hint.
Attachment #637428 - Flags: review?(ttaubert) → review+
Attached patch Upload 3 (obsolete) — — Splinter Review
The correct way to remove a key sequence ... Sorry Tim!
Attachment #637428 - Attachment is obsolete: true
Attachment #637833 - Flags: review?(ttaubert)
Attached patch Upload 4 — — Splinter Review
Rushing. This removes devToolbar.commandkey too.
Attachment #637833 - Attachment is obsolete: true
Attachment #637833 - Flags: review?(ttaubert)
Attachment #637835 - Flags: review?(ttaubert)
Attachment #637835 - Flags: review?(ttaubert) → review+
https://tbpl.mozilla.org/?tree=Fx-Team&rev=124c0182df1c
Whiteboard: [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/5d6c06259bb1
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: