[Mac, Win] Show a user manager tutorial once per profile.

chrome/browser/ui/views/profile_chooser_view.cc

Index: chrome/browser/ui/views/profile_chooser_view.cc
diff --git a/chrome/browser/ui/views/profile_chooser_view.cc b/chrome/browser/ui/views/profile_chooser_view.cc
index 20c3b73c6e7ab32ffbe7d03f85cf1de60bb43848..1fdaea5419f588dcf16a5ee8a45278bdd6ed8fef 100644
--- a/chrome/browser/ui/views/profile_chooser_view.cc
+++ b/chrome/browser/ui/views/profile_chooser_view.cc
@@ -517,13 +517,10 @@ void ProfileChooserView::ButtonPressed(views::Button* sender,
   } else if (sender == end_guest_button_) {
     profiles::CloseGuestProfileWindows();
   } else if (sender == users_button_) {
-    // Only non-guest users appear in the User Manager.
-    base::FilePath profile_path;
-    if (!end_guest_button_) {
-      size_t active_index = avatar_menu_->GetActiveProfileIndex();
-      profile_path = avatar_menu_->GetItemAt(active_index).profile_path;
-    }
-    chrome::ShowUserManager(profile_path);
+    if (end_guest_button_)

[Alexei Svitkine (slow), 2014/03/19 16:37:01]

Can you preserve some comment explaining this if statement?

[Alexei Svitkine (slow), 2014/03/19 16:37:01]

Can you avoid having the logic to call the different functions by just passing a
NULL profile in the guest case?

i.e.

profiles::ShowUserManagerMaybeWithTutorial(
    end_guest_button_ ? NULL : browser_->profile());

[noms (inactive), 2014/03/19 17:10:38]

Done.

[noms (inactive), 2014/03/19 17:10:38]

Done.

+      chrome::ShowUserManager(base::FilePath());
+    else
+      profiles::ShowUserManagerMaybeWithTutorial(browser_->profile());
   } else if (sender == add_user_button_) {
     profiles::CreateAndSwitchToNewProfile(
         browser_->host_desktop_type(),