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

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

Issue 2690563005: cros: Disable the auto management logic for v1app browser window (Closed)
Patch Set: nits Created 3 years, 10 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
« no previous file with comments | « no previous file | chrome/browser/ui/views/frame/browser_frame_ash_browsertest.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_ash.cc
diff --git a/chrome/browser/ui/views/frame/browser_frame_ash.cc b/chrome/browser/ui/views/frame/browser_frame_ash.cc
index 1ec42a01eaea303828f0f3383f11516c73c9523c..d40bb82282e9f3b4c9b1d0174d45492466a8b64a 100644
--- a/chrome/browser/ui/views/frame/browser_frame_ash.cc
+++ b/chrome/browser/ui/views/frame/browser_frame_ash.cc
@@ -178,9 +178,10 @@ BrowserFrameAsh::~BrowserFrameAsh() {
// BrowserFrameAsh, private:
void BrowserFrameAsh::SetWindowAutoManaged() {
- if (!browser_view_->browser()->is_type_popup() ||
- browser_view_->browser()->is_app()) {
- ash::wm::GetWindowState(GetNativeWindow())->
- set_window_position_managed(true);
+ // For browser window in Chrome OS, we should only enable the auto window
+ // management logic for tabbed browser.
+ if (!browser_view_->browser()->is_type_popup()) {
+ ash::wm::GetWindowState(GetNativeWindow())
+ ->set_window_position_managed(true);
}
}
« no previous file with comments | « no previous file | chrome/browser/ui/views/frame/browser_frame_ash_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698