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

chrome/browser/ui/views/profile_chooser_view.cc

 // 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/views/profile_chooser_view.h"
 
 #include "base/prefs/pref_service.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/profiles/profile_info_util.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/profiles/profile_window.h"
 #include "chrome/browser/profiles/profiles_state.h"
 #include "chrome/browser/signin/mutable_profile_oauth2_token_service.h"
 #include "chrome/browser/signin/profile_oauth2_token_service.h"
 #include "chrome/browser/signin/profile_oauth2_token_service_factory.h"
 #include "chrome/browser/signin/signin_manager.h"
 #include "chrome/browser/signin/signin_manager_factory.h"
 #include "chrome/browser/signin/signin_promo.h"
 #include "chrome/browser/ui/browser.h"
-#include "chrome/browser/ui/browser_dialogs.h"
 #include "chrome/browser/ui/singleton_tabs.h"
 #include "chrome/browser/ui/views/user_manager_view.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/url_constants.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
 #include "third_party/skia/include/core/SkColor.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
@@ skipping 484 matching lines @@
                                    profiles::ProfileSwitchingDoneCallback());
   } 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);
+
+    // Show the tutorial if it hasn't been shown before.
+    bool tutorial_shown = browser_->profile()->GetPrefs()->GetBoolean(

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

Same comment about caching profile prefs pointer. Also, add a comment mentioning
the identical logic in the Mac code (and point back to this code from a comment
there).

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

Done.

+        prefs::kProfileUserManagerTutorialShown);
+    if (!tutorial_shown) {
+      browser_->profile()->GetPrefs()->SetBoolean(
+          prefs::kProfileUserManagerTutorialShown, true);
+    }
+    UserManagerView::Show(profile_path, !tutorial_shown);
   } else if (sender == add_user_button_) {
     profiles::CreateAndSwitchToNewProfile(
         browser_->host_desktop_type(),
         profiles::ProfileSwitchingDoneCallback(),
         ProfileMetrics::ADD_NEW_USER_ICON);
   } else if (sender == add_account_button_) {
     ShowView(GAIA_ADD_ACCOUNT_VIEW, avatar_menu_.get());
   } else if (sender == current_profile_photo_->change_photo_button()) {
     avatar_menu_->EditProfile(avatar_menu_->GetActiveProfileIndex());
   } else if (sender == tutorial_ok_button_) {
@@ skipping 405 matching lines @@
   if (!is_primary_account) {
     email_button->set_menu_marker(menu_marker);
     layout->AddPaddingRow(0, views::kRelatedControlVerticalSpacing);
   }
   layout->StartRow(1, 0);
   layout->AddView(email_button);
 
   // Save the original email address, as the button text could be elided.
   current_profile_accounts_map_[email_button] = account;
 }