Use TestingProfile::Builder::AddTestingFactory at ChromeRenderViewHostTestHarness

chrome/browser/policy/cloud/user_policy_signin_service_factory.h

 // Copyright (c) 2012 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_POLICY_CLOUD_USER_POLICY_SIGNIN_SERVICE_FACTORY_H_
 #define CHROME_BROWSER_POLICY_CLOUD_USER_POLICY_SIGNIN_SERVICE_FACTORY_H_
 
 #include "base/memory/singleton.h"
 #include "components/browser_context_keyed_service/browser_context_keyed_service_factory.h"
 
@@ skipping 27 matching lines @@
       DeviceManagementService* device_management_service);
 
  protected:
   // BrowserContextKeyedServiceFactory implementation.
   virtual BrowserContextKeyedService* BuildServiceInstanceFor(
       content::BrowserContext* profile) const OVERRIDE;
 
   // Overridden to cause this object to be created when the profile is created.
   virtual bool ServiceIsCreatedWithBrowserContext() const OVERRIDE;
 
+  // Don't create this service for tests.
+  virtual bool ServiceIsNULLWhileTesting() const OVERRIDE;

[Andrew T Wilson (Slow), 2014/03/10 12:30:45]

I'm fundamentally opposed to this change - we should load UPSS with tests to
make them as close to a real runtime environment as possible.

+
   // Register the preferences related to cloud-based user policy.
   virtual void RegisterProfilePrefs(
       user_prefs::PrefRegistrySyncable* registry) OVERRIDE;
 
  private:
   friend struct DefaultSingletonTraits<UserPolicySigninServiceFactory>;
 
   UserPolicySigninServiceFactory();
   virtual ~UserPolicySigninServiceFactory();
 
   DISALLOW_COPY_AND_ASSIGN(UserPolicySigninServiceFactory);
 };
 
 }  // namespace policy
 
 #endif  // CHROME_BROWSER_POLICY_CLOUD_USER_POLICY_SIGNIN_SERVICE_FACTORY_H_