Remove Animation When "Resuming" Window in Lion

chrome/browser/ui/cocoa/browser_window_cocoa.mm

Index: chrome/browser/ui/cocoa/browser_window_cocoa.mm
diff --git a/chrome/browser/ui/cocoa/browser_window_cocoa.mm b/chrome/browser/ui/cocoa/browser_window_cocoa.mm
index cf51c3f46b0213f708bf1de62b0a8c39ed3ef9db..822ae246adb9e01b87f50af47198dcfc683dda28 100644
--- a/chrome/browser/ui/cocoa/browser_window_cocoa.mm
+++ b/chrome/browser/ui/cocoa/browser_window_cocoa.mm
@@ -92,31 +92,35 @@ void BrowserWindowCocoa::Show() {
   // the previous browser instead if we don't explicitly set it here.
   BrowserList::SetLastActive(browser_);
 
-  ui::WindowShowState show_state = browser_->GetSavedWindowShowState();
-  if (show_state == ui::SHOW_STATE_MINIMIZED) {
-    // Turn off swishing when restoring minimized windows.  When creating
-    // windows from nibs it is necessary to |orderFront:| prior to |orderOut:|
-    // then |miniaturize:| when restoring windows in the minimized state.
-    NSWindowAnimationBehavior savedAnimationBehavior = 0;
-    if ([window() respondsToSelector:@selector(animationBehavior)] &&
-        [window() respondsToSelector:@selector(setAnimationBehavior:)]) {
-      savedAnimationBehavior = [window() animationBehavior];
-      [window() setAnimationBehavior:NSWindowAnimationBehaviorNone];
-    }
-
-    [window() makeKeyAndOrderFront:controller_];
+  bool is_session_restore = browser_->is_session_restore();
+  NSWindowAnimationBehavior saved_animation_behavior = 0;

[Mark Mentovai, 2011/08/31 17:58:54]

This no longer needs to be initialized to 0, since it’ll only be used if
did_save_animation_behavior is true, and in that case it will always have been
set.

[dhollowa, 2011/08/31 18:40:55]

Done.

+  bool did_save_animation_behavior = false;
+  // Turn off swishing when restoring windows.
+  if (is_session_restore &&
+      [window() respondsToSelector:@selector(animationBehavior)] &&
+      [window() respondsToSelector:@selector(setAnimationBehavior:)]) {
+    saved_animation_behavior = [window() animationBehavior];
+    [window() setAnimationBehavior:NSWindowAnimationBehaviorNone];
+    did_save_animation_behavior = true;

[Mark Mentovai, 2011/08/31 17:58:54]

I would do this before changing the value, just because it flows better
conceptually.

[dhollowa, 2011/08/31 18:40:55]

Done.

+  }
+
+  [window() makeKeyAndOrderFront:controller_];
 
+  // When creating windows from nibs it is necessary to |makeKeyAndOrderFront:|
+  // prior to |orderOut:| then |miniaturize:| when restoring windows in the
+  // minimized state.
+  if (browser_->GetSavedWindowShowState() == ui::SHOW_STATE_MINIMIZED) {
     [window() orderOut:controller_];
     [window() miniaturize:controller_];
+  }
 
+  if (did_save_animation_behavior) {
     // Restore window animation behavior.
-    if ([window() respondsToSelector:@selector(animationBehavior)] &&
-        [window() respondsToSelector:@selector(setAnimationBehavior:)]) {
-      [window() setAnimationBehavior:savedAnimationBehavior];
-    }
-  } else {
-    [window() makeKeyAndOrderFront:controller_];
+    [window() setAnimationBehavior:saved_animation_behavior];
   }
+
+  // Once the window has been shown, clear the session restore flag.
+  browser_->set_is_session_restore(false);

[Mark Mentovai, 2011/08/31 17:58:54]

This is the only platform on which this happens, right?

Not so sure about that. Is there a more cross-platformy place to do this?

The parameterized form worked better for this aspect, I think.

[dhollowa, 2011/08/31 18:40:55]

Yes, this is Mac-only logic.  I agree though, this should be paired with the
setter in session_restore.cc.  I've done that.

For the record, I wrote it this way originally to match the usage pattern of
show_state_.  I agree the parameterized version is cleaner in this regard, it
just comes at such a high cost to other callers and sub-classers.  Also,
thinking about future work with Fullscreen, I can imagine needing this state
outside of |Show|.

 }
 
 void BrowserWindowCocoa::ShowInactive() {