752006 - Update Error Console comment in browser.js

Status: RESOLVED FIXED

(Firefox :: General, defect, P3)

Description

[Rob Campbell [:rc] (:robcee)]

The Error Console is permanently enabled, despite what it says in https://developer.mozilla.org/en/Error_Console

http://mxr.mozilla.org/mozilla-central/source/browser/base/content/browser.js#1739

This says "Temporarily Enabled" though we no longer have a plan to disable the error console. We should update this to reflect reality.

Comment 1

[Berker Peksag [:berkerpeksag]]

Updated location is here:

http://mxr.mozilla.org/mozilla-central/source/browser/base/content/browser.js#1559

Comment 2

[Rob Campbell [:rc] (:robcee)]

Let's do this when we turn on the browser console by default.

see bug 602006.

Filter on GUNGNIR.

Comment 3

[Mihai Sucan [:msucan]]

This bug seems to be already fixed. The "// XXX Temporarily always-enabled, see bug 601201" comment is no longer in browser.js. I didn't find which patch/bug fixed removed the comment.

Comment 4

[Mihai Sucan [:msucan]]

(In reply to Mihai Sucan [:msucan] from comment #3)
> This bug seems to be already fixed. The "// XXX Temporarily always-enabled,
> see bug 601201" comment is no longer in browser.js. I didn't find which
> patch/bug removed the comment.

Hah, I found the comment. It now says "// Temporarily enabled. See bug 798925". I see the patch for bug 602006 will remove it. Not much point in reopening the bug for a comment only.