Commit f13cd124 authored by noms's avatar noms Committed by Commit bot

[Mac, Linux] Add People in the menu bar should automatically create a default profile

This will make it have the same behaviour as the User Manager, vs. navigating to
chrome://settings/createProfile.

BUG=452125
TEST=Choose Add Person from the OS menu bar. This should create a profile and
switch to it immediately.

Review URL: https://codereview.chromium.org/870203003

Cr-Commit-Position: refs/heads/master@{#313586}
parent 5ea70900
...@@ -15,6 +15,7 @@ ...@@ -15,6 +15,7 @@
#include "chrome/browser/profiles/profile_info_interface.h" #include "chrome/browser/profiles/profile_info_interface.h"
#include "chrome/browser/profiles/profile_manager.h" #include "chrome/browser/profiles/profile_manager.h"
#include "chrome/browser/profiles/profile_metrics.h" #include "chrome/browser/profiles/profile_metrics.h"
#include "chrome/browser/profiles/profile_window.h"
#include "chrome/browser/ui/browser.h" #include "chrome/browser/ui/browser.h"
#include "chrome/browser/ui/browser_list.h" #include "chrome/browser/ui/browser_list.h"
#include "chrome/browser/ui/browser_list_observer.h" #include "chrome/browser/ui/browser_list_observer.h"
...@@ -98,7 +99,9 @@ class Observer : public chrome::BrowserListObserver, ...@@ -98,7 +99,9 @@ class Observer : public chrome::BrowserListObserver,
} }
- (IBAction)newProfile:(id)sender { - (IBAction)newProfile:(id)sender {
avatarMenu_->AddNewProfile(ProfileMetrics::ADD_NEW_USER_MENU); profiles::CreateAndSwitchToNewProfile(chrome::HOST_DESKTOP_TYPE_NATIVE,
ProfileManager::CreateCallback(),
ProfileMetrics::ADD_NEW_USER_MENU);
} }
- (BOOL)insertItemsIntoMenu:(NSMenu*)menu - (BOOL)insertItemsIntoMenu:(NSMenu*)menu
......
...@@ -21,6 +21,7 @@ ...@@ -21,6 +21,7 @@
#include "chrome/browser/profiles/profile.h" #include "chrome/browser/profiles/profile.h"
#include "chrome/browser/profiles/profile_info_cache.h" #include "chrome/browser/profiles/profile_info_cache.h"
#include "chrome/browser/profiles/profile_manager.h" #include "chrome/browser/profiles/profile_manager.h"
#include "chrome/browser/profiles/profile_window.h"
#include "chrome/browser/sessions/tab_restore_service.h" #include "chrome/browser/sessions/tab_restore_service.h"
#include "chrome/browser/sessions/tab_restore_service_factory.h" #include "chrome/browser/sessions/tab_restore_service_factory.h"
#include "chrome/browser/ui/browser.h" #include "chrome/browser/ui/browser.h"
...@@ -893,5 +894,7 @@ void GlobalMenuBarX11::OnEditProfileItemActivated(DbusmenuMenuitem* sender, ...@@ -893,5 +894,7 @@ void GlobalMenuBarX11::OnEditProfileItemActivated(DbusmenuMenuitem* sender,
void GlobalMenuBarX11::OnCreateProfileItemActivated(DbusmenuMenuitem* sender, void GlobalMenuBarX11::OnCreateProfileItemActivated(DbusmenuMenuitem* sender,
unsigned int timestamp) { unsigned int timestamp) {
avatar_menu_->AddNewProfile(ProfileMetrics::ADD_NEW_USER_MENU); profiles::CreateAndSwitchToNewProfile(chrome::HOST_DESKTOP_TYPE_NATIVE,
ProfileManager::CreateCallback(),
ProfileMetrics::ADD_NEW_USER_MENU);
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment