Some fixes to panel support on Windows:

chrome/browser/ui/panels/panel_browser_view_browsertest.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 "base/command_line.h"
 #include "base/i18n/time_formatting.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_window.h"
@@ skipping 201 matching lines @@
     int titlebar_height =
         browser_view1->GetFrameView()->NonClientTopBorderHeight();
 
     panel1->SetExpansionState(Panel::MINIMIZED);
     EXPECT_EQ(Panel::MINIMIZED, panel1->expansion_state());
     EXPECT_LT(panel1->GetBounds().height(), titlebar_height);
     EXPECT_GT(panel1->GetBounds().height(), 0);
     EXPECT_TRUE(IsMouseWatcherStarted());
     EXPECT_FALSE(panel1->IsActive());
     WaitTillBoundsAnimationFinished(browser_view1);
-    // TODO(jianli): Enable the following checks after the patch to support
-    // minimizing window to 3-pixel line is landed.
-    //EXPECT_FALSE(frame_view1->close_button_->IsVisible());
-    //EXPECT_FALSE(frame_view1->title_icon_->IsVisible());
-    //EXPECT_FALSE(frame_view1->title_label_->IsVisible());
 
     panel1->SetExpansionState(Panel::TITLE_ONLY);
     EXPECT_EQ(Panel::TITLE_ONLY, panel1->expansion_state());
     EXPECT_EQ(titlebar_height, panel1->GetBounds().height());
     WaitTillBoundsAnimationFinished(browser_view1);
     EXPECT_TRUE(frame_view1->close_button_->IsVisible());
     EXPECT_TRUE(frame_view1->title_icon_->IsVisible());
     EXPECT_TRUE(frame_view1->title_label_->IsVisible());
 
     panel1->SetExpansionState(Panel::EXPANDED);
@@ skipping 44 matching lines @@
         panel1->GetBounds().x(), panel1->GetBounds().y()));
     EXPECT_FALSE(panel_manager->ShouldBringUpTitlebarForAllMinimizedPanels(
         panel2->GetBounds().x(), panel2->GetBounds().y()));
     EXPECT_TRUE(panel_manager->ShouldBringUpTitlebarForAllMinimizedPanels(
         panel3->GetBounds().right() - 1, panel3->GetBounds().bottom() - 1));
     EXPECT_TRUE(panel_manager->ShouldBringUpTitlebarForAllMinimizedPanels(
         panel3->GetBounds().right() - 1, panel3->GetBounds().bottom() + 10));
     EXPECT_FALSE(panel_manager->ShouldBringUpTitlebarForAllMinimizedPanels(
         0, 0));
 
+    // Test that the panel in title-only state should not be minimized
+    // regardless the current mouse position when the panel is being dragged.

[jennb, 2011/08/17 23:01:35]

s/regardless the/regardless of the

+    panel1->SetExpansionState(Panel::TITLE_ONLY);
+    EXPECT_FALSE(panel_manager->ShouldBringUpTitlebarForAllMinimizedPanels(
+        0, 0));
+    browser_view1->OnTitlebarMousePressed(panel1->GetBounds().origin());
+    browser_view1->OnTitlebarMouseDragged(
+        panel1->GetBounds().origin().Subtract(gfx::Point(5, 5)));
+    EXPECT_TRUE(panel_manager->ShouldBringUpTitlebarForAllMinimizedPanels(
+        0, 0));
+    browser_view1->OnTitlebarMouseReleased();
+
     panel1->Close();
     EXPECT_TRUE(IsMouseWatcherStarted());
     panel2->Close();
     EXPECT_TRUE(IsMouseWatcherStarted());
     panel3->Close();
     EXPECT_FALSE(IsMouseWatcherStarted());
   }
 
   void TestDrawAttention() {
     PanelBrowserView* browser_view = CreatePanelBrowserView("PanelTest",
@@ skipping 98 matching lines @@
       frame_view->title_label_->x());
   EXPECT_LT(frame_view->title_label_->x() + frame_view->title_label_->width(),
       frame_view->settings_button_->x());
   EXPECT_LT(
       frame_view->settings_button_->x() + frame_view->settings_button_->width(),
       frame_view->close_button_->x());
 
   // Validate that the controls should be updated when the activation state is
   // changed.
   frame_view->UpdateControlStyles(PanelBrowserFrameView::PAINT_AS_ACTIVE);
-  gfx::Font title_label_font1 = frame_view->title_label_->font();
+  SkColor title_label_color1 = frame_view->title_label_->GetColor();
   frame_view->UpdateControlStyles(PanelBrowserFrameView::PAINT_AS_INACTIVE);
-  gfx::Font title_label_font2 = frame_view->title_label_->font();
-  EXPECT_NE(title_label_font1.GetStyle(), title_label_font2.GetStyle());
+  SkColor title_label_color2 = frame_view->title_label_->GetColor();
+  EXPECT_NE(title_label_color1, title_label_color2);
 
   browser_view->panel()->Close();
 }
 
 IN_PROC_BROWSER_TEST_F(PanelBrowserViewTest, ShowPanelActiveOrInactive) {
   TestShowPanelActiveOrInactive();
 }
 
 IN_PROC_BROWSER_TEST_F(PanelBrowserViewTest, ShowOrHideSettingsButton) {
   PanelBrowserView* browser_view = CreatePanelBrowserView("PanelTest",
@@ skipping 65 matching lines @@
 }
 
 IN_PROC_BROWSER_TEST_F(PanelBrowserViewTest, MinimizeAndRestore) {
   TestMinimizeAndRestore();
 }
 
 IN_PROC_BROWSER_TEST_F(PanelBrowserViewTest, DrawAttention) {
   TestDrawAttention();
 }
 #endif