409215 - Closing tabs quickly in succession causes the close button to "get stuck" (Minefield 3.0pre3, browser.tabs.closeButtons = 1)

Status: VERIFIED FIXED

(Firefox :: Tabbed Browser, defect, P2)

Description

[spam_hole]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b3pre) Gecko/2007122005 Minefield/3.0b3pre
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b3pre) Gecko/2007122005 Minefield/3.0b3pre

When I want to close a lot of tabs, but not all of them, I typically just hit the close buttons really quickly (FF automatically highlights the next tab, which then exposes the close button, so I basically click in the close button of the rightmost tab).

This has worked fine in a trunk build from Dec. 05, but as of the trunk build of the 16th, when I attempt to rapidly close two tabs by this method, the first one closes but the second does not respond to the button click (the "depressed" icon is shown when it is clicked, but it fails to close).

Ctrl+W still works rapidly in the same circumstance.

Reproducible: Always

Steps to Reproduce:
1. Have browser.tabs.closeButtons = 1
2. Open around 10 tabs in a new window (hold Ctrl+T is a good way to get this)
3. Repeatedly click (quickly) in the close button of the (currently) active tab: this will close the current tab and attempt to close the other tabs -- n.b. you should not move your mouse pointer, let the tabs and their close buttons come to your cursor
4. Note that the second tab refuses to close, unless you wait between clicks -- seems to be a double-click event which isn't working
Actual Results:  
Continually clicking the second tab's close button fails to close the tab.

I suspect this may be a misfired double-click or something similar.

Expected Results:  
The second tab and all subsequent tabs (until the close button is no longer at the very right) are closed.

Comment 1

[Ria Klaassen (not reading all bugmail)]

Closing 10 tabs in succession needs 20 clicks indeed. This regressed somewhere between 1 and 16 sept 2006, and bug 352021 was fixed within this time span, so the fix of this bug might have caused this.

Comment 4

[Mike Connor [:mconnor]]

Main bug i care about is the "stuck" symptom, which goes away as soon as you click anywhere else.  The unable to close two tabs on doubleclick is less of a concern, and its probably more correct than not.

Comment 6

[:Gavin Sharp [email: [e-mail]]]

Attachment: patch

Avoid ignoring more than one click. Fixes this bug without regressing bug 378344 or bug 352021. Closing tabs is still a little bit slower than it used to be, since we might end up ignoring half the clicks, but at least it doesn't get "stuck".

Comment 7

[Dão Gottwald [:dao]]

(In reply to comment #4)
> Main bug i care about is the "stuck" symptom, which goes away as soon as you
> click anywhere else.

It goes away as soon as you wait a split second so that event detail is 1 again. I'm not sure why event.detail flips from 2 to 3 and back to 2 if you click often.

Comment 8

[Mike Connor [:mconnor]]

Because that's how the event code works.  Don't remember the spec, but dbaron explained it to me a couple of years ago, basically there's no quad-click, so they just cycle back and forth rather than continuing to just return 3.  Think its based on the native event models, actually...

Comment 9

[Dão Gottwald [:dao]]

The spec says "The attribute value is 1 when the user begins this action and increments by 1 for each click", so I would have expected 1, 2, 3, 1, 2, 3, 1... rather than 1, 2, 3, 2, 3, 2...
But I can confirm that this is native on Windows, so even though it doesn't strictly follow the spec, it's not a bug.

Comment 10

[Mike Connor [:mconnor]]

Comment on attachment 307545 [details] [diff] [review]
patch

r=mconnor

think this is probably the right compromise to avoid accidental double clicks closing two tabs..

Comment 11

[:Gavin Sharp [email: [e-mail]]]

mozilla/browser/base/content/tabbrowser.xml 	1.266

Comment 12

[Hasham]

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9) Gecko/2008052906 Firefox/3.0

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9pre) Gecko/2008060306 Minefield/3.0pre

Ok so double-click to close multiple tabs is purposeful, not a bug, right?

If so, this is working properly in RC2 and nightly. I used XP BTW.

Comment 13

[Dão Gottwald [:dao]]

(In reply to comment #12)
> Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9) Gecko/2008052906
> Firefox/3.0
> 
> Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9pre) Gecko/2008060306
> Minefield/3.0pre
> 
> Ok so double-click to close multiple tabs is purposeful

... only if there's a delay between the clicks; a real double click shouldn't close two tabs with browser.tabs.closeButtons = 1 (see bug 352021).

Comment 16

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Verified fixed based on comment 12 and with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.0.6) Gecko/2009011912 Firefox/3.0.6

Comment 17

[Nickolay_Ponomarev]

I recently confirmed bug 367777, morphing it about a visual issue with the double-click - the second [x] button appears to be clicked, even though the second tab isn't closed.