Debugger shows nsDOMClassInfo.cpp as a loaded script

RESOLVED FIXED

Status

Product:
Other Applications
Component:
Venkman JS Debugger
RESOLVED FIXED
Reported:
15 years ago
Modified:
14 years ago

People

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

Tracking

Version:
Trunk
x86
Linux

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Patch
1.69 KB, patch
jst
: review+
brendan
: superreview+
asa
: approval1.5b+
Details | Diff | Splinter Review
(Assignee)

Description

•
15 years ago 
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.
(Assignee)

Comment 1

•
15 years ago 
Actually, I guess I'll take this.
Assignee: rginda → caillon
Attachment #129693 - Flags: superreview?(brendan)
Attachment #129693 - Flags: review?(jst)

Comment 3

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

r=jst
Attachment #129693 - Flags: review?(jst) → review+

Comment 4

•
15 years ago 
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+
 (Assignee)

Comment 5

•
15 years ago 
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+
 (Assignee)

Comment 7

•
15 years ago 
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.