303427 - Assert message inserts bogus newline in XPConnect is being called on a scope without a 'Components' property!\

Status: RESOLVED FIXED

(Core :: XPConnect, defect)

Description

[Bob Clary [:bc] (inactive)]

When trying to process debug log files, it is helpful for the line containing
the ASSERTION to actually contain meaningful text. I am attempting to use this
to make sure that all assertions have bugs on file.

Comment 1

[Bob Clary [:bc] (inactive)]

Attachment: trivial patch

Comment 2

[Johnny Stenback (:jst)]

Comment on attachment 191600 [details] [diff] [review]
trivial patch

r+sr=jst

Comment 3

[Asa Dotzler [:asa]]

bob, time is short, so it would be good if you could get this in ASAP.

Comment 4

[Bob Clary [:bc] (inactive)]

checked in to 1.8 branch:
Checking in xpcwrappednativescope.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/xpcwrappednativescope.cpp,v  <-- 
xpcwrappednativescope.cpp
new revision: 1.22.4.1; previous revision: 1.22

checked into trunk
Checking in xpcwrappednativescope.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/xpcwrappednativescope.cpp,v  <-- 
xpcwrappednativescope.cpp
new revision: 1.23; previous revision: 1.22