Refactor intra-strip panel drags by introducing PanelDragController.

chrome/browser/ui/panels/panel_manager.h

Index: chrome/browser/ui/panels/panel_manager.h
diff --git a/chrome/browser/ui/panels/panel_manager.h b/chrome/browser/ui/panels/panel_manager.h
index 1c21b24cdee0ed919794b639d8fd6e294fd6ac92..cbcc31825042f963c6a8945e21c04bade1346c9a 100644
--- a/chrome/browser/ui/panels/panel_manager.h
+++ b/chrome/browser/ui/panels/panel_manager.h
@@ -16,9 +16,11 @@
 #include "ui/gfx/rect.h"
 
 class Browser;
-class PanelMouseWatcher;
-class OverflowPanelStrip;
+class DetachedPanelStrip;
 class DockedPanelStrip;
+class OverflowPanelStrip;
+class PanelDragController;
+class PanelMouseWatcher;
 
 // This class manages a set of panels.
 class PanelManager : public AutoHidingDesktopBar::Observer {
@@ -45,8 +47,9 @@ class PanelManager : public AutoHidingDesktopBar::Observer {
   void OnPanelRemoved(Panel* panel);
 
   // Drags the given panel.
+  bool CanDrag(Panel* panel) const;

[jennb, 2012/02/16 22:31:00]

Do we want to allow panels to know about drag controller? If so, then we can
leave these methods out of PanelManager and have panel call the drag controller
methods directly.

[jianli, 2012/02/16 23:05:52]

Panels should only know about PanelManager, not drag controller.

   void StartDragging(Panel* panel);
-  void Drag(int delta_x);
+  void Drag(int delta_x, int delta_y);
   void EndDragging(bool cancelled);
 
   // Invoked when a panel's expansion state changes.
@@ -77,6 +80,10 @@ class PanelManager : public AutoHidingDesktopBar::Observer {
   int StartingRightPosition() const;
   std::vector<Panel*> panels() const;
 
+  PanelDragController* drag_controller() const {
+    return drag_controller_.get();
+  }
+
   AutoHidingDesktopBar* auto_hiding_desktop_bar() const {
     return auto_hiding_desktop_bar_;
   }
@@ -85,6 +92,10 @@ class PanelManager : public AutoHidingDesktopBar::Observer {
     return panel_mouse_watcher_.get();
   }
 
+  DetachedPanelStrip* detached_strip() const {
+    return detached_strip_.get();
+  }
+
   DockedPanelStrip* docked_strip() const {
     return docked_strip_.get();
   }
@@ -161,9 +172,12 @@ class PanelManager : public AutoHidingDesktopBar::Observer {
   // Tests may want to shorten time intervals to reduce running time.
   static bool shorten_time_intervals_;
 
+  scoped_ptr<DetachedPanelStrip> detached_strip_;
   scoped_ptr<DockedPanelStrip> docked_strip_;
   scoped_ptr<OverflowPanelStrip> overflow_strip_;
 
+  scoped_ptr<PanelDragController> drag_controller_;
+
   // Use a mouse watcher to know when to bring up titlebars to "peek" at
   // minimized panels. Mouse movement is only tracked when there is a minimized
   // panel.