Prevent submitting twice, opening two PSM windows, etc by double clicking.




Event Handling
17 years ago
4 years ago


(Reporter: hirata masakazu, Assigned: Alec Flett)


Mac System 9.x

Firefox Tracking Flags

(Not tracked)



(1 attachment)



17 years ago
When you double click a submit button, PSM padlock icon, etc, Mozilla tries to
submit twice, open two PSM windows, etc.
This should be avoided. Opening a PSM window is quite dangerous currently,
because of memory leak. Submitting twice usually result in a dialog hidden by a
refreshed window, resulting in apparent unresponsiveness of Mozilla.
Send mail is now disabled from double action, I think, and these can be avoided,

Comment 1

17 years ago
Any progress on this? I took a quick look in viewer and the content model looks 
correct; e.g.,

  iframe@00F57080 width=100% height=100% src= refcount=4<

Yet the content area still shows

Comment 2

17 years ago
Created attachment 26033 [details] [diff] [review]
add logging to nsDocShell.cpp

Comment 3

17 years ago
nsDocShell.cpp thinks it's loading the right URL.

Comment 4

17 years ago
Oops wrong bug. Dammit!

Comment 5

17 years ago
Gerardo, could we get a current test on both mac and windows?  We should fix 
this if it's still happening.  

Can this be fixed in XUL code? 
Keywords: nsbeta1, qawanted

Comment 6

17 years ago
Over to Chris.  If retest shows a problem we'll need our Mac guy on it.
Assignee: joki → saari

Comment 7

17 years ago
Give this to someone else. Make sure the xul is looking for mouseclick and not
mouse down (which you'll see two of).
Assignee: saari → hangas

Comment 8

17 years ago
Oops, hangas was a mistarget. Does this fall into xpapps?
Assignee: hangas → vishy
yes we should fix this. pretriaging as nsbeta1+. reassigning to Alec. 
Assignee: vishy → alecf

Comment 10

17 years ago
4.x never did this - I don't see why this is a beta-stopper!
Suggestting nsbeta1-
Priority: -- → P3
Target Milestone: --- → mozilla1.0

Comment 11

17 years ago
nav triage team:

Marking nsbeta1-, although annoying, proper fix would require a lot of work, not 
a beta1 stopper.
Keywords: nsbeta1 → nsbeta1-

Comment 12

17 years ago
QA contact updated
QA Contact: gerardok → madhur

Comment 13

17 years ago
nav triage team:

Not a mozilla1.0 stopper, pushing out to mozilla1.0.1
Target Milestone: mozilla1.0 → mozilla1.0.1
If double submit goes away, so does this, correct?
Depends on: 72906
hrm, PSM windows, at least, are not using <form>.  The double-submit fix does
not keep multiple windows from opening.  Un-depending.
No longer depends on: 72906


16 years ago
QA Contact: madhur → rakeshmishra

Comment 16

16 years ago
I think this was fixed by jkeiser with the double-submit bug. marking WORKSFORME
unless someone can find the dupe.
Last Resolved: 16 years ago
Resolution: --- → WORKSFORME


16 years ago
QA Contact: rakeshmishra → trix
Keywords: qawanted
You need to log in before you can comment on or make changes to this bug.