Refactor intra-strip panel drags by introducing PanelDragController.

chrome/browser/ui/panels/overflow_panel_strip.cc

Index: chrome/browser/ui/panels/overflow_panel_strip.cc
diff --git a/chrome/browser/ui/panels/overflow_panel_strip.cc b/chrome/browser/ui/panels/overflow_panel_strip.cc
index 446dcb0bd6b377e43e8c18f45ec2e670bf3f2bb3..104ec7aff06e06d1b4c4c7bec132e730f2c4ee9b 100644
--- a/chrome/browser/ui/panels/overflow_panel_strip.cc
+++ b/chrome/browser/ui/panels/overflow_panel_strip.cc
@@ -98,7 +98,6 @@ void OverflowPanelStrip::AddPanel(Panel* panel) {
 
   // Set panel properties for this strip.
   panel->SetAppIconVisibility(false);
-  panel->set_draggable(false);
   panel->ApplyVisualStyleForStrip();
 
   if (num_panels() == 1) {
@@ -182,6 +181,23 @@ void OverflowPanelStrip::RestorePanel(Panel* panel) {
   docked_strip->RestorePanel(panel);
 }
 
+bool OverflowPanelStrip::CanDragPanel(Panel* panel) const {
+  // All overflow panels are not draggable.
+  return false;
+}
+
+void OverflowPanelStrip::StartDraggingPanel(Panel* panel) {
+  // Nothing to do.

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

NOTREACHED() for all three methods? Will ensure we don't accidentally call into
the overflow strip.

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

Done.

+}
+
+void OverflowPanelStrip::DragPanel(int delta_x, int delta_y) {
+  // Nothing to do.
+}
+
+void OverflowPanelStrip::EndDraggingPanel(bool cancelled) {
+  // Nothing to do.
+}
+
 void OverflowPanelStrip::RefreshLayout() {
   if (panels_.empty())
     return;