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

chrome/browser/ui/cocoa/user_manager_mac.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_COCOA_USER_MANAGER_MAC_H_
 #define CHROME_BROWSER_UI_COCOA_USER_MANAGER_MAC_H_
 
 #import <Cocoa/Cocoa.h>
 
 #include "base/mac/scoped_nsobject.h"
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/profiles/profile.h"
 
 class UserManagerMac;
 @class UserManagerWindowController;
 
 namespace content {
 class NavigationController;
 class WebContents;
 }
 
 // Dialog widget that contains the Desktop User Manager webui.
 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();
 
   // Returns whether or not the User Manager is showing.
   static bool IsShowing();
 
   // Called by the cocoa window controller when its window closes and the
   // controller destroyed itself. Deletes the instance.
   void WindowWasClosed();
 
  private:
   explicit UserManagerMac(Profile* profile);
   virtual ~UserManagerMac();
 
   // If the |guest_profile| has been initialized succesfully (according to
   // |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);
 
   // An open User Manager window. There can only be one open at a time. This
   // is reset to NULL when the window is closed.
   static UserManagerMac* instance_;  // Weak.
 
   // Controller of the window.
   base::scoped_nsobject<UserManagerWindowController> window_controller_;
 
   DISALLOW_COPY_AND_ASSIGN(UserManagerMac);
 };
 
 #endif  // CHROME_BROWSER_UI_COCOA_USER_MANAGER_MAC_H_