Show system minimize button for detached panels

BUG=178322
TEST=updated test
TBR=dimich@chromium.org


Review URL: https://chromiumcodereview.appspot.com/12378026

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@186071 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 7 insertions, 6 deletions

diff --git a/chrome/browser/ui/panels/detached_panel_collection.cc b/chrome/browser/ui/panels/detached_panel_collection.cc
index a0d3b5d..e434500 100644
--- a/chrome/browser/ui/panels/detached_panel_collection.cc
+++ b/chrome/browser/ui/panels/detached_panel_collection.cc
@@ -156,23 +156,24 @@ void DetachedPanelCollection::RestorePanel(Panel* panel) {
 
 void DetachedPanelCollection::OnMinimizeButtonClicked(
     Panel* panel, panel::ClickModifier modifier) {
-  // Detached panels do not minimize.
-  NOTREACHED();
+  panel->MinimizeBySystem();
 }
 
 void DetachedPanelCollection::OnRestoreButtonClicked(
     Panel* panel, panel::ClickModifier modifier) {
-  // Detached panels do not minimize.
+  // No restore button is present.
   NOTREACHED();
 }
 
 bool DetachedPanelCollection::CanShowMinimizeButton(const Panel* panel) const {
-  // Detached panels do not minimize.
-  return false;
+  // We also show minimize button for detached panel when stacking mode is
+  // enabled.
+  return PanelManager::IsPanelStackingEnabled();
 }
 
 bool DetachedPanelCollection::CanShowRestoreButton(const Panel* panel) const {
-  // Detached panels do not minimize.
+  // The minimize button is used for system minimize and thus there is no
+  // restore button.
   return false;
 }