219769 - Mozilla and Firebird start in a zero height window

Status: VERIFIED FIXED

(Core :: DOM: Core & HTML, defect, P1)

Description

[Frederic Bezies]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; fr-fr; rv:1.5b) Gecko/20030920 Firebird/0.7+
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; fr-fr; rv:1.5b) Gecko/20030920 Firebird/0.7+

Simple to describe.

When I launch Firebird, its main windows is zero height.

I built Firebird from a current CVS based trunk.

Here is my .mozconfig :

ac_add_options --enable-optimize='-O1 -G6'
ac_add_options --disable-ldap
ac_add_options --disable-mailnews
ac_add_options
--enable-extensions=cookie,xml-rpc,xmlextras,p3p,pref,transformiix,universalchardet,typeaheadfind,webservices
ac_add_options --enable-crypto
ac_add_options --disable-composer
ac_add_options --disable-profilesharing
ac_add_options --disable-installer
ac_add_options --disable-debug
ac_add_options --disable-test
ac_add_options --enable-static
ac_add_options --disable-shared
ac_add_options --enable-strip
ac_add_options --disable-pedantic

mk_add_options MOZ_PHOENIX=1
export MOZ_PHOENIX=1



Reproducible: Always

Steps to Reproduce:
1.Build firebird (see in details)
2.Launch ir :(


Actual Results:  
zero height window :(

Expected Results:  
Open a right size window.

I think this bug is related to a bugfix in the last 12 hours. I built my
previous working Firebird 12 hours ago.

Could be a 0.7 blocker ?

Comment 1

[Stefan Borggraefe]

I compiled a Seamonkey-Build 10 hours ago and it shows the same symptoms. Every
window is opened with minimum height and width. This worked in a build made a
few hours before this.

I'm compiling with an cygwin environment on WinXP with the following options:
ac_add_options --enable-calendar
ac_add_options --disable-ldap
ac_add_options --disable-freetype2
ac_add_options --disable-postscript
ac_add_options --disable-xprint
ac_add_options --disable-installer
ac_add_options --disable-activex
ac_add_options --disable-activex-scripting
ac_add_options --disable-accessibility
ac_add_options --disable-tests
ac_add_options --enable-extensions=wallet,irc,help,inspector,spellcheck,typeahea
dfind,xml-rpc,xmlextras,venkman,p3p,cookie
ac_add_options --enable-crypto
ac_add_options --disable-debug
ac_add_options --disable-jsd
ac_add_options --enable-optimize=-O2

Comment 2

[Frederic Bezies]

Following your comments, make some tweakings to this bug report.

Comment 3

[Frederic Bezies]

Moving bug to right component, I hope. Change it if I made a mistake ;)

Comment 4

[Stefan Borggraefe]

-> Browser/XP Apps

Comment 5

[Stefan Borggraefe]

Also happens with Linux. With kwm (the KDE window manager) you see nothing when
you start Mozilla.

I guess this is because WindowsXP has a minimum allowed window-size, so you see
a very small window at least, and kwm has not.

Comment 6

[Frank Wein [:mcsmurf]]

Ok, this checkin caused this: 
09/19/2003 14:06 
jst%mozilla.jstenback.com  
mozilla/ dom/ src/ base/ nsScreen.h  
1.11  
5/2   
09/19/2003 14:06 
jst%mozilla.jstenback.com  
mozilla/ dom/ src/ base/ nsScreen.cpp  
1.17  
91/144 
Giving nsScreen a facelift. Just code cleanup. r=[e-mail], 
sr=[e-mail].  
 
Found this out by backing out one patch by another 

Comment 7

[Peter Van der Beken [:peterv]]

That's the patch for bug 218462. I'll see if I can figure this out or if we need
to back out.

Comment 8

[Peter Van der Beken [:peterv]]

Doh, patch coming up.

Comment 9

[Peter Van der Beken [:peterv]]

Attachment: Fix obvious error.

Comment 10

[Peter Van der Beken [:peterv]]

Comment on attachment 131793 [details] [diff] [review]
Fix obvious error.

I'll probably check this in right away, jst can r/sr when he wakes up.

Comment 11

[Peter Van der Beken [:peterv]]

Checked in.

Comment 12

[Frank Wein [:mcsmurf]]

This patch has fixed the problem, thx

Comment 13

[Frederic Bezies]

Wonderful ! Thanks a lot for the bugfix :)

Comment 14

[Johnny Stenback (:jst)]

Comment on attachment 131793 [details] [diff] [review]
Fix obvious error.

Geez, duh, how lame! Gotta love cut n' paste.

r+sr=jst

Comment 15

[Peter Van der Beken [:peterv]]

Fixed.

Comment 16

[Frank Wein [:mcsmurf]]

jo V.