Support detaching/attaching panels via inter-strip drags.

chrome/browser/ui/panels/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_PANEL_STRIP_H_
 #define CHROME_BROWSER_UI_PANELS_PANEL_STRIP_H_
 #pragma once
 
+#include "ui/gfx/point.h"
 #include "ui/gfx/rect.h"
 
 class Panel;
 
 // Common base class for a collection of panels. Subclasses manage
 // various layouts for displaying panels in the collection.
 class PanelStrip {
  public:
   // Types of layout for the panel collections.
   enum Type {
@@ skipping 40 matching lines @@
 
   // Updates the display to show |panel| as  minimized/restored.
   virtual void MinimizePanel(Panel* panel) = 0;
   virtual void RestorePanel(Panel* panel) = 0;
 
   virtual bool IsPanelMinimized(const Panel* panel) const = 0;
 
   // Returns true if |panel| can be shown as active.
   virtual bool CanShowPanelAsActive(const Panel* panel) const = 0;
 
+  // Saves/restores/discards the placement information of |panel|. This is
+  // useful in bringing back the dragging panel to its original positioning
+  // when the drag is cancelled. After the placement information is saved,
+  // the caller should only call one of RestorePanelToSavedPlacement and

[jennb, 2012/03/08 23:41:09]

s/and/or

[jianli, 2012/03/09 21:48:58]

Done.

+  // DiscardSavedPanelPlacement.
+  virtual void SavePanelPlacement(Panel* panel) = 0;
+  virtual void RestorePanelToSavedPlacement() = 0;
+  virtual void DiscardSavedPanelPlacement() = 0;
+
   // Returns true if |panel| is draggable.
   virtual bool CanDragPanel(const Panel* panel) const = 0;
 
-  // Drags |panel| in the bounds of this strip.
-  virtual void StartDraggingPanel(Panel* panel) = 0;
-  // |delta_x| and |delta_y| denotes how much the mouse has been moved since
-  // last time when DragPanel or StartDraggingPanel is called.
-  virtual void DragPanel(Panel* panel, int delta_x, int delta_y) = 0;
-  virtual void EndDraggingPanel(Panel* panel, bool cancelled) = 0;
+  // Starts dragging |panel| within this strip. The panel should already be
+  // in this strip.
+  virtual void StartDraggingPanelWithinStrip(Panel* panel) = 0;
+
+  // Drags |panel| within this strip.
+  // |delta_x| and |delta_y| represents the offset from the last mouse location
+  // when StartDraggingPanelWithinStrip or DragPanelWithinStrip is called.
+  virtual void DragPanelWithinStrip(Panel* panel, int delta_x, int delta_y) = 0;

[jennb, 2012/03/08 23:41:09]

why deltas instead of mouse position?

[jianli, 2012/03/09 21:48:58]

If mouse location, the strip has to keep track of last mouse location to compute
the delta.

+
+  // Ends dragging |panel| within this strip. |aborted| means the drag within
+  // this strip is aborted due to one of the following:
+  // 1) the drag leaves this strip and enters other strip
+  // 2) the drag gets cancelled
+  // If |aborted| is true, |panel| will stay as it is; otherwise, it will be

[jennb, 2012/03/08 23:41:09]

Perhaps the flag should be finalize_position instead?

[jianli, 2012/03/09 21:48:58]

I would prefer to use |abort| because for now the panel is moved to its
finalized position when |abort| is false. But in the future, we might add some
more things to end the drag.

+  // moved to its finalized position.
+  // The drag controller is responsible for restoring the panel back to its
+  // original strip and position when the drag gets cancelled.
+  virtual void EndDraggingPanelWithinStrip(Panel* panel, bool aborted) = 0;
 
  protected:
   explicit PanelStrip(Type type);
   virtual ~PanelStrip();
 
   const Type type_;  // Type of this panel strip.
 };
 
 #endif  // CHROME_BROWSER_UI_PANELS_PANEL_STRIP_H_