Refactor intra-strip panel drags by introducing PanelDragController.

chrome/browser/ui/panels/panel_drag_controller.h

Index: chrome/browser/ui/panels/panel_drag_controller.h
diff --git a/chrome/browser/ui/panels/panel_drag_controller.h b/chrome/browser/ui/panels/panel_drag_controller.h
new file mode 100644
index 0000000000000000000000000000000000000000..e06d406a6341584689b2348d41df5a6446d4f2c0
--- /dev/null
+++ b/chrome/browser/ui/panels/panel_drag_controller.h
@@ -0,0 +1,35 @@
+// 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.
+
+#ifndef CHROME_BROWSER_UI_PANELS_PANEL_DRAG_CONTROLLER_H_
+#define CHROME_BROWSER_UI_PANELS_PANEL_DRAG_CONTROLLER_H_
+#pragma once
+
+#include "base/basictypes.h"
+#include "ui/gfx/point.h"
+
+class Panel;
+
+// Responsible for handling drags initiated for all panel strips, including
+// both intra-strip and inter-strip drags.
+class PanelDragController {
+ public:
+  PanelDragController();
+  ~PanelDragController();
+
+  bool CanDrag(Panel* panel) const;

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

Doesn't seem like we need this here as the current panel strip can be asked
directly if a panel can be dragged.

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

Done.

+  void StartDragging(Panel* panel);
+  void Drag(int delta_x, int delta_y);
+  void EndDragging(bool cancelled);
+
+  Panel* dragging_panel() const { return dragging_panel_; }
+
+ private:
+  // Panel currently being dragged.
+  Panel* dragging_panel_;

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

Seems like panel's original bounds and original strip should also be tracked
here.

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

Yes, it will be added when we handle inter-strip drags.

+
+  DISALLOW_COPY_AND_ASSIGN(PanelDragController);
+};
+
+#endif  // CHROME_BROWSER_UI_PANELS_PANEL_DRAG_CONTROLLER_H_