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

Unified Diff: chrome/browser/ui/views/frame/browser_frame.cc

Issue 8895003: Aura: Add --aura-laptop-mode to fill the workspace with a single window. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase, fix unit test build Created 9 years 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
« no previous file with comments | « chrome/browser/ui/views/frame/browser_frame.h ('k') | chrome/browser/ui/views/frame/browser_view.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/frame/browser_frame.cc
diff --git a/chrome/browser/ui/views/frame/browser_frame.cc b/chrome/browser/ui/views/frame/browser_frame.cc
index d4dc34a0140d56db5296fdda61f61d9d33d37d9a..249470c8a86e52dbb98cb235e357238d30169cc8 100644
--- a/chrome/browser/ui/views/frame/browser_frame.cc
+++ b/chrome/browser/ui/views/frame/browser_frame.cc
@@ -15,6 +15,7 @@
#include "chrome/browser/ui/views/frame/native_browser_frame.h"
#include "chrome/common/chrome_switches.h"
#include "ui/base/theme_provider.h"
+#include "ui/gfx/screen.h"
#include "ui/views/widget/native_widget.h"
#if defined(OS_WIN) && !defined(USE_AURA)
@@ -67,13 +68,26 @@ void BrowserFrame::InitBrowserFrame() {
CommandLine* command_line = CommandLine::ForCurrentProcess();
if (command_line->HasSwitch(switches::kAuraTranslucentFrames))
params.transparent = true;
+ // Aura laptop mode fills the monitor with with its windows.
+ if (command_line->HasSwitch(switches::kAuraLaptopMode) &&
+ browser_view_->IsBrowserTypeNormal()) {
+ params.bounds = gfx::Screen::GetPrimaryMonitorBounds();
+ params.show_state = ui::SHOW_STATE_MAXIMIZED;
+ }
#endif
Init(params);
-#if defined(OS_CHROMEOS) && !defined(USE_AURA)
- // On ChromeOS we always want top-level windows to appear active.
- if (!browser_view_->IsBrowserTypePopup())
- DisableInactiveRendering();
+
+ // On ChromeOS and Aura laptop mode we always want top-level windows
+ // to appear active.
+ bool disable_inactive_rendering = false;
+#if defined(USE_AURA)
+ disable_inactive_rendering =
+ CommandLine::ForCurrentProcess()->HasSwitch(switches::kAuraLaptopMode);
+#elif defined(OS_CHROMEOS)
+ disable_inactive_rendering = true;
#endif
+ if (disable_inactive_rendering && !browser_view_->IsBrowserTypePopup())
+ DisableInactiveRendering();
}
int BrowserFrame::GetMinimizeButtonOffset() const {
@@ -105,8 +119,20 @@ void BrowserFrame::TabStripDisplayModeChanged() {
native_browser_frame_->TabStripDisplayModeChanged();
}
+bool BrowserFrame::IsSingleWindowMode() const {
+ bool single_window_mode = false;
+#if defined(USE_AURA)
+ single_window_mode =
+ CommandLine::ForCurrentProcess()->HasSwitch(switches::kAuraLaptopMode);
+#elif defined(OS_CHROMEOS)
+ single_window_mode =
+ chromeos::system::runtime_environment::IsRunningOnChromeOS();
+#endif
+ return single_window_mode;
+}
+
///////////////////////////////////////////////////////////////////////////////
-// BrowserFrameWin, views::Window overrides:
+// BrowserFrame, views::Widget overrides:
bool BrowserFrame::IsMaximized() const {
#if defined(OS_CHROMEOS) && !defined(USE_AURA)
« no previous file with comments | « chrome/browser/ui/views/frame/browser_frame.h ('k') | chrome/browser/ui/views/frame/browser_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698