Mac: Added a delay before panels go to minimized state from title-only state.

chrome/browser/ui/panels/panel_manager.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_PANELS_PANEL_MANAGER_H_
 #define CHROME_BROWSER_UI_PANELS_PANEL_MANAGER_H_
 #pragma once
 
 #include <vector>
 #include "base/basictypes.h"
@@ skipping 134 matching lines @@
 
   // Rearranges the positions of the panels starting from the given iterator.
   // This is called when the display space has been changed, i.e. working
   // area being changed or a panel being closed.
   void Rearrange(Panels::iterator iter_to_start, int rightmost_position);
 
   // Help functions to drag the given panel.
   void DragLeft();
   void DragRight();
 
-  // Checks if the titlebars have been brought up or down. If not, do not wait
-  // for the notifications to trigger it any more, and start to bring them up or
-  // down immediately.
-  void DelayedBringUpOrDownTitlebarsCheck();
-
   // Does the real job of bringing up or down the titlebars.
   void DoBringUpOrDownTitlebars(bool bring_up);
 
   int GetMaxPanelWidth() const;
   int GetMaxPanelHeight() const;
   int GetRightMostAvailablePosition() const;
 
   Panels panels_;
 
   // Stores the panels that are pending to remove. We want to delay the removal
@@ skipping 24 matching lines @@
   // the original position when we cancel the dragging.
   int dragging_panel_original_x_;
 
   // Bounds of the panel to drag. It is first set to the original bounds when
   // the dragging happens. Then it is updated to the position that will be set
   // to when the dragging ends.
   gfx::Rect dragging_panel_bounds_;
 
   scoped_refptr<AutoHidingDesktopBar> auto_hiding_desktop_bar_;
 
+  // Delayed transitions support. Sometimes transitions between minimized and
+  // title-only states are delayed, for better usability with Taskbars/Docks.
   TitlebarAction delayed_titlebar_action_;
-
+  static int sLastTaskIndex;

[jianli, 2011/10/28 01:04:11]

The variable name does not follow the coding style guideline. In addition, can
we avoid using the static integer?

[Dmitry Titov, 2011/10/28 20:58:23]

Fixed style.
Tried to figure out simple way that would work w/o a static counter. Do you have
a good idea?

   ScopedRunnableMethodFactory<PanelManager> method_factory_;
+  void DelayedBringUpOrDownTitlebarsCheck(int taskIndex);
 
   // Whether or not bounds will be updated when the preferred content size is
   // changed. The testing code could set this flag to false so that other tests
   // will not be affected.
   bool auto_sizing_enabled_;
 
   bool mouse_watching_disabled_;  // For tests to simulate mouse movements.
 
   static const int kPanelsHorizontalSpacing = 4;
 
   // Minimum width and height of a panel.
   // Note: The minimum size of a widget (see widget.cc) is fixed to 100x100.
   // TODO(jianli): Need to fix this to support smaller panel.
   static const int kPanelMinWidth = 100;
   static const int kPanelMinHeight = 100;
 
   DISALLOW_COPY_AND_ASSIGN(PanelManager);
 };
 
 #endif  // CHROME_BROWSER_UI_PANELS_PANEL_MANAGER_H_