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

Unified Diff: chrome/browser/ui/panels/panel_manager.h

Issue 8341098: Mac: Added a delay before panels go to minimized state from title-only state. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/ui/panels/panel_manager.cc » ('j') | chrome/browser/ui/panels/panel_manager.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/panels/panel_manager.h
diff --git a/chrome/browser/ui/panels/panel_manager.h b/chrome/browser/ui/panels/panel_manager.h
index b419c8f91116cf1955e58a794e86f0fa770af259..f5ce7b688e5423e336b854fb5f93cccfe1ee8c68 100644
--- a/chrome/browser/ui/panels/panel_manager.h
+++ b/chrome/browser/ui/panels/panel_manager.h
@@ -152,11 +152,6 @@ class PanelManager : public PanelMouseWatcher::Observer,
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);
@@ -201,9 +196,12 @@ class PanelManager : public PanelMouseWatcher::Observer,
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
Dmitry Titov 2011/10/28 20:58:23 Fixed style. Tried to figure out simple way that w
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
« no previous file with comments | « no previous file | chrome/browser/ui/panels/panel_manager.cc » ('j') | chrome/browser/ui/panels/panel_manager.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698