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

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

Issue 7242017: Support minimizing the panel into 3-pixel line on Windows. Also support bringing up/down the titl... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 9 years, 6 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
Index: chrome/browser/ui/panels/panel_manager.cc
===================================================================
--- chrome/browser/ui/panels/panel_manager.cc (revision 90291)
+++ chrome/browser/ui/panels/panel_manager.cc (working copy)
@@ -39,11 +39,18 @@
// static
PanelManager* PanelManager::GetInstance() {
if (!panel_instance.get()) {
- panel_instance.reset(new PanelManager());
+ panel_instance.reset(PanelManager::Create());
}
return panel_instance.get();
}
+// static
+#if !defined(OS_WIN)
+PanelManager* PanelManager::Create() {
jennb 2011/06/27 23:18:02 Why only when not Windows?
jianli 2011/06/29 01:28:12 Not needed. Reverted.
+ return new PanelManager();
+}
+#endif
+
PanelManager::PanelManager()
: max_width_(0),
max_height_(0),
@@ -318,17 +325,25 @@
return true;
}
+void PanelManager::Minimize(Panel* panel) {
+ panel->Minimize();
+}
+
+void PanelManager::Restore(Panel* panel, bool titlebar_only) {
+ panel->Restore(titlebar_only);
+}
+
void PanelManager::MinimizeAll() {
for (ActivePanels::const_iterator iter = active_panels_.begin();
iter != active_panels_.end(); ++iter) {
- (*iter)->Minimize();
+ Minimize(*iter);
}
}
void PanelManager::RestoreAll() {
for (ActivePanels::const_iterator iter = active_panels_.begin();
iter != active_panels_.end(); ++iter) {
- (*iter)->Restore();
+ Restore(*iter, false);
}
}

Powered by Google App Engine
This is Rietveld 408576698