Refactor intra-strip panel drags by introducing PanelDragController.

chrome/browser/ui/panels/detached_panel_strip.h

 // 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_DETACHED_PANEL_STRIP_H_
 #define CHROME_BROWSER_UI_PANELS_DETACHED_PANEL_STRIP_H_
 #pragma once
 
 #include <set>
 #include "base/basictypes.h"
 #include "chrome/browser/ui/panels/panel.h"
 #include "chrome/browser/ui/panels/panel_strip.h"
+#include "ui/gfx/point.h"
 #include "ui/gfx/rect.h"
 
 class Browser;
 class PanelManager;
 
 // This class manages a group of free-floating panels.
 class DetachedPanelStrip : public PanelStrip {
  public:
   typedef std::set<Panel*> Panels;
 
   explicit DetachedPanelStrip(PanelManager* panel_manager);
   virtual ~DetachedPanelStrip();
 
   // PanelStrip OVERRIDES:
   virtual void SetDisplayArea(const gfx::Rect& display_area) OVERRIDE;
   virtual void RefreshLayout() OVERRIDE;
   virtual void AddPanel(Panel* panel) OVERRIDE;
   virtual bool RemovePanel(Panel* panel) OVERRIDE;
   virtual void CloseAll() OVERRIDE;
   virtual void ResizePanelWindow(
       Panel* panel,
       const gfx::Size& preferred_window_size) OVERRIDE;
   virtual void OnPanelAttentionStateChanged(Panel* panel) OVERRIDE;
   virtual void ActivatePanel(Panel* panel) OVERRIDE;
   virtual void MinimizePanel(Panel* panel) OVERRIDE;
   virtual void RestorePanel(Panel* panel) OVERRIDE;
+  virtual bool CanDragPanel(Panel* panel) const OVERRIDE;
+  virtual void StartDraggingPanel(Panel* panel) OVERRIDE;
+  virtual void DragPanel(int delta_x, int delta_y) OVERRIDE;
+  virtual void EndDraggingPanel(bool cancelled) OVERRIDE;
 
   int num_panels() const { return panels_.size(); }
   const Panels& panels() const { return panels_; }
 
  private:
   PanelManager* panel_manager_;  // Weak, owns us.
 
   // All panels in the panel strip must fit within this area.
   gfx::Rect display_area_;
 
   // Collection of all panels.
   Panels panels_;
 
+  // Original position of the panel being dragged. This is used to get back to
+  // the original position when we cancel the dragging.
+  gfx::Point dragging_panel_original_position_;

[jennb, 2012/02/17 21:28:45]

This feels like it belongs in drag controller. 

Also, which coordinate system?

[jianli, 2012/02/17 23:52:56]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(DetachedPanelStrip);
 };
 
 #endif  // CHROME_BROWSER_UI_PANELS_DETACHED_PANEL_STRIP_H_