Enable user resizing for docked Panels (GTK and Mac).

chrome/browser/ui/panels/panel_resize_browsertest.cc

 // Copyright (c) 2012 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/base_panel_browser_test.h"
 #include "chrome/browser/ui/panels/detached_panel_strip.h"
 #include "chrome/browser/ui/panels/panel.h"
 #include "chrome/browser/ui/panels/panel_manager.h"
 #include "chrome/browser/ui/panels/panel_resize_controller.h"
 
 class PanelResizeBrowserTest : public BasePanelBrowserTest {
  public:
   PanelResizeBrowserTest() : BasePanelBrowserTest() {
   }
 
   virtual ~PanelResizeBrowserTest() {
   }
 
   virtual void SetUpOnMainThread() OVERRIDE {
     BasePanelBrowserTest::SetUpOnMainThread();
 
     // All the tests here assume 800x600 work area. Do the check now.
     gfx::Rect display_area = PanelManager::GetInstance()->
         display_settings_provider()->GetDisplayArea();
     DCHECK(display_area.width() == 800);
     DCHECK(display_area.height() == 600);
   }
 };
 
-IN_PROC_BROWSER_TEST_F(PanelResizeBrowserTest, DockedPanelsAreNotResizable) {
+IN_PROC_BROWSER_TEST_F(PanelResizeBrowserTest, DockedPanelResizability) {
   PanelManager* panel_manager = PanelManager::GetInstance();
   Panel* panel = CreatePanel("Panel");
 
-  EXPECT_FALSE(panel->CanResizeByMouse());
+  EXPECT_EQ(panel::ALL_SIDES_EXCEPT_BOTTOM, panel->CanResizeByMouse());
 
   gfx::Rect bounds = panel->GetBounds();
 
-  // Try resizing by the top left corner; verify resize won't work.
+  // Try resizing by the top left corner; verify resize works.

[jianli, 2012/04/13 00:52:36]

I think it would be better to test all other edges now since some of edges can
be used for resizing while some others cannot.

[jennb, 2012/04/13 19:48:04]

Done.

   gfx::Point mouse_location = bounds.origin();
   panel_manager->StartResizingByMouse(panel, mouse_location,
                                       panel::RESIZE_TOP_LEFT);
   mouse_location.Offset(-20, -20);
   panel_manager->ResizeByMouse(mouse_location);
+  bounds.Offset(-20, -20);
   EXPECT_EQ(bounds, panel->GetBounds());
 
   panel_manager->EndResizingByMouse(false);
   EXPECT_EQ(bounds, panel->GetBounds());
 
   // Try resizing by the bottom side; verify resize won't work.
   mouse_location = bounds.origin().Add(gfx::Point(10, bounds.height() - 1));
   panel_manager->StartResizingByMouse(panel, mouse_location,
                                       panel::RESIZE_BOTTOM);
   mouse_location.Offset(30, -10);
@@ skipping 15 matching lines @@
   panel_manager->EndResizingByMouse(false);
   EXPECT_EQ(bounds, panel->GetBounds());
 
   panel->Close();
 }
 
 IN_PROC_BROWSER_TEST_F(PanelResizeBrowserTest, ResizeDetachedPanel) {
   PanelManager* panel_manager = PanelManager::GetInstance();
   Panel* panel = CreateDetachedPanel("Panel", gfx::Rect(300, 200, 150, 100));
 
-  EXPECT_TRUE(panel->CanResizeByMouse());
+  EXPECT_EQ(panel::ALL_SIDES, panel->CanResizeByMouse());
 
   gfx::Rect bounds = panel->GetBounds();
 
   // Try resizing by the right side; verify resize will change width only.
   gfx::Point mouse_location = bounds.origin().Add(
       gfx::Point(bounds.width() - 1, 30));
   panel_manager->StartResizingByMouse(panel, mouse_location,
                                       panel::RESIZE_RIGHT);
   mouse_location.Offset(5, 25);
   panel_manager->ResizeByMouse(mouse_location);
@@ skipping 46 matching lines @@
   panel_manager->EndResizingByMouse(false);
   EXPECT_EQ(bounds, panel->GetBounds());
 
   PanelManager::GetInstance()->CloseAll();
 }
 
 IN_PROC_BROWSER_TEST_F(PanelResizeBrowserTest, ResizeDetachedPanelToClampSize) {
   PanelManager* panel_manager = PanelManager::GetInstance();
   Panel* panel = CreateDetachedPanel("Panel", gfx::Rect(300, 200, 150, 100));
 
-  EXPECT_TRUE(panel->CanResizeByMouse());
+  EXPECT_EQ(panel::ALL_SIDES, panel->CanResizeByMouse());
 
   gfx::Rect bounds = panel->GetBounds();
 
   // Make sure the panel does not resize smaller than its min size.
   gfx::Point mouse_location = bounds.origin().Add(
       gfx::Point(30, bounds.height() - 2));
   panel_manager->StartResizingByMouse(panel, mouse_location,
                                       panel::RESIZE_BOTTOM);
   mouse_location.Offset(-20, -500);
   panel_manager->ResizeByMouse(mouse_location);
@@ skipping 81 matching lines @@
   EXPECT_EQ(panel1_bounds, panel1->GetBounds());
 
   panel_manager->CloseAll();
 }
 
 IN_PROC_BROWSER_TEST_F(PanelResizeBrowserTest, ResizeAndCancel) {
   PanelManager* panel_manager = PanelManager::GetInstance();
   Panel* panel = CreateDetachedPanel("Panel", gfx::Rect(300, 200, 150, 100));
   PanelResizeController* resize_controller = panel_manager->resize_controller();
 
-  EXPECT_TRUE(panel->CanResizeByMouse());
+  EXPECT_EQ(panel::ALL_SIDES, panel->CanResizeByMouse());
 
   gfx::Rect original_bounds = panel->GetBounds();
 
   // Resizing the panel, then cancelling should return it to the original state.
   // Try resizing by the top right side.
   gfx::Rect bounds = panel->GetBounds();
   gfx::Point mouse_location = bounds.origin().Add(
       gfx::Point(bounds.width() - 1, 1));
   panel_manager->StartResizingByMouse(panel, mouse_location,
                                       panel::RESIZE_TOP_RIGHT);
@@ skipping 19 matching lines @@
   bounds.set_size(gfx::Size(bounds.width() + 10, bounds.height() + 15));
   bounds.Offset(-10, 0);
   EXPECT_EQ(bounds, panel->GetBounds());
 
   panel_manager->EndResizingByMouse(true);
   EXPECT_EQ(original_bounds, panel->GetBounds());
   EXPECT_FALSE(resize_controller->IsResizing());
 
   panel_manager->CloseAll();
 }