Support detaching/attaching panels via inter-strip drags.

chrome/browser/ui/panels/panel.cc

Index: chrome/browser/ui/panels/panel.cc
diff --git a/chrome/browser/ui/panels/panel.cc b/chrome/browser/ui/panels/panel.cc
index 9544590d0559f1aa49cea24967fc2601cbc159cd..a00eb9a35de2206a5b3a6cb9077fda775ed5491d 100644
--- a/chrome/browser/ui/panels/panel.cc
+++ b/chrome/browser/ui/panels/panel.cc
@@ -50,9 +50,10 @@ Panel::Panel(Browser* browser, const gfx::Size& requested_size)
       has_temporary_layout_(false),
       restored_size_(requested_size),
       auto_resizable_(false),
+      always_on_top_(false),
+      in_preview_mode_(false),
       expansion_state_(EXPANDED),
-      old_expansion_state_(EXPANDED),
-      app_icon_visible_(true) {
+      old_expansion_state_(EXPANDED) {
 }
 
 Panel::~Panel() {
@@ -151,12 +152,21 @@ void Panel::SetSizeRange(const gfx::Size& min_size, const gfx::Size& max_size) {
 }
 
 void Panel::SetAppIconVisibility(bool visible) {
-  if (app_icon_visible_ == visible)
-    return;
-  app_icon_visible_ = visible;
   native_panel_->SetPanelAppIconVisibility(visible);
 }
 
+void Panel::SetAlwaysOnTop(bool on_top) {
+  if (always_on_top_ == on_top)
+    return;
+  always_on_top_ = on_top;
+  native_panel_->SetPanelAlwaysOnTop(on_top);

[Andrei, 2012/03/07 22:55:13]

Actually the native panel needs to be on top iff always_on_top_ ||
in_preview_mode_

[jianli, 2012/03/08 01:18:26]

Per suggestion from Jenn, we're not going to make dragging panel always visible
for now.

+}
+
+void Panel::SetPreviewMode(bool in_preview) {
+  DCHECK_NE(in_preview_mode_, in_preview);
+  in_preview_mode_ = in_preview;
+}
+
 void Panel::SetExpansionState(ExpansionState new_state) {
   if (expansion_state_ == new_state)
     return;