Support detaching/attaching panels via inter-strip drags.

chrome/browser/ui/panels/docked_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_DOCKED_PANEL_STRIP_H_
 #define CHROME_BROWSER_UI_PANELS_DOCKED_PANEL_STRIP_H_
 #pragma once
 
 #include <list>
 #include <set>
@@ skipping 35 matching lines @@
   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 CanShowPanelAsActive(const Panel* panel) const OVERRIDE;
+  virtual void SavePanelPlacement(Panel* panel) OVERRIDE;
+  virtual void LoadSavedPanelPlacement() OVERRIDE;
+  virtual void DiscardSavedPanelPlacement() OVERRIDE;
   virtual bool CanDragPanel(const Panel* panel) const OVERRIDE;
-  virtual void StartDraggingPanel(Panel* panel) OVERRIDE;
-  virtual void DragPanel(Panel* panel, int delta_x, int delta_y) OVERRIDE;
-  virtual void EndDraggingPanel(Panel* panel, bool cancelled) OVERRIDE;
+  virtual void StartDraggingPanelLocally(Panel* panel) OVERRIDE;
+  virtual void DragPanelLocally(Panel* panel,
+                                int delta_x,
+                                int delta_y) OVERRIDE;
+  virtual void EndDraggingPanelLocally(Panel* panel, bool cancelled) OVERRIDE;
+  virtual void AddDraggingPanel(Panel* panel,
+                                const gfx::Point& position) OVERRIDE;
 
   // Invoked when a panel's expansion state changes.
   void OnPanelExpansionStateChanged(Panel* panel);
 
   // Returns true if we should bring up the titlebars, given the current mouse
   // point.
   bool ShouldBringUpTitlebars(int mouse_x, int mouse_y) const;
 
   // Brings up or down the titlebars for all minimized panels.
   void BringUpOrDownTitlebars(bool bring_up);
@@ skipping 27 matching lines @@
   }
 #endif
 
  private:
   enum TitlebarAction {
     NO_ACTION,
     BRING_UP,
     BRING_DOWN
   };
 
+  struct PanelPlacement {
+    Panel* panel;
+    // When |panel| is restored, it will be bring back before

[jennb, 2012/03/03 02:19:33]

The wording here is confusing to me. How about:

Remember the panel to the left, if any, for use when restoring the position of
the panel being dragged. Will be updated if the panel to the left is closed.

[jianli, 2012/03/07 19:14:31]

Done.

+    // |panel_to_place_before_|.
+    Panel* panel_to_place_before_;
+
+    PanelPlacement() : panel(NULL) { }
+  };
+
   // Overridden from PanelMouseWatcherObserver:
   virtual void OnMouseMove(const gfx::Point& mouse_position) OVERRIDE;
 
+  // Inserts |panel| to the collection at |position|.
+  // Returns the iterator that points to the newly inserted panel.
+  void InsertPanelToCollection(Panel* panel, Panels::iterator position);

[jennb, 2012/03/03 02:19:33]

Doesn't return anything. This sounds like a wrapper for list::insert?

[jianli, 2012/03/07 19:14:31]

Removed. We now don't need a wrapper. Instead, I introduced OnPanelAdded in
order to set common panel properties.

+
+  // Ensures that there is sufficient space in the strip to add a panel with
+  // the specified width. We might need to bump panels in the strip to make
+  // room for this panel.
+  void EnsureAvailableSpace(int width);
+
   // Keep track of the minimized panels to control mouse watching.
   void IncrementMinimizedPanels();
   void DecrementMinimizedPanels();
 
   // Help functions to drag the given panel.
   void DragLeft(Panel* dragging_panel);
   void DragRight(Panel* dragging_panel);
 
   // Does the real job of bringing up or down the titlebars.
   void DoBringUpOrDownTitlebars(bool bring_up);
@@ skipping 27 matching lines @@
   bool are_titlebars_up_;
 
   // |True| to temporarily prevent refreshing panel layout, e.g. while
   // moving panels to overflow area to make room for a panel in this strip.
   bool disable_layout_refresh_;
 
   // Referring to current position in |panels_| where the dragging panel
   // resides.
   Panels::iterator dragging_panel_current_iterator_;
 
-  // Referring to original position in |panels_| where the dragging panel
-  // resides.
-  Panels::iterator dragging_panel_original_iterator_;
-
   // Delayed transitions support. Sometimes transitions between minimized and
   // title-only states are delayed, for better usability with Taskbars/Docks.
   TitlebarAction delayed_titlebar_action_;
 
   // Owned by MessageLoop after posting.
   base::WeakPtrFactory<DockedPanelStrip> titlebar_action_factory_;
 
+  // Used to save the placement information for a panel.
+  PanelPlacement saved_panel_placement_;
+
   static const int kPanelsHorizontalSpacing = 4;
 
   // Absolute minimum width and height for panels, including non-client area.
   // Should only be big enough to accomodate a close button on the reasonably
   // recognisable titlebar.
   static const int kPanelMinWidth;
   static const int kPanelMinHeight;
 
   DISALLOW_COPY_AND_ASSIGN(DockedPanelStrip);
 };
 
 #endif  // CHROME_BROWSER_UI_PANELS_DOCKED_PANEL_STRIP_H_