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

chrome/browser/ui/panels/panel.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.h"
 
 #include "base/logging.h"
 #include "chrome/browser/extensions/extension_prefs.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/panels/native_panel.h"
 #include "chrome/browser/ui/panels/panel_manager.h"
 #include "chrome/browser/web_applications/web_app.h"
 #include "chrome/common/extensions/extension.h"
 #include "ui/gfx/rect.h"
 
 // static
 const Extension* Panel::GetExtension(Browser* browser) {
   // Find the extension. When we create a panel from an extension, the extension
   // ID is passed as the app name to the Browser.
   ExtensionService* extension_service =
       browser->GetProfile()->GetExtensionService();
   return extension_service->GetExtensionById(
       web_app::GetExtensionIdFromApplicationName(browser->app_name()), false);
 }
 
-Panel::Panel(Browser* browser, const gfx::Rect& bounds) {
+Panel::Panel(Browser* browser, const gfx::Rect& bounds)
+    : native_panel_(NULL),

[jennb, 2011/06/29 23:17:54]

Why does native_panel_ need to be set to NULL here?

[jianli, 2011/06/30 01:28:33]

I think it is good to initialize the pointer here, just to be safe, in case that
people move the code to set native_panel_ out of the constructor.

+      expansion_state_(EXPANDED) {
   native_panel_ = CreateNativePanel(browser, this, bounds);
 }
 
 Panel::~Panel() {
   // Invoked by native panel so do not access native_panel_ here.
 }
 
 PanelManager* Panel::manager() const {
   return PanelManager::GetInstance();
 }
 
 void Panel::SetPanelBounds(const gfx::Rect& bounds) {
   native_panel_->SetPanelBounds(bounds);
 }
 
-void Panel::Minimize() {
-  native_panel_->MinimizePanel();
+void Panel::SetExpansionState(ExpansionState expansion_state) {
+  if (expansion_state_ == expansion_state)
+    return;
+  expansion_state_ = expansion_state;
+
+  native_panel_->SetPanelExpansionState(expansion_state);
 }
 
-void Panel::Restore() {
-  native_panel_->RestorePanel();
+bool Panel::ShouldBringUpTitleBar(int mouse_x, int mouse_y) const {
+  // Skip the expanded panel.
+  if (expansion_state_ == Panel::EXPANDED)

[jennb, 2011/06/29 23:17:54]

if (expansion_state_ != Panel::MINIMIZED)

[jianli, 2011/06/30 01:28:33]

If the panel state is TITLEBAR_ONLY, we still want to check if the mouse point
is inside the titlebar bounds.

+    return false;
+
+  // Let the native panel decide.
+  return native_panel_->ShouldBringUpPanelTitleBar(mouse_x, mouse_y);
 }
 
 void Panel::Show() {
   native_panel_->ShowPanel();
 }
 
 void Panel::ShowInactive() {
   native_panel_->ShowPanelInactive();
 }
 
@@ skipping 313 matching lines @@
 
 #if defined(OS_CHROMEOS)
 void Panel::ShowKeyboardOverlay(gfx::NativeWindow owning_window) {
   NOTIMPLEMENTED();
 }
 #endif
 
 void Panel::DestroyBrowser() {
   native_panel_->DestroyPanelBrowser();
 }