Support detaching/attaching panels via inter-strip drags.

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"
@@ skipping 17 matching lines @@
 
   RefreshLayout();
 }
 
 void DetachedPanelStrip::RefreshLayout() {
   NOTIMPLEMENTED();
 }
 
 void DetachedPanelStrip::AddPanel(Panel* panel) {
   DCHECK_EQ(this, panel->panel_strip());
+
+  panel->SetAlwaysOnTop(false);
   panels_.insert(panel);
 }
 
 bool DetachedPanelStrip::RemovePanel(Panel* panel) {
   return panels_.erase(panel) != 0;
 }
 
 void DetachedPanelStrip::CloseAll() {
   // Make a copy as closing panels can modify the iterator.
   Panels panels_copy = panels_;
@@ skipping 26 matching lines @@
 void DetachedPanelStrip::RestorePanel(Panel* panel) {
   DCHECK_EQ(this, panel->panel_strip());
   NOTIMPLEMENTED();
 }
 
 bool DetachedPanelStrip::CanShowPanelAsActive(const Panel* panel) const {
   // All detached panels can be shown as active.
   return true;
 }
 
+void DetachedPanelStrip::SavePanelPlacement(Panel* panel) {
+  DCHECK(!saved_panel_placement_.panel);
+  saved_panel_placement_.panel = panel;
+  saved_panel_placement_.position = panel->GetBounds().origin();
+}
+
+void DetachedPanelStrip::LoadSavedPanelPlacement() {
+  DCHECK(saved_panel_placement_.panel);
+
+  gfx::Rect new_bounds(saved_panel_placement_.panel->GetBounds());
+  new_bounds.set_origin(saved_panel_placement_.position);
+  saved_panel_placement_.panel->SetPanelBounds(new_bounds);
+
+  saved_panel_placement_.panel = NULL;

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

Should this call DiscardSavedPanelPlacement() instead? Ensures the same cleanup
logic applies in both places. Ditto for docked strip.

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

Done.

+}
+
+void DetachedPanelStrip::DiscardSavedPanelPlacement() {
+  DCHECK(saved_panel_placement_.panel);
+  saved_panel_placement_.panel = NULL;
+}
+
 bool DetachedPanelStrip::CanDragPanel(const Panel* panel) const {
   // All detached panels are draggable.
   return true;
 }
 
-void DetachedPanelStrip::StartDraggingPanel(Panel* panel) {
+void DetachedPanelStrip::StartDraggingPanelLocally(Panel* panel) {
+  // Move the panel to this strip if needed.
+  if (!HasPanel(panel)) {

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

Reminder that you meant to revert this change.

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

Done.

+    // TODO(jianli): Remove this when set_panel_strip is part of AddPanel.
+    panel->set_panel_strip(this);
+    AddPanel(panel);
+  }
 }
 
-void DetachedPanelStrip::DragPanel(Panel* panel, int delta_x, int delta_y) {
+void DetachedPanelStrip::DragPanelLocally(Panel* panel,
+                                          int delta_x,
+                                          int delta_y) {
   gfx::Rect new_bounds(panel->GetBounds());
   new_bounds.Offset(delta_x, delta_y);
   panel->SetPanelBounds(new_bounds);
 }
 
-void DetachedPanelStrip::EndDraggingPanel(Panel* panel, bool cancelled) {
-  if (cancelled) {
-    gfx::Rect new_bounds(panel->GetBounds());
-    new_bounds.set_origin(
-        panel_manager_->drag_controller()->dragging_panel_original_position());
-    panel->SetPanelBounds(new_bounds);
-  }
+void DetachedPanelStrip::EndDraggingPanelLocally(Panel* panel, bool cancelled) {
+}
+
+void DetachedPanelStrip::AddDraggingPanel(Panel* panel,

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

As discussed, separate out moving panels between strips from starting drag.

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

Done.

+                                          const gfx::Point& position) {
+  // TODO(jianli): remove this when set_panel_strip is part of AddPanel.
+  panel->set_panel_strip(this);
+  AddPanel(panel);
+
+  gfx::Rect new_bounds(panel->GetBounds());
+  new_bounds.set_origin(position);
+  panel->SetPanelBounds(new_bounds);
 }
 
 bool DetachedPanelStrip::HasPanel(Panel* panel) const {
   return panels_.find(panel) != panels_.end();
 }