The default bug view has changed. See this FAQ.

If FF Window is minimized and restarted, the window is minimized/maximized then restored

RESOLVED FIXED in Firefox 8

Status

()

Firefox
Session Restore
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Cpuroast, Assigned: zpao)

Tracking

Trunk
Firefox 8
All
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

6 years ago
User-Agent:       Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0) Gecko/20100101 Firefox/4.0
Build Identifier: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0) Gecko/20100101 Firefox/4.0

If a Firefox window is in a minimized state when Firefox is quit. The next time Firefox is launched the window will go through a quick and odd minimize/restore sequence.

Reproducible: Always

Steps to Reproduce:
1.Minimize a Window
2.Quit Firefox
3.Re-launch Firefox
Actual Results:  
On re-launch the Window goes through a quick and odd minimize/restore sequence.

Expected Results:  
On re-launch the window is restored without being minimized first.
(Reporter)

Updated

6 years ago
Version: unspecified → 4.0 Branch

Comment 1

6 years ago
Able to reproduce using  Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0)
Gecko/20100101 Firefox/4.0 but only if "When Firefox starts" from Preferences->General is set to "Show my windows and tabs from last time".

Updated

6 years ago
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: When a Firefox Window is minimized and Firefox is quit then re-launched, the window goes through a quick and odd minimize/restore sequence. → If FF Window is minimized and restarted, the window is minimized/maximized then restored
Is it safe to assume that you only have one Firefox window open at the time?
(Reporter)

Comment 3

6 years ago
Yup just 1 window, 

behavior is still present in Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:5.0) Gecko/20100101 Firefox/5.0
Created attachment 548957 [details] [diff] [review]
Patch v0.1

Simple fix.
Assignee: nobody → paul
Status: NEW → ASSIGNED
Attachment #548957 - Flags: review?(dietrich)
Comment on attachment 548957 [details] [diff] [review]
Patch v0.1

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

Not sure it's possible to reasonably write a test for this, so please flag for manual testing. r=me.

::: browser/components/sessionstore/src/nsSessionStore.js
@@ +352,5 @@
>            this._sessionStartTime = this._initialState.session &&
>                                     this._initialState.session.startTime ||
>                                     this._sessionStartTime;
>  
> +          let firstWindow = this._initialState.windows[0];

Actually, I liked it without the temp var, since it's abundantly clear that we're manipulating the data-to-be-restored before the restore happens.
Attachment #548957 - Flags: review?(dietrich) → review+
Created attachment 551833 [details] [diff] [review]
Patch v1.0

Addressed comments
http://hg.mozilla.org/integration/mozilla-inbound/rev/f19cab5d3f77
Hardware: x86_64 → All
Whiteboard: [inbound]
Version: 4.0 Branch → Trunk
http://hg.mozilla.org/mozilla-central/rev/f19cab5d3f77
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → Firefox 8
You need to log in before you can comment on or make changes to this bug.