Support detaching/attaching panels via inter-strip drags.

chrome/browser/ui/panels/panel_drag_browsertest.cc

Index: chrome/browser/ui/panels/panel_drag_browsertest.cc
diff --git a/chrome/browser/ui/panels/panel_drag_browsertest.cc b/chrome/browser/ui/panels/panel_drag_browsertest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..b0637b5f9af25d30bf9475e20378f8c540f468e4
--- /dev/null
+++ b/chrome/browser/ui/panels/panel_drag_browsertest.cc
@@ -0,0 +1,543 @@
+// 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/base_panel_browser_test.h"
+#include "chrome/browser/ui/panels/detached_panel_strip.h"
+#include "chrome/browser/ui/panels/docked_panel_strip.h"
+#include "chrome/browser/ui/panels/overflow_panel_strip.h"
+#include "chrome/browser/ui/panels/native_panel.h"
+#include "chrome/browser/ui/panels/panel.h"
+#include "chrome/browser/ui/panels/panel_drag_controller.h"
+#include "chrome/browser/ui/panels/panel_manager.h"
+
+class PanelDragBrowserTest : public BasePanelBrowserTest {

[jennb, 2012/03/01 00:33:38]

Is this file supposed to contain all panel dragging tests? We currently have
dragging tests in 4 different files, plus this one makes 5. Please consolidate.

[jianli, 2012/03/02 22:42:43]

Yes. I will move other dragging related tests to this file after this patch is
landed.

+ public:
+  PanelDragBrowserTest() : BasePanelBrowserTest() {
+  }
+
+  virtual ~PanelDragBrowserTest() {
+  }
+
+  virtual void SetUpOnMainThread() OVERRIDE {
+    BasePanelBrowserTest::SetUpOnMainThread();
+
+    // All the tests here assume 800x600 work area. Do the check now.
+    DCHECK(PanelManager::GetInstance()->work_area().width() == 800);
+    DCHECK(PanelManager::GetInstance()->work_area().height() == 600);
+  }
+
+  Panel* CreateDockedPanel(const std::string& name, const gfx::Rect& bounds) {
+    Panel* panel = CreatePanelWithBounds(name, bounds);
+    EXPECT_EQ(PanelStrip::DOCKED, panel->panel_strip()->type());
+    return panel;
+  }
+
+  Panel* CreateDetachedPanel(const std::string& name, const gfx::Rect& bounds) {
+    Panel* panel = CreatePanelWithBounds(name, bounds);
+    panel->MoveToStrip(panel->manager()->detached_strip());
+    panel->SetPanelBounds(bounds);
+    EXPECT_EQ(PanelStrip::DETACHED, panel->panel_strip()->type());
+    return panel;
+  }
+
+  Panel* CreateOverflowPanel(const std::string& name, const gfx::Rect& bounds) {
+    CreatePanelParams params(name, bounds, SHOW_AS_INACTIVE);
+    Panel* panel = CreatePanelWithParams(params);
+    WaitForLayoutModeChanged(panel, PanelStrip::IN_OVERFLOW);
+    EXPECT_EQ(PanelStrip::IN_OVERFLOW, panel->panel_strip()->type());

[jennb, 2012/03/01 00:33:38]

Redundant as WaitForLayoutModeChanged already checks this.

[jianli, 2012/03/02 22:42:43]

Just want to be safe because only after you read WaitForLayoutModeChanged code
you will know this check is redundant.

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

Should be removed as it deviates from the pattern in other tests where we do the
WaitForX without a check X after line.

+    return panel;
+  }
+
+  void DragPanelAndFinish(Panel* panel, const gfx::Point& delta) {
+    scoped_ptr<NativePanelTesting> panel_testing(
+        NativePanelTesting::Create(panel->native_panel()));
+    gfx::Point mouse_location(panel->GetBounds().origin());
+    panel_testing->PressLeftMouseButtonTitlebar(mouse_location);
+    panel_testing->DragTitlebar(mouse_location.Add(delta));
+    panel_testing->FinishDragTitlebar();
+  }
+
+  void DragPanelAndCancel(Panel* panel, const gfx::Point& delta) {
+    scoped_ptr<NativePanelTesting> panel_testing(
+        NativePanelTesting::Create(panel->native_panel()));
+    gfx::Point mouse_location(panel->GetBounds().origin());
+    panel_testing->PressLeftMouseButtonTitlebar(mouse_location);
+    panel_testing->DragTitlebar(mouse_location.Add(delta));
+    panel_testing->CancelDragTitlebar();
+  }
+};
+
+IN_PROC_BROWSER_TEST_F(PanelDragBrowserTest, Detach) {
+  PanelManager* panel_manager = PanelManager::GetInstance();
+  DockedPanelStrip* docked_strip = panel_manager->docked_strip();
+  DetachedPanelStrip* detached_strip = panel_manager->detached_strip();
+
+  gfx::Point small_drag_delta(-5, -10);

[jennb, 2012/03/01 00:33:38]

Could you write these tests in such a way that they will not break if the offset
amount to trigger a detach is changed?

[jianli, 2012/03/02 22:42:43]

Done.

+  gfx::Point drag_delta_to_detach(-5, -150);
+
+  // Create one docked panel.
+  Panel* panel = CreateDockedPanel("1", gfx::Rect(0, 0, 100, 100));
+  ASSERT_EQ(1, docked_strip->num_panels());
+  ASSERT_EQ(0, detached_strip->num_panels());
+
+  gfx::Rect panel_old_bounds = panel->GetBounds();
+
+  // Press on title-bar.
+  scoped_ptr<NativePanelTesting> panel_testing(
+      NativePanelTesting::Create(panel->native_panel()));
+  gfx::Point mouse_location(panel->GetBounds().origin());
+  panel_testing->PressLeftMouseButtonTitlebar(mouse_location);
+
+  // Drag up the panel in a small offset that does not trigger the detach.
+  // Expect that the panel is still docked and only x coordinate of its position
+  // is changed.
+  mouse_location = mouse_location.Add(small_drag_delta);
+  panel_testing->DragTitlebar(mouse_location);
+  ASSERT_EQ(1, docked_strip->num_panels());
+  ASSERT_EQ(0, detached_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DOCKED, panel->panel_strip()->type());
+  gfx::Rect panel_new_bounds = panel_old_bounds;
+  panel_new_bounds.Offset(small_drag_delta.x(), 0);
+  EXPECT_EQ(panel_new_bounds, panel->GetBounds());
+
+  // Continue dragging up the panel in big offset that triggers the detach.
+  // Expect that the panel is previewed as detached.
+  mouse_location = mouse_location.Add(drag_delta_to_detach);
+  panel_testing->DragTitlebar(mouse_location);
+  ASSERT_EQ(0, docked_strip->num_panels());
+  ASSERT_EQ(1, detached_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DETACHED, panel->panel_strip()->type());
+  panel_new_bounds.Offset(drag_delta_to_detach.x(),
+                          drag_delta_to_detach.y() + small_drag_delta.y());
+  EXPECT_EQ(panel_new_bounds, panel->GetBounds());
+
+  // Finish the drag.
+  // Expect that the panel stays as detached.
+  panel_testing->FinishDragTitlebar();
+  ASSERT_EQ(0, docked_strip->num_panels());
+  ASSERT_EQ(1, detached_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DETACHED, panel->panel_strip()->type());
+  EXPECT_EQ(panel_new_bounds, panel->GetBounds());
+
+  panel_manager->CloseAll();
+}
+
+IN_PROC_BROWSER_TEST_F(PanelDragBrowserTest, DetachAndCancel) {
+  PanelManager* panel_manager = PanelManager::GetInstance();
+  DockedPanelStrip* docked_strip = panel_manager->docked_strip();
+  DetachedPanelStrip* detached_strip = panel_manager->detached_strip();
+
+  gfx::Point small_drag_delta(-5, -10);
+  gfx::Point drag_delta_to_detach(-5, -150);
+
+  // Create one docked panel.
+  Panel* panel = CreateDockedPanel("1", gfx::Rect(0, 0, 100, 100));
+  ASSERT_EQ(1, docked_strip->num_panels());
+  ASSERT_EQ(0, detached_strip->num_panels());
+
+  gfx::Rect panel_old_bounds = panel->GetBounds();
+
+  // Press on title-bar.
+  scoped_ptr<NativePanelTesting> panel_testing(
+      NativePanelTesting::Create(panel->native_panel()));
+  gfx::Point mouse_location(panel->GetBounds().origin());
+  panel_testing->PressLeftMouseButtonTitlebar(mouse_location);
+
+  // Drag up the panel in a small offset that does not trigger the detach.
+  // Expect that the panel is still docked and only x coordinate of its position
+  // is changed.
+  mouse_location = mouse_location.Add(small_drag_delta);
+  panel_testing->DragTitlebar(mouse_location);
+  ASSERT_EQ(1, docked_strip->num_panels());
+  ASSERT_EQ(0, detached_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DOCKED, panel->panel_strip()->type());
+  gfx::Rect panel_new_bounds = panel_old_bounds;
+  panel_new_bounds.Offset(small_drag_delta.x(), 0);
+  EXPECT_EQ(panel_new_bounds, panel->GetBounds());
+
+  // Continue dragging up the panel in big offset that triggers the detach.
+  // Expect that the panel is previewed as detached.
+  mouse_location = mouse_location.Add(drag_delta_to_detach);
+  panel_testing->DragTitlebar(mouse_location);
+  ASSERT_EQ(0, docked_strip->num_panels());
+  ASSERT_EQ(1, detached_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DETACHED, panel->panel_strip()->type());
+  panel_new_bounds.Offset(drag_delta_to_detach.x(),
+                          drag_delta_to_detach.y() + small_drag_delta.y());
+  EXPECT_EQ(panel_new_bounds, panel->GetBounds());
+
+  // Cancel the drag.
+  // Expect that the panel is back as docked.
+  panel_testing->CancelDragTitlebar();
+  ASSERT_EQ(1, docked_strip->num_panels());
+  ASSERT_EQ(0, detached_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DOCKED, panel->panel_strip()->type());
+  EXPECT_EQ(panel_old_bounds, panel->GetBounds());
+
+  panel_manager->CloseAll();
+}
+
+IN_PROC_BROWSER_TEST_F(PanelDragBrowserTest, Attach) {
+  PanelManager* panel_manager = PanelManager::GetInstance();
+  DockedPanelStrip* docked_strip = panel_manager->docked_strip();
+  DetachedPanelStrip* detached_strip = panel_manager->detached_strip();
+
+  gfx::Point drag_delta_to_move_down(-5, 150);
+  gfx::Point drag_delta_to_attach(-30, 45);
+
+  // Create one detached panel.
+  Panel* panel = CreateDetachedPanel("1", gfx::Rect(400, 300, 100, 100));
+  ASSERT_EQ(0, docked_strip->num_panels());
+  ASSERT_EQ(1, detached_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DETACHED, panel->panel_strip()->type());
+
+  gfx::Rect panel_old_bounds = panel->GetBounds();
+
+  // Press on title-bar.
+  scoped_ptr<NativePanelTesting> panel_testing(
+      NativePanelTesting::Create(panel->native_panel()));
+  gfx::Point mouse_location(panel->GetBounds().origin());
+  panel_testing->PressLeftMouseButtonTitlebar(mouse_location);
+
+  // Drag down the panel but not close enough to the bottom of work area.
+  // Expect that the panel is still detached.
+  mouse_location = mouse_location.Add(drag_delta_to_move_down);
+  panel_testing->DragTitlebar(mouse_location);
+  ASSERT_EQ(0, docked_strip->num_panels());
+  ASSERT_EQ(1, detached_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DETACHED, panel->panel_strip()->type());
+  gfx::Rect panel_new_bounds = panel_old_bounds;
+  panel_new_bounds.Offset(drag_delta_to_move_down);
+  EXPECT_EQ(panel_new_bounds, panel->GetBounds());
+
+  // Continue dragging down the panel to make it close enough to the bottom of
+  // work area.
+  // Expect that the panel is previewed as docked.
+  mouse_location = mouse_location.Add(drag_delta_to_attach);
+  panel_testing->DragTitlebar(mouse_location);
+  ASSERT_EQ(1, docked_strip->num_panels());
+  ASSERT_EQ(0, detached_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DOCKED, panel->panel_strip()->type());
+  panel_new_bounds.Offset(drag_delta_to_attach);
+  EXPECT_EQ(panel_new_bounds, panel->GetBounds());
+
+  // Finish the drag.
+  // Expect that the panel stays as docked and moves to the final position.
+  panel_testing->FinishDragTitlebar();
+  ASSERT_EQ(1, docked_strip->num_panels());
+  ASSERT_EQ(0, detached_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DOCKED, panel->panel_strip()->type());
+  panel_new_bounds.set_x(
+      docked_strip->StartingRightPosition() - panel_new_bounds.width());
+  panel_new_bounds.set_y(
+      docked_strip->display_area().bottom() - panel_new_bounds.height());
+  EXPECT_EQ(panel_new_bounds, panel->GetBounds());
+
+  panel_manager->CloseAll();
+}
+
+IN_PROC_BROWSER_TEST_F(PanelDragBrowserTest, AttachAndCancel) {
+  PanelManager* panel_manager = PanelManager::GetInstance();
+  DockedPanelStrip* docked_strip = panel_manager->docked_strip();
+  DetachedPanelStrip* detached_strip = panel_manager->detached_strip();
+
+  gfx::Point drag_delta_to_move_down(-5, 150);
+  gfx::Point drag_delta_to_attach(-30, 45);
+
+  // Create one detached panel.
+  Panel* panel = CreateDetachedPanel("1", gfx::Rect(400, 300, 100, 100));
+  ASSERT_EQ(0, docked_strip->num_panels());
+  ASSERT_EQ(1, detached_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DETACHED, panel->panel_strip()->type());
+
+  gfx::Rect panel_old_bounds = panel->GetBounds();
+
+  // Press on title-bar.
+  scoped_ptr<NativePanelTesting> panel_testing(
+      NativePanelTesting::Create(panel->native_panel()));
+  gfx::Point mouse_location(panel->GetBounds().origin());
+  panel_testing->PressLeftMouseButtonTitlebar(mouse_location);
+
+  // Drag down the panel but not close enough to the bottom of work area.
+  // Expect that the panel is still detached.
+  mouse_location = mouse_location.Add(drag_delta_to_move_down);
+  panel_testing->DragTitlebar(mouse_location);
+  ASSERT_EQ(0, docked_strip->num_panels());
+  ASSERT_EQ(1, detached_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DETACHED, panel->panel_strip()->type());
+  gfx::Rect panel_new_bounds = panel_old_bounds;
+  panel_new_bounds.Offset(drag_delta_to_move_down);
+  EXPECT_EQ(panel_new_bounds, panel->GetBounds());
+
+  // Continue dragging down the panel to make it close enough to the bottom of
+  // work area.
+  // Expect that the panel is previewed as docked.
+  mouse_location = mouse_location.Add(drag_delta_to_attach);
+  panel_testing->DragTitlebar(mouse_location);
+  ASSERT_EQ(1, docked_strip->num_panels());
+  ASSERT_EQ(0, detached_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DOCKED, panel->panel_strip()->type());
+  panel_new_bounds.Offset(drag_delta_to_attach);
+  EXPECT_EQ(panel_new_bounds, panel->GetBounds());
+
+  // Cancel the drag.
+  // Expect that the panel is back as detached.
+  panel_testing->CancelDragTitlebar();
+  ASSERT_EQ(0, docked_strip->num_panels());
+  ASSERT_EQ(1, detached_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DETACHED, panel->panel_strip()->type());
+  EXPECT_EQ(panel_old_bounds, panel->GetBounds());
+
+  panel_manager->CloseAll();
+}
+
+IN_PROC_BROWSER_TEST_F(PanelDragBrowserTest, DetachAttachAndCancel) {
+  PanelManager* panel_manager = PanelManager::GetInstance();
+  DockedPanelStrip* docked_strip = panel_manager->docked_strip();
+  DetachedPanelStrip* detached_strip = panel_manager->detached_strip();
+
+  gfx::Point drag_delta_to_detach(-35, -150);
+  gfx::Point drag_delta_to_reattach(-25, 145);
+  gfx::Point drag_delta_to_detach_again(30, -200);
+
+  // Create one docked panel.
+  Panel* panel = CreateDockedPanel("1", gfx::Rect(0, 0, 100, 100));
+  ASSERT_EQ(1, docked_strip->num_panels());
+  ASSERT_EQ(0, detached_strip->num_panels());
+
+  gfx::Rect panel_old_bounds = panel->GetBounds();
+
+  // Press on title-bar.
+  scoped_ptr<NativePanelTesting> panel_testing(
+      NativePanelTesting::Create(panel->native_panel()));
+  gfx::Point mouse_location(panel->GetBounds().origin());
+  panel_testing->PressLeftMouseButtonTitlebar(mouse_location);
+
+  // Drag up the panel to trigger the detach.
+  // Expect that the panel is previewed as detached.
+  mouse_location = mouse_location.Add(drag_delta_to_detach);
+  panel_testing->DragTitlebar(mouse_location);
+  ASSERT_EQ(0, docked_strip->num_panels());
+  ASSERT_EQ(1, detached_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DETACHED, panel->panel_strip()->type());
+  gfx::Rect panel_new_bounds = panel_old_bounds;
+  panel_new_bounds.Offset(drag_delta_to_detach);
+  EXPECT_EQ(panel_new_bounds, panel->GetBounds());
+
+  // Continue dragging down the panel to trigger the re-attach.
+  mouse_location = mouse_location.Add(drag_delta_to_reattach);
+  panel_testing->DragTitlebar(mouse_location);
+  ASSERT_EQ(1, docked_strip->num_panels());
+  ASSERT_EQ(0, detached_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DOCKED, panel->panel_strip()->type());
+  panel_new_bounds.Offset(drag_delta_to_reattach);
+  EXPECT_EQ(panel_new_bounds, panel->GetBounds());
+
+  // Continue dragging up the panel to trigger the detach again.
+  mouse_location = mouse_location.Add(drag_delta_to_detach_again);
+  panel_testing->DragTitlebar(mouse_location);
+  ASSERT_EQ(0, docked_strip->num_panels());
+  ASSERT_EQ(1, detached_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DETACHED, panel->panel_strip()->type());
+  panel_new_bounds.Offset(drag_delta_to_detach_again);
+  EXPECT_EQ(panel_new_bounds, panel->GetBounds());
+
+  // Cancel the drag.
+  // Expect that the panel stays as docked.
+  panel_testing->CancelDragTitlebar();
+  ASSERT_EQ(1, docked_strip->num_panels());
+  ASSERT_EQ(0, detached_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DOCKED, panel->panel_strip()->type());
+  EXPECT_EQ(panel_old_bounds, panel->GetBounds());
+
+  panel_manager->CloseAll();
+}
+
+IN_PROC_BROWSER_TEST_F(PanelDragBrowserTest, DetachInOverflow) {

[jennb, 2012/03/01 00:33:38]

s/InOverflow/WithOverflow ?

[jianli, 2012/03/02 22:42:43]

Done.

+  PanelManager* panel_manager = PanelManager::GetInstance();
+  DockedPanelStrip* docked_strip = panel_manager->docked_strip();
+  DetachedPanelStrip* detached_strip = panel_manager->detached_strip();
+  OverflowPanelStrip* overflow_strip = panel_manager->overflow_strip();
+
+  gfx::Point drag_delta_to_detach(-5, -150);
+
+  // Create some docked and overflow panels.
+  //   docked:    P3  P2  P1
+  //   overflow:  P5  P4
+  Panel* panel1 = CreateDockedPanel("1", gfx::Rect(0, 0, 200, 100));
+  Panel* panel2 = CreateDockedPanel("2", gfx::Rect(0, 0, 200, 100));
+  Panel* panel3 = CreateDockedPanel("3", gfx::Rect(0, 0, 200, 100));
+  Panel* panel4 = CreateOverflowPanel("4", gfx::Rect(0, 0, 200, 100));
+  Panel* panel5 = CreateOverflowPanel("5", gfx::Rect(0, 0, 200, 100));
+  ASSERT_EQ(0, detached_strip->num_panels());
+  ASSERT_EQ(3, docked_strip->num_panels());
+  ASSERT_EQ(2, overflow_strip->num_panels());
+
+  gfx::Point docked_position1 = panel1->GetBounds().origin();
+  gfx::Point docked_position2 = panel2->GetBounds().origin();
+  gfx::Point docked_position3 = panel3->GetBounds().origin();
+
+  // Drag to detach the middle docked panel.
+  // Expect to have:
+  //   detached:  P2
+  //   docked:    P4  P3  P1
+  //   overflow:  P5
+  DragPanelAndFinish(panel2, drag_delta_to_detach);
+  ASSERT_EQ(1, detached_strip->num_panels());
+  ASSERT_EQ(3, docked_strip->num_panels());
+  ASSERT_EQ(1, overflow_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DOCKED, panel1->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DETACHED, panel2->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DOCKED, panel3->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DOCKED, panel4->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::IN_OVERFLOW, panel5->panel_strip()->type());
+  EXPECT_EQ(docked_position1, panel1->GetBounds().origin());
+  gfx::Point panel2_new_position = docked_position2.Add(drag_delta_to_detach);
+  EXPECT_EQ(panel2_new_position, panel2->GetBounds().origin());
+  EXPECT_EQ(docked_position2, panel3->GetBounds().origin());
+  EXPECT_EQ(docked_position3, panel4->GetBounds().origin());
+
+  // Drag to detach the left-most docked panel.
+  // Expect to have:
+  //   detached:  P2  P4
+  //   docked:    P5  P3  P1
+  DragPanelAndFinish(panel4, drag_delta_to_detach);
+  ASSERT_EQ(2, detached_strip->num_panels());
+  ASSERT_EQ(3, docked_strip->num_panels());
+  ASSERT_EQ(0, overflow_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DOCKED, panel1->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DETACHED, panel2->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DOCKED, panel3->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DETACHED, panel4->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DOCKED, panel5->panel_strip()->type());
+  EXPECT_EQ(docked_position1, panel1->GetBounds().origin());
+  EXPECT_EQ(panel2_new_position, panel2->GetBounds().origin());
+  EXPECT_EQ(docked_position2, panel3->GetBounds().origin());
+  gfx::Point panel4_new_position = docked_position3.Add(drag_delta_to_detach);
+  EXPECT_EQ(panel4_new_position, panel4->GetBounds().origin());
+  EXPECT_EQ(docked_position3, panel5->GetBounds().origin());
+
+  // Drag to detach the right-most docked panel.
+  // Expect to have:
+  //   detached:  P1  P2  P4
+  //   docked:    P5  P3
+  DragPanelAndFinish(panel1, drag_delta_to_detach);
+  ASSERT_EQ(3, detached_strip->num_panels());
+  ASSERT_EQ(2, docked_strip->num_panels());
+  ASSERT_EQ(0, overflow_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DETACHED, panel1->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DETACHED, panel2->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DOCKED, panel3->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DETACHED, panel4->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DOCKED, panel5->panel_strip()->type());
+  gfx::Point panel1_new_position = docked_position1.Add(drag_delta_to_detach);
+  EXPECT_EQ(panel1_new_position, panel1->GetBounds().origin());
+  EXPECT_EQ(panel2_new_position, panel2->GetBounds().origin());
+  EXPECT_EQ(docked_position1, panel3->GetBounds().origin());
+  EXPECT_EQ(panel4_new_position, panel4->GetBounds().origin());
+  EXPECT_EQ(docked_position2, panel5->GetBounds().origin());
+
+  panel_manager->CloseAll();
+}
+
+IN_PROC_BROWSER_TEST_F(PanelDragBrowserTest, AttachInOverflow) {
+  PanelManager* panel_manager = PanelManager::GetInstance();
+  DockedPanelStrip* docked_strip = panel_manager->docked_strip();
+  DetachedPanelStrip* detached_strip = panel_manager->detached_strip();
+  OverflowPanelStrip* overflow_strip = panel_manager->overflow_strip();
+
+  gfx::Point drag_delta_to_detach(-5, -150);
+
+  // Create some detached, docked and overflow panels.
+  //   detached:  P1  P2  P3
+  //   docked:    P6  P5  P4
+  //   overflow:  P7
+  Panel* panel1 = CreateDetachedPanel("1", gfx::Rect(100, 300, 200, 100));
+  Panel* panel2 = CreateDetachedPanel("2", gfx::Rect(200, 300, 200, 100));
+  Panel* panel3 = CreateDetachedPanel("3", gfx::Rect(400, 300, 200, 100));
+  Panel* panel4 = CreateDockedPanel("4", gfx::Rect(0, 0, 200, 100));
+  Panel* panel5 = CreateDockedPanel("5", gfx::Rect(0, 0, 200, 100));
+  Panel* panel6 = CreateDockedPanel("6", gfx::Rect(0, 0, 200, 100));
+  Panel* panel7 = CreateOverflowPanel("7", gfx::Rect(0, 0, 200, 100));
+  ASSERT_EQ(3, detached_strip->num_panels());
+  ASSERT_EQ(3, docked_strip->num_panels());
+  ASSERT_EQ(1, overflow_strip->num_panels());
+
+  gfx::Point detached_position1 = panel1->GetBounds().origin();
+  gfx::Point detached_position2 = panel2->GetBounds().origin();
+  gfx::Point detached_position3 = panel3->GetBounds().origin();
+  gfx::Point docked_position1 = panel4->GetBounds().origin();
+  gfx::Point docked_position2 = panel5->GetBounds().origin();
+  gfx::Point docked_position3 = panel6->GetBounds().origin();
+
+  // Drag to attach a detached panel between 2 docked panels.
+  // Expect to have:
+  //   detached:  P1  P2
+  //   docked:    P5  P3  P4
+  //   overflow:  P7  P6
+  gfx::Point drag_delta_to_attach(
+      docked_position2.x() - detached_position3.x() + 10, 210);
+  DragPanelAndFinish(panel3, drag_delta_to_attach);
+  ASSERT_EQ(2, detached_strip->num_panels());
+  ASSERT_EQ(3, docked_strip->num_panels());
+  ASSERT_EQ(2, overflow_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DETACHED, panel1->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DETACHED, panel2->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DOCKED, panel3->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DOCKED, panel4->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DOCKED, panel5->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::IN_OVERFLOW, panel6->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::IN_OVERFLOW, panel7->panel_strip()->type());
+  EXPECT_EQ(detached_position1, panel1->GetBounds().origin());
+  EXPECT_EQ(detached_position2, panel2->GetBounds().origin());
+  EXPECT_EQ(docked_position2, panel3->GetBounds().origin());
+  EXPECT_EQ(docked_position1, panel4->GetBounds().origin());
+  EXPECT_EQ(docked_position3, panel5->GetBounds().origin());
+
+  // Drag to attach a detached panel to most-right.
+  // Expect to have:
+  //   detached:  P1
+  //   docked:    P3  P4  P2
+  //   overflow:  P7  P6  P5
+  gfx::Point drag_delta_to_attach2(
+      docked_position1.x() - detached_position2.x() + 10, 210);
+  DragPanelAndFinish(panel2, drag_delta_to_attach2);
+  ASSERT_EQ(1, detached_strip->num_panels());
+  ASSERT_EQ(3, docked_strip->num_panels());
+  ASSERT_EQ(3, overflow_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DETACHED, panel1->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DOCKED, panel2->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DOCKED, panel3->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DOCKED, panel4->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::IN_OVERFLOW, panel5->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::IN_OVERFLOW, panel6->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::IN_OVERFLOW, panel7->panel_strip()->type());
+  EXPECT_EQ(detached_position1, panel1->GetBounds().origin());
+  EXPECT_EQ(docked_position1, panel2->GetBounds().origin());
+  EXPECT_EQ(docked_position3, panel3->GetBounds().origin());
+  EXPECT_EQ(docked_position2, panel4->GetBounds().origin());
+
+  // Drag to attach a detached panel to most-left.
+  // Expect to have:
+  //   docked:    P1  P4  P2
+  //   overflow:  P7  P6  P5  P3
+  gfx::Point drag_delta_to_attach3(
+      docked_position3.x() - detached_position1.x() - 50, 210);
+  DragPanelAndFinish(panel1, drag_delta_to_attach3);
+  ASSERT_EQ(0, detached_strip->num_panels());
+  ASSERT_EQ(3, docked_strip->num_panels());
+  ASSERT_EQ(4, overflow_strip->num_panels());
+  EXPECT_EQ(PanelStrip::DOCKED, panel1->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DOCKED, panel2->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::IN_OVERFLOW, panel3->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::DOCKED, panel4->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::IN_OVERFLOW, panel5->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::IN_OVERFLOW, panel6->panel_strip()->type());
+  EXPECT_EQ(PanelStrip::IN_OVERFLOW, panel7->panel_strip()->type());
+  EXPECT_EQ(docked_position3, panel1->GetBounds().origin());
+  EXPECT_EQ(docked_position1, panel2->GetBounds().origin());
+  EXPECT_EQ(docked_position2, panel4->GetBounds().origin());
+
+  panel_manager->CloseAll();
+}