65127 - Remove useless/non-error dump()s

Status: VERIFIED FIXED

(SeaMonkey :: UI Design, defect)

Description

[Blake Ross]

So, I'd like to start removing all the useless dump()s we have littered 
everywhere.   I've already removed most of them from quite a few xpfe files.  
For now, I'll keep the ones that report errors or live in |catch|es; I'm mainly 
removing ones likes "***foopy" or "in function foo" that serve only to slow 
things down and fill up the console in release builds, without any real 
debugging gain.

Anyone brave enough to give me a blank r=/sr= for this?

Comment 1

[Blake Ross]

s/blank/blanket

Comment 2

[Jesse Ruderman]

I actually like a few of the dumps, such as "document foo loaded successfully", 
and the old "webshell+ =5" (now debug-only:( )  They make it easier to figure 
out when an error happened relative to other things.

Comment 3

[Blake Ross]

Those are printf()s and other C++ console output.  I'm removing dump()s, which 
are frontend output, and usually useless.

Comment 4

[Blake Ross]

Largely done, will remove others when touching code in the area.

Comment 5

[sairuh (rarely reading bugmail)]

rubberstamp, unless you'd like me to check some specific places...