879581 - Tabs are initially too high in the titlebar on Windows with sizemode=normal

Status: RESOLVED FIXED

(Firefox :: Theme, defect)

Description

[Justin Dolske [:Dolske]]

(Filed from Australis tabs ui-review #1)

There seems to be a recent regression causing Windows 7 tabs-in-titlebar to be crammed up against the top of the window. Happens sporadically, and seems to fix itself when the window is resized.

Comment 1

[u428464]

Dupe of bug 871279 ?

Comment 2

[:Gijs (he/him)]

(In reply to Guillaume C. [:ge3k0s] from comment #1)
> Dupe of bug 871279 ?

I suspect something else. Is the window unfocused / in the background when it is constructed?

Comment 3

[Matthew N. [:MattN]]

This is a temporary issue which is resolved upon maximize/restore and happens right upon startup with a single primary browser window. I do see that those changes are not necessarily popup-specific though so you may be right. 

I actually suspect a race condition with the TabsInTitlebar measurement code.  My guess is that we're measuring the height of some elements before they're ready.

Comment 4

[Matthew N. [:MattN]]

(In reply to :Gijs Kruitbosch from comment #2)
> I suspect something else. Is the window unfocused / in the background when
> it is constructed?

Yep, I can reproduce this if the window is not in the foreground upon launch.

Comment 5

[:Gijs (he/him)]

So this is caused by the window initially not having a sizemode attribute, which breaks our styles. :-\

Comment 6

[:Gijs (he/him)]

Attachment: Patch

So this fixes it. However, AIUI, widget code should really be taking care of this, so I'm very reluctant to say we need to take this. Might be a good fallback if we don't find anything better, though...

Comment 7

[:Gijs (he/him)]

By the way, I did check that this gets overridden when you quit in maximized mode, and it does.

Comment 8

[Mike de Boer [:mikedeboer]]

taking this to Australis M7

Comment 9

[:Gijs (he/him)]

Comment on attachment 758505 [details] [diff] [review]
Patch

Neil said that if we depend on this, we should add it in the XUL. I think we should do this.

Comment 10

[Matthew N. [:MattN]]

Comment on attachment 758505 [details] [diff] [review]
Patch

Review of attachment 758505 [details] [diff] [review]:
-----------------------------------------------------------------

Since we don't support the window feature "fullscreen" and I don't know of another way to open a window fullscreeen or maximized by default, this seems to be fine.

Nit: Update the commit message to be more specific that the sizemode is for the browser window. Specifying that the default is "normal" would also be good.

e.g. Default the sizemode attribute to "normal" on navigator:browser windows.

Comment 11

[Matthew N. [:MattN]]

(In reply to :Gijs Kruitbosch from comment #9)
> Neil said that…

By the way, this is ambiguous since there is no Neil CC'd and there is more than one that you could be referring to. No big deal in this case :)

Comment 12

[:Gijs (he/him)]

Pushed: https://hg.mozilla.org/projects/ux/rev/13e40012da07

Comment 13

[:Gijs (he/him)]

https://hg.mozilla.org/mozilla-central/rev/13e40012da07