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

chrome/browser/ui/cocoa/user_manager_mac.mm

 // 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.
 
 #include "chrome/browser/ui/cocoa/user_manager_mac.h"
 
 #include "base/strings/string_number_conversions.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/browser_dialogs.h"
 #include "chrome/common/url_constants.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_view.h"
 #include "grit/generated_resources.h"
 #include "ui/base/l10n/l10n_util_mac.h"
 
 // Default window size. Taken from the views implementation in
 // chrome/browser/ui/views/user_manager_view.cc.
 // TODO(noms): Figure out if this size can be computed dynamically or adjusted
 // for smaller screens.
 const int kWindowWidth = 900;
 const int kWindowHeight = 700;
 
 namespace chrome {
 
 // Declared in browser_dialogs.h so others don't have to depend on this header.
 void ShowUserManager(const base::FilePath& profile_path_to_focus) {
-  UserManagerMac::Show(profile_path_to_focus);
+  UserManagerMac::Show(profile_path_to_focus, false);
 }
 
 void HideUserManager() {
   UserManagerMac::Hide();
 }
 
 }  // namespace chrome
 
 // Window controller for the User Manager view.
 @interface UserManagerWindowController : NSWindowController <NSWindowDelegate> {
@@ skipping 89 matching lines @@
 
 UserManagerMac::UserManagerMac(Profile* profile) {
   window_controller_.reset([[UserManagerWindowController alloc]
       initWithProfile:profile withObserver:this]);
 }
 
 UserManagerMac::~UserManagerMac() {
 }
 
 // static
-void UserManagerMac::Show(const base::FilePath& profile_path_to_focus) {
+void UserManagerMac::Show(const base::FilePath& profile_path_to_focus,
+                          bool show_tutorial) {
   if (instance_) {
     [instance_->window_controller_ show];
     return;
   }
 
   // Create the guest profile, if necessary, and open the User Manager
   // from the guest profile.
   ProfileManager* profile_manager = g_browser_process->profile_manager();
   profile_manager->CreateProfileAsync(
       ProfileManager::GetGuestProfilePath(),
       base::Bind(&UserManagerMac::OnGuestProfileCreated,
-                 profile_path_to_focus),
+                 profile_path_to_focus,
+                 show_tutorial),
       base::string16(),
       base::string16(),
       std::string());
 }
 
 // static
 void UserManagerMac::Hide() {
   if (instance_)
     [instance_->window_controller_ close];
 }
 
 // static
 bool UserManagerMac::IsShowing() {
   return instance_ ? [instance_->window_controller_ isVisible]: false;
 }
 
 void UserManagerMac::WindowWasClosed() {
   instance_ = NULL;
   delete this;
 }
 
 void UserManagerMac::OnGuestProfileCreated(
     const base::FilePath& profile_path_to_focus,
+    bool show_tutorial,
     Profile* guest_profile,
     Profile::CreateStatus status) {
   if (status != Profile::CREATE_STATUS_INITIALIZED)
     return;
 
   instance_ = new UserManagerMac(guest_profile);
 
   // Tell the webui which user pod should be focused.
   std::string page = chrome::kChromeUIUserManagerURL;
 
-  if (!profile_path_to_focus.empty()) {
+  if (show_tutorial) {
+    page += "#tutorial";

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

So this case ignores the |profile_path_to_focus| param? Can you document the
behaviour in the header comment?

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

Done.

+  } else if (!profile_path_to_focus.empty()) {
     ProfileInfoCache& cache =
         g_browser_process->profile_manager()->GetProfileInfoCache();
     size_t index = cache.GetIndexOfProfileWithPath(profile_path_to_focus);
     if (index != std::string::npos) {
       page += "#";
       page += base::IntToString(index);
     }
   }
   [instance_->window_controller_ showURL:GURL(page)];
 }
-