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

Unified Diff: chrome/browser/profiles/avatar_menu_actions_desktop.cc

Issue 1662783002: Remove HostDesktopType from Browser::CreateParams (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@metro-mode-12
Patch Set: remove desktop_type_ in bookmark_bubble_sign_in_delegate Created 4 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
Index: chrome/browser/profiles/avatar_menu_actions_desktop.cc
diff --git a/chrome/browser/profiles/avatar_menu_actions_desktop.cc b/chrome/browser/profiles/avatar_menu_actions_desktop.cc
index faba359a7e2f79a15aff561dddc817c03db6f70e..3a3a87294dd4c1b0d43623ad218cfb4afdf58bac 100644
--- a/chrome/browser/profiles/avatar_menu_actions_desktop.cc
+++ b/chrome/browser/profiles/avatar_menu_actions_desktop.cc
@@ -41,8 +41,7 @@ void AvatarMenuActionsDesktop::AddNewProfile(ProfileMetrics::ProfileAdd type) {
Browser* settings_browser = browser_;
if (!settings_browser) {
- const Browser::CreateParams params(ProfileManager::GetLastUsedProfile(),
- chrome::GetActiveDesktop());
+ const Browser::CreateParams params(ProfileManager::GetLastUsedProfile());
settings_browser = new Browser(params);
}
chrome::ShowSettingsSubPage(settings_browser, chrome::kCreateProfileSubPage);
@@ -52,8 +51,7 @@ void AvatarMenuActionsDesktop::AddNewProfile(ProfileMetrics::ProfileAdd type) {
void AvatarMenuActionsDesktop::EditProfile(Profile* profile, size_t index) {
Browser* settings_browser = browser_;
if (!settings_browser) {
- settings_browser = new Browser(
- Browser::CreateParams(profile, chrome::GetActiveDesktop()));
+ settings_browser = new Browser(Browser::CreateParams(profile));
}
// TODO(davidben): The manageProfile page only allows editting the profile
// associated with the browser it is opened in. AvatarMenuActionsDesktop
« no previous file with comments | « chrome/browser/printing/print_dialog_cloud_win.cc ('k') | chrome/browser/profiles/profile_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698