Closed Bug 100677 Opened 23 years ago Closed 22 years ago

Modal windows are not debuggable

Categories

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

x86
Linux
defect

Tracking

(Not tracked)

RESOLVED FIXED
Future

People

(Reporter: colinp, Assigned: rginda)

References

Details

(Whiteboard: [native])

Attachments

(1 file)

Modal windows are not debuggable because of the modality of the window being
debugged.  The debugger does in fact halt at the breakpoint, however, the only
way to exit from the modal window is by closing (or cancelling the window) since
no operations are allowed.

I don't have a debug build (unfortunately), so you'll have to wait for a stack
trace.
At some point, the debugger should disable the target window (if there is one),
to keep it from generating new events while being debugged.  This bug should be
taken into account when that happens.
Status: NEW → ASSIGNED
Priority: -- → P3
I just ran into this one.  I was unable to debug the cookie viewer.  There was
no way to continue from the breakpoint, and I had to close the cookie viewer,
which eventually led to the crash described in bug 110387.  FWIW, my talkback ID
is TB1613636Y
QA Contact: rginda → caillon
This one scares me.
Whiteboard: [native]
Target Milestone: --- → mozilla1.0
Depends on: 122695
Depends on: 126786
Moving Netscape owned 0.9.9 and 1.0 bugs that don't have an nsbeta1, nsbeta1+,
topembed, topembed+, Mozilla0.9.9+ or Mozilla1.0+ keyword.  Please send any
questions or feedback about this to adt@netscape.com.  You can search for
"Moving bugs not scheduled for a project" to quickly delete this bugmail.
Target Milestone: mozilla1.0 → mozilla1.2
Target Milestone: mozilla1.2 → mozilla1.0
Depends on: 129591
Attached patch patchSplinter Review
Here's the fix.  This should work immediatley on all platforms except Linux. 
Linux depends on bug 129591.
I checked the patch in, linux users will have to watch bug 129591.
if bug 129591 get's fixed for 1.0, this will too.  It doesn't look like 129591
is going to make it, so speak up now if you care.
Target Milestone: mozilla1.0 → Future
fixed by check-in for bug 129591
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Product: Core → Other Applications
Product: Other Applications → Other Applications Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: