Closed
Bug 529674
Opened 15 years ago
Closed 15 years ago
Restore windows at the saved position without moving them around on the screen
Categories
(Firefox :: Session Restore, defect)
Tracking
()
VERIFIED
FIXED
Firefox 3.7a1
People
(Reporter: whimboo, Assigned: zpao)
References
Details
Attachments
(1 file)
1.79 KB,
patch
|
zeniko
:
review+
|
Details | Diff | Splinter Review |
Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.2b4pre) Gecko/20091118 Namoroka/3.6b4pre ID:20091118034701 When a couple of windows are open and you start/stop the PB mode or restart Firefox all windows are restored at the position of the first window and moved to their target (saved) position afterward. That doesn't look nice. Can we directly restore the windows at their saved position? Steps: 1. Open a couple of windows 2. Start/Stop the Private Browsing mode Once you stop the Private Browsing mode you will see the mentioned behavior from above.
Assignee | ||
Comment 1•15 years ago
|
||
We should be able to now, thanks to bug 509828 (which I think was the problem). I haven't tried yet, but it should be a trivial change. This is on my (growing) list of changes to make to sessionstore after 3.6.
Assignee | ||
Comment 2•15 years ago
|
||
So I think this is all we need. I think we want to leave restoreDimensions as is (for when we reuse a window and setting sizemode). As for testing... we call restoreDimensions in a setTimeout, so perhaps we could observe domwindowopened with window watcher & that'll fire before the restoreDimensions gets called? I can try. Otherwise is there a way to test this automatically or would we need a litmus test?
Assignee: nobody → paul
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•15 years ago
|
||
Comment on attachment 414337 [details] [diff] [review] Patch v0.1 There's a mochitest already to ensure that openWindow with height, etc. work. I tried to write another here, but it's not working out so well, so it doesn't seem worth it. I also looked into doing sizemode at the same time, but that'd be a bigger change down in windowwatcher/widget to get working, so not worth doing here.
Attachment #414337 -
Attachment description: Patch v0.1 (WIP) → Patch v0.1
Attachment #414337 -
Flags: review?(zeniko)
Updated•15 years ago
|
Attachment #414337 -
Flags: review?(zeniko) → review+
Assignee | ||
Comment 4•15 years ago
|
||
Pushed http://hg.mozilla.org/mozilla-central/rev/c89bea57ceb7
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.7a1
Assignee | ||
Comment 5•15 years ago
|
||
We can test this with a litmus test. Just save a session with multiple windows open & positioned around the screen. Quit and restore. The first window will still move around because we don't open that, but the subsequent windows should open where there were at quit.
Flags: in-litmus?
2 test cases added: https://litmus.mozilla.org/show_test.cgi?id=11511 https://litmus.mozilla.org/show_test.cgi?id=11512 Marking in-litmus+
Flags: in-litmus? → in-litmus+
You need to log in
before you can comment on or make changes to this bug.
Description
•