Albert Brand Reporter
	Description • 16 years ago

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2a1pre) Gecko/20090331 Minefield/3.6a1pre
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2a1pre) Gecko/20090331 Minefield/3.6a1pre





Reproducible: Always

Steps to Reproduce:
Set browser.tabs.closeWindowWithLastTab to false and close the last tab with Ctrl-W or middleclick on the tab.
Actual Results:  
Several things briefly appear and disappear.

1. The tab closes and disappears from the strip, which moves the 'new tab' button briefly to the outer left
2. An empty tab appears with the 'x' or close button briefly present, the new tab button moves back to its previous place
3. The close button disappears from the tab.

Expected Results:  
The tab shouldn't disappear briefly from the strip, the new tab button shouldn't move and the close button shouldn't appear briefly.

	Nochum Sossonko [:Natch]
	Comment 1 • 16 years ago

The tab closing should probably be stopped entirely in this case. Confirmed.

	Dão Gottwald [:dao] Assignee
	Comment 3 • 16 years ago

err, "&& !aCloseWindow && !this._windowIsClosing" was superfluous.

	AKIHIRO Misaki (a.k.a Kuden)
	Comment 4 • 16 years ago

(In reply to comment #3)
> Created an attachment (id=371110) [details]
> patch

This works fine. 
Bug 486129 also fixed.

However, .tab-close-button is still displayed on the new blank tab momentarily.

	Dão Gottwald [:dao] Assignee
	Comment 5 • 16 years ago

http://hg.mozilla.org/mozilla-central/rev/5f4cbbe377fc

(In reply to comment #4)
> However, .tab-close-button is still displayed on the new blank tab momentarily.

I have an idea how to fix that, but it won't happen in this bug. If you file a new one and cc me, I won't forget to look into that.

	AKIHIRO Misaki (a.k.a Kuden)
	Comment 6 • 16 years ago

> I have an idea how to fix that, but it won't happen in this bug. If you file a
> new one and cc me, I won't forget to look into that.

OK. I filed Bug 490166.

	Mike Beltzner [:beltzner, not reading bugmail]
	Comment 7 • 16 years ago

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

a191=beltzner, please land with bug 457096

	Dão Gottwald [:dao] Assignee
	Comment 8 • 16 years ago

http://hg.mozilla.org/releases/mozilla-1.9.1/rev/c0ba402b9d74

	Mike Beltzner [:beltzner, not reading bugmail]
	Comment 9 • 16 years ago

This checkin was in a range identified with a Ts Shutdown regression on Windows:

Regression: Ts Shutdown increase 27.64% on WINNT 5.1 Firefox3.5
   Previous results:
       357.263 from build 20090506145316 of revision 486b76052a94 at 2009-05-06 14:53:00
   New results:
       456.0 from build 20090506155401 of revision 7aa4483585bd at 2009-05-06 15:54:00
http://graphs-new.mozilla.org/graph.html#tests=[{"machine":32,"test":36,"branch":3},{"machine":33,"test":36,"branch":3},{"machine":34,"test":36,"branch":3},{"machine":35,"test":36,"branch":3},{"machine":48,"test":36,"branch":3}]&sel=1241564040,1241736840

	Alfred Kayser
	Comment 10 • 16 years ago

With Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090525 Minefield/3.6a1pre (.NET CLR 3.5.30729) I still see flickering of the whole page when trying to close the last tab...

	Aakash Desai [:aakashd]
	Comment 11 • 16 years ago

...Alfred is this working for you now? Because it worked for me on two OS's on Shiretoko as well as Trunk:



verified FIXED on Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1pre) 
Gecko/20090526 Shiretoko/3.5pre (.NET CLR 3.5.30729) ID:20090526042535

and

Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2a1pre) Gecko/20090526 Minefield/3.6a1pre (.NET CLR 3.5.30729) ID:20090526044156

Same goes for the same build ID's on OS X 10.5.


I'll keep this in the same state until the end of this week if there's no answer.

	Aakash Desai [:aakashd]
	Comment 12 • 16 years ago

Never mind, I misread comment #10, his complaint is being resolved on bug 490166. Moving the bug to verified FIXED.

	Alfred Kayser
	Comment 13 • 16 years ago

As long as the 'closing the last tab (closeWindowWithLastTab=false)' on empty page doesn't do nothing, but keeps on killing the tab and then recreating a new empty one, UI elements (not just the close button) will keep on flashing (in the default theme, and custom themes). 
I agree that bug 490166 is very similar, so one could argue one is a DUP of another, or that bug 490166 is more specific, just pointing to one flickering UI element, and this bug is about all flickering...
I would propose to keep bug 490166 for the closebutton with its specific patch to just prevent the closebutton to be momentarily visible, and this bug for the whole unwanted behaviour of closing the last empty tab, and then recreating a new one. It should just keep the last empty tab open.

	Dão Gottwald [:dao] Assignee
	Comment 14 • 16 years ago

The last /empty/ tab is a special case, and not what this bug is about. In general, the last tab /should/ be replaced by a blank one, given closeWindowWithLastTab=false.

	Max
	Comment 15 • 16 years ago

This bug is not fixed on my mac system, which has:

browser.tabs.closeWindowWithLastTab-false
browser.tabs.closeButtons=3

Clicking on the "Close Tab" icon when there's a single tab (empty or not)
causes a temporary display of a second tab (labeled "Untitled") before leaving
a single Untitled tab.

The whole idea of "Close Tab" being promoted to "Close Window" is foolish. 
I couldn't believe it was intentional until someone wrote that it's copied
from Microsoft.

	Alex Faaborg [:faaborg] (Firefox UX)
	Comment 16 • 15 years ago

This bug's priority relative to the set of other polish bugs is:
P1 - Polish issue that appears in the main window, or is something that the user may encounter several times a day.