215985 - Debugger shows nsDOMClassInfo.cpp as a loaded script

Status: RESOLVED FIXED

(Other Applications Graveyard :: Venkman JS Debugger, defect)

Description

[Christopher Aillon (sabbatical, not receiving bugmail)]

Just loading up venkman shows nsDOMClassInfo.cpp as a loaded script.  Exclude
browser files makes no difference, and it makes no difference when you load venkman.

Looks like
http://lxr.mozilla.org/mozilla/source/dom/src/base/nsDOMClassInfo.cpp#4122 is
the culprit, and brendan changed things around there recently.

Comment 1

[Christopher Aillon (sabbatical, not receiving bugmail)]

Actually, I guess I'll take this.

Comment 2

[Christopher Aillon (sabbatical, not receiving bugmail)]

Attachment: Patch

Comment 3

[Johnny Stenback (:jst)]

Comment on attachment 129693 [details] [diff] [review]
Patch

r=jst

Comment 4

[Brendan Eich [:brendan]]

Comment on attachment 129693 [details] [diff] [review]
Patch

Thanks -- I shoulda thought of venkman when I made that change.  jst and I are
hip to it now.

/be

Comment 5

[Christopher Aillon (sabbatical, not receiving bugmail)]

Comment on attachment 129693 [details] [diff] [review]
Patch

Looking for approval for this small, low-risk fix to show the correct filename
for venkman.

Comment 6

[Asa Dotzler [:asa]]

Comment on attachment 129693 [details] [diff] [review]
Patch

a=asa (on behalf of drivers) for checkin to Mozilla 1.5beta.

Comment 7

[Christopher Aillon (sabbatical, not receiving bugmail)]

Checked into the 1.5beta trunk, with a slight modification suggested by rginda
to also include __FILE__ in a JavaScript comment in the filename parameter.