[startup-notification] Two windows started when Firefox launched for the first times

RESOLVED FIXED in Firefox 4.0

Status

()

P4
trivial
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: vicky.sun, Assigned: ginnchen+exoracle)

Tracking

3.6 Branch
Firefox 4.0
All
OpenSolaris
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

8 years ago
Created attachment 486567 [details]
Screenshot

Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.2.12) Gecko/20101027 Firefox/3.6.12

Steps to reproduce:
1. Remove .mozilla
2. Launch FF3.6.12 from menu (not terminal)

Actual Result:
Two windows appear in panel: "Welcome to Firefox..." and "Starting Firefox Web Browser"
For screenshot,please refer to attachment 

Expected Result:
Only "Welcome to Firefox..." should be started
(Reporter)

Updated

8 years ago
Priority: -- → P4
(Assignee)

Comment 1

8 years ago
StartupNotify problem.
(Assignee)

Comment 2

8 years ago
If $/.mozilla is not there, we do LaunchChild() in ImportProfiles().
But we didn't set DESKTOP_STARTUP_ID in this path.

We might want to always set DESKTOP_STARUP_ID again, and unset it when Firefox is really started.
Summary: Two windows started when Firefox launched for the first times → [startup-notification] Two windows started when Firefox launched for the first times
(Assignee)

Comment 3

8 years ago
Created attachment 493652 [details] [diff] [review]
patch

See also Bug 416053, Bug 534845
Assignee: nobody → ginn.chen
Status: NEW → ASSIGNED
Attachment #493652 - Flags: review?(benjamin)

Updated

8 years ago
Attachment #493652 - Flags: review?(benjamin) → review+
(Assignee)

Comment 4

8 years ago
Comment on attachment 493652 [details] [diff] [review]
patch

low risk, actually startupid is not used without --enable-startup-notification
Attachment #493652 - Flags: approval2.0?
Comment on attachment 493652 [details] [diff] [review]
patch

But is that flag used by any Linux distros?
Comment on attachment 493652 [details] [diff] [review]
patch

Clearing flag; Ginn, renominate if you have an answer for the previous comment.
Attachment #493652 - Flags: approval2.0?
(Assignee)

Comment 7

8 years ago
Comment on attachment 493652 [details] [diff] [review]
patch

Firefox in Ubuntu 10.10 uses --enable-startup-notification.
Attachment #493652 - Flags: approval2.0?
Comment on attachment 493652 [details] [diff] [review]
patch

I'd like to get glandium's sign off on this, then.
Attachment #493652 - Flags: superreview?(mh+mozilla)
Comment on attachment 493652 [details] [diff] [review]
patch

I can reproduce the issue on Debian (we do use --enable-startup-notification on our builds), and I can confirm the patch fixes it.

Note that there is another related issue that the patch doesn't fix (and that I hadn't noticed before): when you have several profiles in $HOME/.mozilla/firefox, and uncheck the "Don't ask at startup" box, the "waiting icon" is displayed while being in the profile manager.
Attachment #493652 - Flags: superreview?(mh+mozilla) → superreview+
(Assignee)

Comment 10

8 years ago
That is a different problem.
We don't set startup id for the profile manager dialog.
Comment on attachment 493652 [details] [diff] [review]
patch

a=beltzner
Attachment #493652 - Flags: approval2.0? → approval2.0+
(Assignee)

Comment 12

8 years ago
http://hg.mozilla.org/mozilla-central/rev/f011b244508d
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0
You need to log in before you can comment on or make changes to this bug.