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

Unified Diff: chrome/browser/ui/webui/signin/user_manager_screen_handler.cc

Issue 631163004: Mac - show user manager before opening browser. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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/webui/signin/user_manager_screen_handler.cc
diff --git a/chrome/browser/ui/webui/signin/user_manager_screen_handler.cc b/chrome/browser/ui/webui/signin/user_manager_screen_handler.cc
index 7512ce809639c30c20543ff91f791516314248fd..76324cfbaf1a1b531ea5da39022ee52f802db4a9 100644
--- a/chrome/browser/ui/webui/signin/user_manager_screen_handler.cc
+++ b/chrome/browser/ui/webui/signin/user_manager_screen_handler.cc
@@ -32,6 +32,7 @@
#include "chrome/grit/chromium_strings.h"
#include "chrome/grit/generated_resources.h"
#include "content/public/browser/notification_service.h"
+#include "content/public/browser/user_metrics.h"
#include "content/public/browser/web_contents.h"
#include "content/public/browser/web_ui.h"
#include "google_apis/gaia/gaia_auth_fetcher.h"
@@ -717,11 +718,15 @@ void UserManagerScreenHandler::OnBrowserWindowReady(Browser* browser) {
DCHECK(browser);
DCHECK(browser->window());
if (url_hash_ == profiles::kUserManagerSelectProfileTaskManager) {
- base::MessageLoop::current()->PostTask(
- FROM_HERE, base::Bind(&chrome::ShowTaskManager, browser));
+ content::RecordAction(base::UserMetricsAction("TaskManager"));
Mark Mentovai 2014/10/07 20:22:57 Seems weird that some things have their UMA action
Mike Lerman 2014/10/08 15:41:37 This generally happens in the chrome::OpenTaskMana
+ base::MessageLoop::current()->PostTask(
+ FROM_HERE, base::Bind(&chrome::ShowTaskManager, browser));
} else if (url_hash_ == profiles::kUserManagerSelectProfileAboutChrome) {
- base::MessageLoop::current()->PostTask(
- FROM_HERE, base::Bind(&chrome::ShowAboutChrome, browser));
+ base::MessageLoop::current()->PostTask(
+ FROM_HERE, base::Bind(&chrome::ShowAboutChrome, browser));
+ } else if (url_hash_ == profiles::kUserManagerSelectProfileChromeSettings) {
+ base::MessageLoop::current()->PostTask(
+ FROM_HERE, base::Bind(&chrome::ShowSettings, browser));
}
// This call is last as it deletes this object.
« chrome/browser/profiles/profile_window.cc ('K') | « chrome/browser/profiles/profile_window.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698