UserManager stack refactoring.

chrome/browser/chromeos/login/test/oobe_base_test.h

 // Copyright 2014 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_CHROMEOS_LOGIN_TEST_OOBE_BASE_TEST_H_
 #define CHROME_BROWSER_CHROMEOS_LOGIN_TEST_OOBE_BASE_TEST_H_
 
 #include <string>
 
 #include "base/callback.h"
 #include "base/command_line.h"
 #include "chrome/browser/chromeos/login/ui/login_display_host_impl.h"
 #include "chrome/browser/chromeos/login/ui/webui_login_display.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "content/public/test/test_utils.h"
 #include "google_apis/gaia/fake_gaia.h"
 #include "net/test/embedded_test_server/embedded_test_server.h"
 
 namespace content {
 class WebUI;
 }  // namespace content
 
+namespace user_manager {
+class FakeUserManager;
+}  // namespace user_manager

[ygorshenin1, 2015/01/20 18:33:40]

No need in "// namespace .*" in case of forward-declarations in header files.
Could you please remove it at lines #22 and #26?

[merkulova, 2015/01/21 14:10:33]

Done.

+
 namespace chromeos {
 
-class FakeUserManager;
 class NetworkPortalDetectorTestImpl;
 
 // Base class for OOBE and Kiosk tests.
 class OobeBaseTest : public ExtensionApiTest {
  public:
   OobeBaseTest();
   ~OobeBaseTest() override;
 
  protected:
   // InProcessBrowserTest overrides.
@@ skipping 25 matching lines @@
   NetworkPortalDetectorTestImpl* network_portal_detector_;
 
   // Whether to use background networking. Note this is only effective when it
   // is set before SetUpCommandLine is invoked.
   bool needs_background_networking_;
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_LOGIN_TEST_OOBE_BASE_TEST_H_