Mac Panels interaction with Dock modifications. (step 1)

chrome/browser/ui/panels/panel_window_controller_cocoa.mm

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/panels/panel_window_controller_cocoa.h"
 
 #import <Cocoa/Cocoa.h>
 
 #include "base/logging.h"
 #include "base/mac/mac_util.h"
@@ skipping 22 matching lines @@
 #include "chrome/common/chrome_notification_types.h"
 #include "content/browser/renderer_host/render_widget_host_view.h"
 #include "content/browser/tab_contents/tab_contents.h"
 #include "content/public/browser/notification_service.h"
 #include "grit/ui_resources.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/image/image.h"
 #include "ui/gfx/mac/nsimage_cache.h"
 
 const int kMinimumWindowSize = 1;
-const double kBoundsChangeAnimationDuration = 0.18;  // Seconds.
+const double kBoundsChangeAnimationSpeed = 1000;  // Pixels per second.

[jennb, 2011/11/17 23:45:40]

Prefer units in var name. kBoundsAnimationPixelsPerSec? kBoundsAnimationMaxSecs?

[Dmitry Titov, 2011/11/18 00:15:11]

Done.

+const double kBoundsChangeAnimationMaxDuration = 0.18;  // Seconds.
 
 // Replicate specific 10.6 SDK declarations for building with prior SDKs.
 #if !defined(MAC_OS_X_VERSION_10_6) || \
     MAC_OS_X_VERSION_MAX_ALLOWED < MAC_OS_X_VERSION_10_6
 
 enum {
   NSWindowCollectionBehaviorParticipatesInCycle = 1 << 5
 };
 
 #endif  // MAC_OS_X_VERSION_10_6
@@ skipping 50 matching lines @@
   return [BrowserWindowUtils themePatternPhaseFor:windowView withTabStrip:nil];
 }
 
 - (void)awakeFromNib {
   NSWindow* window = [self window];
 
   DCHECK(window);
   DCHECK(titlebar_view_);
   DCHECK_EQ(self, [window delegate]);
 
-  // Using NSModalPanelWindowLevel (8) rather then NSStatusWindowLevel (25)
-  // ensures notification balloons on top of regular windows, but below
-  // popup menus which are at NSPopUpMenuWindowLevel (101) and Spotlight
-  // drop-out, which is at NSStatusWindowLevel-2 (23) for OSX 10.6/7.
-  // See http://crbug.com/59878.
-  [window setLevel:NSModalPanelWindowLevel];
+  [window setLevel:NSStatusWindowLevel];

[jennb, 2011/11/17 23:45:40]

I'm assuming the comment no longer applies because we hacked notifications to
always display above panels?

[Dmitry Titov, 2011/11/18 00:15:11]

It never actually applied because it was a copy-paste from Notifications code
and this specific fix was there to fix interaction with Spotlight window. I
initially moved this constant here because it is a better choice in general
(what if a panel will overlap with Spotlight popout?) but now that we want to be
on top of the Dock the more traditional recommendation of using
ModalPanelWindowLevel for always-on-top works better.

 
   if (base::mac::IsOSSnowLeopardOrLater()) {
     [window setCollectionBehavior:
         NSWindowCollectionBehaviorParticipatesInCycle];
   }
 
   [titlebar_view_ attach];
 
   // Set initial size of the window to match the size of the panel to give
   // the renderer the proper size to work with earlier, avoiding a resize
@@ skipping 306 matching lines @@
 
   NSArray *animations = [NSArray arrayWithObjects:windowResize, nil];
 
   // Terminate previous animation, if it is still playing.
   [self terminateBoundsAnimation];
   boundsAnimation_ =
       [[NSViewAnimation alloc] initWithViewAnimations:animations];
   [boundsAnimation_ setDelegate:self];
 
   [boundsAnimation_ setAnimationBlockingMode: NSAnimationNonblocking];
-  [boundsAnimation_ setDuration: kBoundsChangeAnimationDuration];
+
+  NSRect currentFrame = [[self window] frame];
+  // Compute duration. We use constant speed of animation, however if the change
+  // is too large, we clip the duration (effectively increasing speed) to
+  // limit total duration of animation. This helps to make 'big changes' take
+  // specific well-percieved time, while making smaller updates quicker, which

[jennb, 2011/11/17 23:45:40]

perceived

[jennb, 2011/11/17 23:45:40]

Did you mean slower instead of quicker? (I admit to having trouble parsing this
comment.)

[Dmitry Titov, 2011/11/18 00:15:11]

Done.

[Dmitry Titov, 2011/11/18 00:15:11]

Re-worded, made shorter.

[Dmitry Titov, 2011/11/18 00:15:11]

Done.

+  // is intuitively 'correct'.
+  // 'distance' is the max travel  between 4 potentially traveling corners.

[jennb, 2011/11/17 23:45:40]

extra space in line

[Dmitry Titov, 2011/11/18 00:15:11]

Done.

+  double distanceX = std::max(abs(NSMinX(currentFrame) - NSMinX(frame)),
+                              abs(NSMaxX(currentFrame) - NSMaxX(frame)));
+  double distanceY = std::max(abs(NSMinY(currentFrame) - NSMinY(frame)),
+                              abs(NSMaxY(currentFrame) - NSMaxY(frame)));
+  double distance = std::max(distanceX, distanceY);
+  double duration = std::min(distance / kBoundsChangeAnimationSpeed,
+                             kBoundsChangeAnimationMaxDuration);
+  [boundsAnimation_ setDuration: duration];
   [boundsAnimation_ startAnimation];
 }
 
 - (void)animationDidEnd:(NSAnimation*)animation {
   if (windowShim_->panel()->expansion_state() == Panel::EXPANDED)
     [self enableTabContentsViewAutosizing];
 
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_PANEL_BOUNDS_ANIMATIONS_FINISHED,
       content::Source<Panel>(windowShim_->panel()),
@@ skipping 101 matching lines @@
 - (BOOL)canBecomeKeyWindow {
   // Panel can only gain focus if it is expanded. Minimized panels do not
   // participate in Cmd-~ rotation.
   // TODO(dimich): If it will be ever desired to expand/focus the Panel on
   // keyboard navigation or via main menu, the care should be taken to avoid
   // cases when minimized Panel is getting keyboard input, invisibly.
   return windowShim_->panel()->expansion_state() == Panel::EXPANDED;
 }
 
 @end