non-new-profile-management creates a "no-op" style account_reconcilor,

components/signin/core/browser/test_signin_client.h

Index: components/signin/core/browser/test_signin_client.h
diff --git a/components/signin/core/browser/test_signin_client.h b/components/signin/core/browser/test_signin_client.h
index d5ad38c747a6ec6123c5f8a185a7573009726d0b..294aa59c91be8af90381a9d280a865d57cbd9c86 100644
--- a/components/signin/core/browser/test_signin_client.h
+++ b/components/signin/core/browser/test_signin_client.h
@@ -17,12 +17,19 @@
 #include "ios/public/test/fake_profile_oauth2_token_service_ios_provider.h"
 #endif
 
+class Profile;
+
+namespace content {
+class BrowserContext;
+}

[Roger Tawa OOO till Jul 10th, 2014/05/27 19:15:38]

Do we need these forward declares?  I'm not sure there should be in //component
code.

Should forward declare PrefService?

[Mike Lerman, 2014/05/27 19:43:03]

Done.

+
 // An implementation of SigninClient for use in unittests. Instantiates test
 // versions of the various objects that SigninClient is required to provide as
 // part of its interface.
 class TestSigninClient : public SigninClient {
  public:
   TestSigninClient();
+  TestSigninClient(PrefService* pref_service);
   virtual ~TestSigninClient();
 
   // SigninClient implementation that is specialized for unit tests.
@@ -74,6 +81,8 @@ class TestSigninClient : public SigninClient {
   scoped_refptr<TokenWebData> database_;
   int signin_host_id_;
 
+  PrefService* pref_service_;
+
 #if defined(OS_IOS)
   scoped_ptr<ios::FakeProfileOAuth2TokenServiceIOSProvider> iosProvider_;
 #endif