Debugger shows nsDOMClassInfo.cpp as a loaded script

RESOLVED FIXED

Status

Other Applications
Venkman JS Debugger
RESOLVED FIXED
15 years ago
14 years ago

People

(Reporter: Christopher Aillon (sabbatical, not receiving bugmail), Assigned: Christopher Aillon (sabbatical, not receiving bugmail))

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

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.
Actually, I guess I'll take this.
Assignee: rginda → caillon
Attachment #129693 - Flags: superreview?(brendan)
Attachment #129693 - Flags: review?(jst)
Comment on attachment 129693 [details] [diff] [review]
Patch

r=jst
Attachment #129693 - Flags: review?(jst) → review+
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
Attachment #129693 - Flags: superreview?(brendan) → superreview+
Comment on attachment 129693 [details] [diff] [review]
Patch

Looking for approval for this small, low-risk fix to show the correct filename
for venkman.
Attachment #129693 - Flags: approval1.5b?

Comment 6

15 years ago
Comment on attachment 129693 [details] [diff] [review]
Patch

a=asa (on behalf of drivers) for checkin to Mozilla 1.5beta.
Attachment #129693 - Flags: approval1.5b? → approval1.5b+
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.
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
Product: Core → Other Applications
You need to log in before you can comment on or make changes to this bug.