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

Unified Diff: chrome/browser/ui/browser.cc

Issue 11085053: Improving window auto management between workspaces (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed Created 8 years, 2 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/browser.cc
diff --git a/chrome/browser/ui/browser.cc b/chrome/browser/ui/browser.cc
index 99e19a0babab69abd24e81c8daae768a5fdb0e62..73b80a74dc69d2e8ff4b88c4a757660270511d81 100644
--- a/chrome/browser/ui/browser.cc
+++ b/chrome/browser/ui/browser.cc
@@ -260,6 +260,7 @@ Browser::CreateParams::CreateParams()
host_desktop_type(kDefaultHostDesktopType),
app_type(APP_TYPE_HOST),
initial_show_state(ui::SHOW_STATE_DEFAULT),
+ initial_user_has_changed_window_or_position(false),
is_session_restore(false),
window(NULL) {
}
@@ -270,6 +271,7 @@ Browser::CreateParams::CreateParams(Profile* profile)
host_desktop_type(kDefaultHostDesktopType),
app_type(APP_TYPE_HOST),
initial_show_state(ui::SHOW_STATE_DEFAULT),
+ initial_user_has_changed_window_or_position(false),
is_session_restore(false),
window(NULL) {
}
@@ -280,6 +282,7 @@ Browser::CreateParams::CreateParams(Type type, Profile* profile)
host_desktop_type(kDefaultHostDesktopType),
app_type(APP_TYPE_HOST),
initial_show_state(ui::SHOW_STATE_DEFAULT),
+ initial_user_has_changed_window_or_position(false),
is_session_restore(false),
window(NULL) {
}
@@ -292,6 +295,7 @@ Browser::CreateParams::CreateParams(Type type,
host_desktop_type(host_desktop_type),
app_type(APP_TYPE_HOST),
initial_show_state(ui::SHOW_STATE_DEFAULT),
+ initial_user_has_changed_window_or_position(false),
is_session_restore(false),
window(NULL) {
}
@@ -309,6 +313,7 @@ Browser::CreateParams Browser::CreateParams::CreateForApp(
params.app_name = app_name;
params.app_type = APP_TYPE_CHILD;
params.initial_bounds = window_bounds;
+ params.initial_user_has_changed_window_or_position = false;
return params;
}
@@ -340,6 +345,9 @@ Browser::Browser(const CreateParams& params)
cancel_download_confirmation_state_(NOT_PROMPTED),
override_bounds_(params.initial_bounds),
initial_show_state_(params.initial_show_state),
+ initial_user_has_changed_window_or_position_(
+ params.is_session_restore &&
+ params.initial_user_has_changed_window_or_position),
is_session_restore_(params.is_session_restore),
host_desktop_type_(params.host_desktop_type),
ALLOW_THIS_IN_INITIALIZER_LIST(

Powered by Google App Engine
This is Rietveld 408576698