853039 - Deprecation warnings default to ON

Status: RESOLVED INVALID

(Toolkit :: General, defect)

Description

[:Irving Reid (No longer working on Firefox)]

The code checked in with bug 812859 sets pref("devtools.errorconsole.deprecation_warnings", true);

Comment 1

[:Gavin Sharp [email: [e-mail]]]

I know Yoric said it was unexpected, but is it a problem?

Comment 2

[David Teller [:Yoric] - still alive but not very active]

I believe that it is a bit spammish.

Comment 3

[:Gavin Sharp [email: [e-mail]]]

Spammy only to the error console. Having deprecation warnings in the error console by default seems reasonable to me. If they're behind a hidden pref they won't be seen by anyone.

Comment 4

[Justin Dolske [:Dolske]]

What Gavin said. What's the point of a deprecation warning/module if it's silent?

Comment 5

[Marco Bonardo [:mak]]

I honestly think they should stay enabled, don't even see the reason for the pref to disable them.
It's hard enough to convert add-on authors to new APIs, allowing them to ignore the warnings is not going to help.

Comment 6

[David Teller [:Yoric] - still alive but not very active]

The original reason to have the preference is because we felt that non-developer users would probably be surprised by these warnings.

Comment 7

[:Gavin Sharp [email: [e-mail]]]

Non-developer users generally don't pay attention to the error console :) Sounds like this is RESOLVED WEREOKWITHIT