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

chrome/browser/ui/cocoa/user_manager_mac.h

Index: chrome/browser/ui/cocoa/user_manager_mac.h
diff --git a/chrome/browser/ui/cocoa/user_manager_mac.h b/chrome/browser/ui/cocoa/user_manager_mac.h
index 0ce144e8bc21a76ddeef68e64a41269db25d152d..1de76b5343392d05a40f307d1c4e400028e76c11 100644
--- a/chrome/browser/ui/cocoa/user_manager_mac.h
+++ b/chrome/browser/ui/cocoa/user_manager_mac.h
@@ -23,8 +23,10 @@ class WebContents;
 class UserManagerMac {
  public:
   // Shows the User Manager or re-activates an existing one, focusing the
-  // profile given by |profile_path_to_focus|.
-  static void Show(const base::FilePath& profile_path_to_focus);
+  // profile given by |profile_path_to_focus|. If |show_tutorial| is true,
+  // then the user manager tutorial will be shown.
+  static void Show(const base::FilePath& profile_path_to_focus,
+                   bool show_tutorial);

[Alexei Svitkine (slow), 2014/03/14 21:04:40]

Can this be an enum instead so that it's obvious what the true/false means at
the call sites? Alternatively, have the method name be more meaningful - e.g.
ShowWithTutorial().

[noms (inactive), 2014/03/17 21:52:23]

Added an enum as in the future we might (very likely) have a second kind of
tutorial to show. I've put it under profiles:: as it needs to be accessed form
all the places.

On 2014/03/14 21:04:40, Alexei Svitkine wrote:

 
   // Hide the User Manager.
   static void Hide();
@@ -44,6 +46,7 @@ class UserManagerMac {
   // |status|), creates a new UserManagerMac instance with the user with path
   // |profile_path_to_focus| focused.
   static void OnGuestProfileCreated(const base::FilePath& profile_path_to_focus,
+                                    bool show_tutorial,
                                     Profile* guest_profile,
                                     Profile::CreateStatus status);