Chromium Code Reviews| 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); |
| } |
| } |