Refactor intra-strip panel drags by introducing PanelDragController.

chrome/browser/ui/panels/detached_panel_strip.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/detached_panel_strip.h"
 
 #include <algorithm>
 #include "base/logging.h"
+#include "chrome/browser/ui/panels/panel_drag_controller.h"
 #include "chrome/browser/ui/panels/panel_manager.h"
 
 DetachedPanelStrip::DetachedPanelStrip(PanelManager* panel_manager)
     : PanelStrip(PanelStrip::DETACHED),
       panel_manager_(panel_manager) {
 }
 
 DetachedPanelStrip::~DetachedPanelStrip() {
   DCHECK(panels_.empty());
 }
@@ skipping 53 matching lines @@
 void DetachedPanelStrip::MinimizePanel(Panel* panel) {
   DCHECK_EQ(this, panel->panel_strip());
   NOTIMPLEMENTED();
 }
 
 void DetachedPanelStrip::RestorePanel(Panel* panel) {
   DCHECK_EQ(this, panel->panel_strip());
   NOTIMPLEMENTED();
 }
 
+bool DetachedPanelStrip::CanDragPanel(Panel* panel) const {
+  // All detached panels are draggable.
+  return true;
+}
+
+void DetachedPanelStrip::StartDraggingPanel(Panel* panel) {
+  dragging_panel_original_position_ = panel->GetBounds().origin();
+}
+
+void DetachedPanelStrip::DragPanel(int delta_x, int delta_y) {

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

Maybe we should pass the dragging_panel into this method and EndDraggingPanel?

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

Done.

+  Panel* dragging_panel = panel_manager_->drag_controller()->dragging_panel();
+  DCHECK(dragging_panel);
+
+  gfx::Rect new_bounds(dragging_panel->GetBounds());
+  new_bounds.Offset(delta_x, delta_y);

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

Do we need to adjust the deltas to keep the panel on screen? Mouse position
could be on screen, but actual panel might be partially off screen.

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

The detached panel should be allowed at any position, even some of its area is
off-screen.

+  dragging_panel->SetPanelBounds(new_bounds);
+}
+
+void DetachedPanelStrip::EndDraggingPanel(bool cancelled) {
+  Panel* dragging_panel = panel_manager_->drag_controller()->dragging_panel();
+  DCHECK(dragging_panel);
+
+  if (cancelled) {
+    gfx::Rect new_bounds(dragging_panel->GetBounds());
+    new_bounds.set_origin(dragging_panel_original_position_);
+    dragging_panel->SetPanelBounds(new_bounds);
+  }
+}