586721 - Kill dev menu

Status: RESOLVED FIXED

(Firefox Graveyard :: Panorama, defect, P3)

Description

[Ian Gilman [:iangilman]]

From our dolske's review of bug 574217:

> >+  // Function: _addDevMenu
> >+  // Fills out the "dev menu" in the TabView UI.
> >+  _addDevMenu: function() {
> 
> Ought to kill this for landing.

Comment 1

[Antoine Turmel [:GeekShadow]]

and maybe add a pref to enable it ?

Comment 2

[Ian Gilman [:iangilman]]

Does anyone know how to add a pref?

Comment 3

[Ian Gilman [:iangilman]]

Attachment: Patch v1

This patch is to hide the dev menu for the general public; we'd like this in tonight's nightly. 

I propose we keep this bug open after the patch lands, and connect it to the "shipping" meta bug (bug 585689) for full removal.

Comment 4

[Ian Gilman [:iangilman]]

(In reply to comment #1)
> and maybe add a pref to enable it ?

We decided to go with an in-code switch... it's really only of use for the developers.

Comment 5

[(no longer active)]

Mass moving all Tab Candy bugs from Mozilla Labs to Firefox::Tab Candy.  Filter the bugmail spam with "tabcandymassmove".

Comment 6

[Ed Lee :Mardak]

http://hg.mozilla.org/mozilla-central/rev/0113f5c48837

file a followup to remove the logic or add pref or something

Comment 7

[Ed Lee :Mardak]

backed out http://hg.mozilla.org/mozilla-central/rev/c14d91f2d939

Comment 8

[Ed Lee :Mardak]

http://hg.mozilla.org/mozilla-central/rev/efc6f1cbb38f

Comment 9

[Ian Gilman [:iangilman]]

Follow up filed, bug 587835.