122695 - Need ability un-modalify (and re-modalify) a xul window

Status: RESOLVED WONTFIX

(Core :: XUL, defect)

Description

[Robert Ginda]

The debugger needs a way to temporarily disable the modality of a modal window
for bug 100677.

Dan and I discussed this and he says he could add an |isModal| property to
nsIXULWindow to take care of this.

0.9.9, right?

Comment 1

[Dan M]

Attachment: switch on and off a modal window's modal UI behaviour

Robert -- try this patch; let me know if it does what you need. You'll probably
hate the name I chose for the attribute. Heh. I wanted something that didn't
collide with other terms in use or imply too much. Obscurity was called for.
Embrace your appearance; learn to live within your miens.

Comment 2

[Robert Ginda]

How about we call the attribute |modalificated|?  I really thought I was on to
something with the bug summary (if you excuse the missing preposition.)

I'll give this a shot tonight and report on my findings.

Comment 3

[Robert Ginda]

Attachment: venkman patch -- not for checkin, yet.

modalMien works for me.  This patch, which depends on bug 121039, actually uses
it.

To test, start venkman and enter "fbreak commonDialog.js 200", then "evald
alert(1)", and click "ok" on the alert window.	Without Dan's patch the app
locks up as the script context of the modal window is disabled while it is
still modal.  With the patch everything goes much smoother.

Comment 4

[Robert Ginda]

Comment on attachment 67323 [details] [diff] [review]
switch on and off a modal window's modal UI behaviour

r=rginda

Comment 5

[Dan M]

I believe that this bug and patch (the first patch, anyway) are no longer in
fashion, to be replaced by the alternate implementation covered by bug 126786.
0.99 cleanup time: I'm closing this bug in favour of its better half. Reopen, of
course, if I'm mistaken.