[Win] Add a fast profile switcher to the Windows taskbar item.

chrome/browser/chrome_browser_main.cc

Index: chrome/browser/chrome_browser_main.cc
diff --git a/chrome/browser/chrome_browser_main.cc b/chrome/browser/chrome_browser_main.cc
index 48e78ea8a92c668dd6211a47ed568f96ede7503c..044dd06f037180541a72c97e263680e7cbb7b8aa 100644
--- a/chrome/browser/chrome_browser_main.cc
+++ b/chrome/browser/chrome_browser_main.cc
@@ -165,6 +165,7 @@
 #include "chrome/browser/component_updater/sw_reporter_installer_win.h"
 #include "chrome/browser/first_run/try_chrome_dialog_view.h"
 #include "chrome/browser/first_run/upgrade_util_win.h"
+#include "chrome/browser/jumplist_metrics.h"
 #include "chrome/browser/ui/network_profile_bubble.h"
 #include "chrome/installer/util/helper.h"
 #include "chrome/installer/util/install_util.h"
@@ -455,6 +456,14 @@ bool ProcessSingletonNotificationCallback(
   if (!g_browser_process || g_browser_process->IsShuttingDown())
     return false;
 
+#if defined(OS_WIN)
+  // Log whether this process was a result of an action in the Windows Jumplist.
+  if (command_line.HasSwitch(switches::kWinJumplistAction)) {
+    JumplistMetrics::LogJumplistActionFromSwitchValue(
+        command_line.GetSwitchValueASCII(switches::kWinJumplistAction));

[tapted, 2014/11/10 23:47:41]

If you want to catch metrics recording for the cases Chrome isn't running, I
think this will need to go somewhere else. I'm thinking mainly about the
pinned-to-start-menu case (as opposed to pinned-to-taskbar). i.e. like
https://drive.google.com/a/google.com/file/d/0Bw8WCbRaSxWbYTlaTVRfNXRWM00/vie...
[corp]

You probably want this in StartupBrowserCreator::ProcessCmdLineImpl(). Maybe the
not-running case is different enough to want to have a separate UMA (or
different enum values). Otherwise (if you stay with just one UMA axis), you'll
also want to move the activate-existing-browser part back into
ProcessCmdLineImpl (from ProcessCommandLineAlreadyRunning), otherwise the logic
will bail out before recording UMA.

edit: after seeing sky's comment - perhaps moving it back into
ProcessCmdLineImpl is better anyway :)

[noms (inactive), 2014/11/12 19:04:11]

Ahh, I misunderstood when this was called. I was under the impression it was
called every time you open a new browser, but I don't think that's true. Done!

+  }
+#endif
+
   if (command_line.HasSwitch(switches::kOriginalProcessStartTime)) {
     std::string start_time_string =
         command_line.GetSwitchValueASCII(switches::kOriginalProcessStartTime);