Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(296)

Unified Diff: chrome/browser/ui/panels/panel.h

Issue 9546001: Support detaching/attaching panels via inter-strip drags. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix per feedback Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/panels/panel.h
diff --git a/chrome/browser/ui/panels/panel.h b/chrome/browser/ui/panels/panel.h
index 495f05c6e130741cd0d2fb59fbd5bb9e4e1dc738..ba67467b36726eb43811724004cbd26f0b9e1013 100644
--- a/chrome/browser/ui/panels/panel.h
+++ b/chrome/browser/ui/panels/panel.h
@@ -230,6 +230,8 @@ class Panel : public BrowserWindow,
const gfx::Size& max_size() const { return max_size_; }
bool auto_resizable() const { return auto_resizable_; }
+ bool in_preview_mode() const { return in_preview_mode_; }
+
bool draggable() const;
// The restored size is the size of the panel when it is expanded.
@@ -259,6 +261,13 @@ class Panel : public BrowserWindow,
// Sets whether the panel app icon is visible in the taskbar.
void SetAppIconVisibility(bool visible);
+ // Sets whether the panel window is always on top.
+ void SetAlwaysOnTop(bool on_top);
+
+ // Sets whether the panel is shown in preview mode. When the panel is
+ // being dragged, it is in preview mode.
+ void SetPreviewMode(bool in_preview_mode);
+
// Newly created panels may be placed in a temporary layout until their
// final position is determined.
bool has_temporary_layout() const { return has_temporary_layout_; }
@@ -312,6 +321,14 @@ class Panel : public BrowserWindow,
// True if this panel auto resizes based on content.
bool auto_resizable_;
+ // True if this panel should stay always on top of other windows.
jennb 2012/03/08 23:41:09 s/stay always/always stay
jianli 2012/03/09 21:48:58 Done.
+ bool always_on_top_;
+
+ // True if this panel is in preview mode. That is, panel bounds should not be
jennb 2012/03/08 23:41:09 s/That is,/When in preview mode
jianli 2012/03/09 21:48:58 Done.
+ // affected by layout refresh. This is currently used by drag controller to
+ // add a panel to the strip without causing its bounds changed.
jennb 2012/03/08 23:41:09 s/changed/to change
jianli 2012/03/09 21:48:58 Done.
+ bool in_preview_mode_;
+
// Platform specifc implementation for panels. It'd be one of
// PanelBrowserWindowGtk/PanelBrowserView/PanelBrowserWindowCocoa.
NativePanel* native_panel_; // Weak, owns us.
@@ -319,9 +336,6 @@ class Panel : public BrowserWindow,
ExpansionState expansion_state_;
ExpansionState old_expansion_state_;
- // Indicates whether the panel app icon is visible in the taskbar.
- bool app_icon_visible_;
-
content::NotificationRegistrar registrar_;
DISALLOW_COPY_AND_ASSIGN(Panel);

Powered by Google App Engine
This is Rietveld 408576698