summaryrefslogtreecommitdiffstats
path: root/apps/ui
diff options
context:
space:
mode:
authoroshima@chromium.org <oshima@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-11-19 02:08:17 +0000
committeroshima@chromium.org <oshima@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-11-19 02:08:17 +0000
commit44c10a8cd1ade56e9be8b76b0dc105ad312804b9 (patch)
tree4e61f399f81170298a6e5a00cedfbec8a367d6b1 /apps/ui
parent479af6862c431082a4375ee9ad6d820ed908d5ef (diff)
downloadchromium_src-44c10a8cd1ade56e9be8b76b0dc105ad312804b9.zip
chromium_src-44c10a8cd1ade56e9be8b76b0dc105ad312804b9.tar.gz
chromium_src-44c10a8cd1ade56e9be8b76b0dc105ad312804b9.tar.bz2
Revert 235861 "Add ash-enable-immersive-all-windows command line..."
> Add ash-enable-immersive-all-windows command line flag to enable immersive fullscreen for non-frameless v2 apps. (The goal is to add support for immersive fullscreen for v1 apps and miscellaneous windows such as the task manager shortly) When the flag is set, <F4> will put the app into immersive fullscreen. > > In immersive fullscreen, the window header (the title bar and window controls) slide out as an overlay over the web contents when the mouse is hovered at the top of the display. > > chrome.app.window.current().isFullscreen() returns false for immersive fullscreen windows > Entering fullscreen via the app.window api or webkitRequestFullScreen() quits immersive fullscreen. > > BUG=307622 > R=oshima,benwells > TBR=sky (For adding VIEWS_EXPORT to ui/views/widget/widget_observer.h) > TEST=Manual, see steps below: > 1) Install: https://chrome.google.com/webstore/detail/window-state-sample/hcbhfbnaaancmblfhdknlnojpafjohbi > 2) Enable "Enable immersive fullscreen for non browser windows" in chrome://flags > 3) Hit <F4> > 4) Ensure that the "isFullscreen" checkbox in the app is checked > 5) Ensure that the shelf light bar is visible (The shelf is not completely hidden) > 6) Hover the mouse at the top of the screen. Ensure that the window header (maximize and close button slide down and overlay the web contents) > 7) Move the mouse off of the header. Ensure that the window header slides off screen > 8) Press the Fullscreen button in the app (It may be useful to set the "chrome.app.window actions delay" to zero seconds) > 9) Ensure that the shelf is completely hidden. > 10) Hover the mouse at the top of the screen. Ensure that the header slides out again. > 11) Press <F4> > 12) Ensure that the "isFullscreen" checkbox is unchecked > 13) Ensure that the window header is visible and does not slide out > > Review URL: https://codereview.chromium.org/59043013 BUG=307622,320938 TBR=pkotwicz@chromium.org Review URL: https://codereview.chromium.org/73623008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@235895 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'apps/ui')
-rw-r--r--apps/ui/native_app_window.h7
1 files changed, 2 insertions, 5 deletions
diff --git a/apps/ui/native_app_window.h b/apps/ui/native_app_window.h
index 73f6f88..222a680 100644
--- a/apps/ui/native_app_window.h
+++ b/apps/ui/native_app_window.h
@@ -18,11 +18,8 @@ namespace apps {
class NativeAppWindow : public ui::BaseWindow,
public web_modal::WebContentsModalDialogHost {
public:
- // Sets whether the window is fullscreen and the type of fullscreen.
- // |fullscreen_types| is a bit field of ShellWindow::FullscreenType.
- virtual void SetFullscreen(int fullscreen_types) = 0;
-
- // Returns whether the window is fullscreen or about to enter fullscreen.
+ // Fullscreen changes may be asynchronous on some platforms.
+ virtual void SetFullscreen(bool fullscreen) = 0;
virtual bool IsFullscreenOrPending() const = 0;
// Returns true if the window is a panel that has been detached.