Support minimizing the panel into 3-pixel line on Windows. Also support bringing up/down the titl...

chrome/browser/ui/panels/panel_browser_view.cc

 // 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.
 
 #include "chrome/browser/ui/panels/panel_browser_view.h"
 
 #include "base/logging.h"
 #include "chrome/browser/ui/panels/panel.h"
 #include "chrome/browser/ui/panels/panel_browser_frame_view.h"
 #include "chrome/browser/ui/panels/panel_manager.h"
 #include "chrome/browser/ui/views/frame/browser_frame.h"
 #include "grit/chromium_strings.h"
 #include "ui/base/animation/slide_animation.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "views/widget/widget.h"
 
 namespace {
 // This value is experimental and subjective.
 const int kSetBoundsAnimationMs = 200;
+
+// The panel can be fully minimized to 3-pixel lines.
+const int kFullyMinimizedHeight = 3;
 }
 
 NativePanel* Panel::CreateNativePanel(Browser* browser, Panel* panel,
                                       const gfx::Rect& bounds) {
   PanelBrowserView* view = new PanelBrowserView(browser, panel, bounds);
-  (new BrowserFrame(view))->InitBrowserFrame();
+  BrowserFrame* frame = new BrowserFrame(view);
+  frame->set_frame_type(views::Widget::FRAME_TYPE_FORCE_CUSTOM);

[jennb, 2011/06/27 23:18:02]

Curious - why did set_frame_type have to be moved here from the BrowserFrame
constructor?

[jianli, 2011/06/29 01:28:12]

Not needed now. Reverted.

+  frame->InitBrowserFrame();
   return view;
 }
 
 PanelBrowserView::PanelBrowserView(Browser* browser, Panel* panel,
                                    const gfx::Rect& bounds)
   : BrowserView(browser),
     panel_(panel),
     bounds_(bounds),
     original_height_(bounds.height()),
-    minimized_(false),
+    expand_state_(FULLY_RESTORED),
     closed_(false),
     focused_(false),
     mouse_pressed_(false),
     mouse_dragging_(false) {
 }
 
 PanelBrowserView::~PanelBrowserView() {
 }
 
 void PanelBrowserView::Init() {
@@ skipping 100 matching lines @@
 
 gfx::Rect PanelBrowserView::GetPanelBounds() const {
   return bounds_;
 }
 
 void PanelBrowserView::SetPanelBounds(const gfx::Rect& bounds) {
   SetBounds(bounds);
 }
 
 void PanelBrowserView::MinimizePanel() {
-  if (minimized_)
+  if (expand_state_ == FULLY_MINIMIZED)
     return;
+  expand_state_ = FULLY_MINIMIZED;
 
-  minimized_ = true;
-  gfx::Rect bounds = GetPanelBounds();
-  original_height_ = bounds.height();
-  bounds.set_height(GetFrameView()->NonClientTopBorderHeight());
-  bounds.set_y(bounds.y() + original_height_ - bounds.height());
-  SetPanelBounds(bounds);
+  AdjustHeight(kFullyMinimizedHeight);
 }
 
-void PanelBrowserView::RestorePanel() {
-  if (!minimized_)
-    return;
-
-  minimized_ = false;
-  gfx::Rect bounds = GetPanelBounds();
-  bounds.set_y(bounds.y() - original_height_ + bounds.height());
-  bounds.set_height(original_height_);
-  SetPanelBounds(bounds);
+void PanelBrowserView::RestorePanel(bool titlebar_only) {
+  if (titlebar_only) {
+    // It is not allowed to "restore" from full height to titlebar height.
+    if (expand_state_ != FULLY_MINIMIZED)

[jennb, 2011/06/27 23:18:02]

Should this be a DCHECK instead?

[jianli, 2011/06/29 01:28:12]

Not needed due to logic change.

+      return;
+    expand_state_ = TITLEBAR_RESTORED;
+    AdjustHeight(GetFrameView()->NonClientTopBorderHeight());
+  } else {
+    if (expand_state_ == FULLY_RESTORED)
+      return;
+    expand_state_ = FULLY_RESTORED;
+    AdjustHeight(original_height_);
+  }
 }
 
 void PanelBrowserView::ClosePanel() {
   Close();
 }
 
 void PanelBrowserView::ActivatePanel() {
   Activate();
 }
 
@@ skipping 76 matching lines @@
   mouse_pressed_ = false;
 
   if (!mouse_dragging_)
     cancelled = true;
   mouse_dragging_ = false;
   panel_->manager()->EndDragging(cancelled);
   return true;
 }
 
 void PanelBrowserView::MinimizeOrRestore() {
-  if (minimized_)
-    panel_->Restore();
+  if (expand_state_ != FULLY_RESTORED)
+    panel_->manager()->Restore(panel_.get(), false);
   else
-    panel_->Minimize();
+    panel_->manager()->Minimize(panel_.get());
 }
+
+void PanelBrowserView::AdjustHeight(int height) {
+  gfx::Rect bounds = bounds_;
+  bounds.set_y(bounds.y() + bounds.height() - height);
+  bounds.set_height(height);
+  SetBounds(bounds);
+}