89540 - access-proxy must not print debug msg to console in opt build

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

[cls]

access-proxy has a hardcoded:
#define NS_DEBUG_ACCESS_BUILTIN 1
which causes a ton of debug messages to hit the console for every event even in
an optimized build.  This define should be wrapped in a DEBUG ifdef at minimum.

Comment 1

[Aaron Leventhal]

checked in

Comment 2

[Aaron Leventhal]

Something didn't make it in the checkin.

Comment 3

[Aaron Leventhal]

Attachment: This change was missed

Comment 4

[John Gaunt (redfive)]

for the attachement I give r=.

Aaron, what is your comment "checked in" referring too? 

Comment 5

[Aaron Leventhal]

"Checked in" - originally the fix to get rid of printf's was reviewed and
checked in as part of another bug.

Had to reopen it when I noticed another printf.

Comment 6

[Randell Jesup [:jesup] (needinfo me)]

r= jesup for whatever that's worth

Comment 7

[Brendan Eich [:brendan]]

rs=[e-mail].

Comment 8

[Aaron Leventhal]

fix checked into trunk