Make FakeChromeUserManager subclass of ChromeUserManager

chrome/browser/ui/webui/settings/profile_info_handler_unittest.cc

 // Copyright 2016 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/webui/settings/profile_info_handler.h"
 
 #include <memory>
 
 #include "chrome/common/pref_names.h"
 #include "chrome/test/base/testing_browser_process.h"
 #include "chrome/test/base/testing_profile.h"
 #include "chrome/test/base/testing_profile_manager.h"
 #include "components/prefs/scoped_user_pref_update.h"
 #include "content/public/browser/web_ui_data_source.h"
 #include "content/public/test/test_browser_thread_bundle.h"
 #include "content/public/test/test_web_ui.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 #if defined(OS_CHROMEOS)
+#include "chrome/browser/chromeos/login/users/fake_chrome_user_manager.h"
 #include "chrome/browser/chromeos/login/users/scoped_user_manager_enabler.h"
-#include "components/user_manager/fake_user_manager.h"
 #endif
 
 namespace settings {
 
 namespace {
 
 class TestProfileInfoHandler : public ProfileInfoHandler {
  public:
   explicit TestProfileInfoHandler(Profile* profile)
       : ProfileInfoHandler(profile) {}
 
   using ProfileInfoHandler::set_web_ui;
 };
 
 }  // namespace
 
 class ProfileInfoHandlerTest : public testing::Test {
  public:
   ProfileInfoHandlerTest()
       : profile_manager_(TestingBrowserProcess::GetGlobal()),
 #if defined(OS_CHROMEOS)
-        user_manager_(new user_manager::FakeUserManager),
+        user_manager_(new chromeos::FakeChromeUserManager),
         user_manager_enabler_(user_manager_),
 #endif
         profile_(nullptr) {
   }
 
   void SetUp() override {
     ASSERT_TRUE(profile_manager_.SetUp());
 
 #if defined(OS_CHROMEOS)
     profile_ = profile_manager_.CreateTestingProfile("fake_id@gmail.com");
@@ skipping 26 matching lines @@
   content::TestWebUI* web_ui() { return &web_ui_; }
   Profile* profile() const { return profile_; }
   TestProfileInfoHandler* handler() const { return handler_.get(); }
 
  private:
   content::TestBrowserThreadBundle thread_bundle_;
   TestingProfileManager profile_manager_;
   content::TestWebUI web_ui_;
 
 #if defined(OS_CHROMEOS)
   // |user_manager_| is destroyed by |user_manager_enabler_|.

[msw, 2016/11/17 18:31:22]

ditto nit: this comment is sufficient, but the |user_manager_| isn't accessed,
so it could be removed and the instantiation inlined.

[yoshiki, 2016/11/18 15:54:15]

Done.

-  user_manager::FakeUserManager* user_manager_;
+  chromeos::FakeChromeUserManager* user_manager_;
   chromeos::ScopedUserManagerEnabler user_manager_enabler_;
 #endif
 
   Profile* profile_;
   std::unique_ptr<TestProfileInfoHandler> handler_;
 };
 
 TEST_F(ProfileInfoHandlerTest, GetProfileInfo) {
   base::ListValue list_args;
   list_args.AppendString("get-profile-info-callback-id");
@@ skipping 75 matching lines @@
   ASSERT_TRUE(data.arg1()->GetAsString(&event_id));
   EXPECT_EQ(ProfileInfoHandler::kProfileManagesSupervisedUsersChangedEventName,
             event_id);
 
   bool has_supervised_users = false;
   ASSERT_TRUE(data.arg2()->GetAsBoolean(&has_supervised_users));
   EXPECT_TRUE(has_supervised_users);
 }
 
 }  // namespace settings