Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(37)

Issue 660395: [Mac] Save window position into the SessionService, so we can restore it afte... (Closed)

Created:
10 years, 9 months ago by rohitrao (ping after 24h)
Modified:
9 years, 7 months ago
Reviewers:
viettrungluu
CC:
chromium-reviews, John Grabowski, pam+watch_chromium.org, ben+cc_chromium.org
Visibility:
Public.

Description

[Mac] Save window position into the SessionService, so we can restore it after a crash. BUG=20909 TEST=Start chrome, load some pages/tabs, move the window somewhere else. Click away and back to the window (to trigger saving window position). Kill chrome (you may have to use kill -9 to actually kill it uncleanly). Restart, click to restore pages. Window should jump to where it was before the crash. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=41248

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
M chrome/browser/cocoa/browser_window_controller_private.mm View 1 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
rohitrao (ping after 24h)
Nothing too exciting, code copied from browser_window_gtk.cc. I spent a while trying to mock out ...
10 years, 9 months ago (2010-03-02 20:57:54 UTC) #1
viettrungluu
lgtm
10 years, 9 months ago (2010-03-02 20:59:42 UTC) #2
viettrungluu
So are you planning on committing this any time soon, or shall I?
10 years, 9 months ago (2010-03-11 02:08:57 UTC) #3
rohitrao (ping after 24h)
10 years, 9 months ago (2010-03-11 02:41:33 UTC) #4
On 2010/03/11 02:08:57, viettrungluu wrote:
> So are you planning on committing this any time soon, or shall I?

Ok, ok, committing.

Powered by Google App Engine
This is Rietveld 408576698