This is the fourth CL of several that will eventually replace TokenService with

chrome/test/base/testing_profile_manager.h

Index: chrome/test/base/testing_profile_manager.h
diff --git a/chrome/test/base/testing_profile_manager.h b/chrome/test/base/testing_profile_manager.h
index 01f331760e10360e63b0785f666dc70b0f2857d2..b0596a93f7e42016407f40a0ea5a2d8c1aafd035 100644
--- a/chrome/test/base/testing_profile_manager.h
+++ b/chrome/test/base/testing_profile_manager.h
@@ -14,6 +14,7 @@
 #include "base/memory/scoped_ptr.h"
 #include "base/strings/string16.h"
 #include "chrome/test/base/scoped_testing_local_state.h"
+#include "chrome/test/base/testing_profile.h"
 
 class PrefServiceSyncable;
 class ProfileInfoCache;
@@ -47,6 +48,16 @@ class TestingProfileManager {
   // ProfileInfoCache and provide the user-visible profile metadata. This will
   // register the TestingProfile with the profile subsystem as well. The
   // subsystem owns the Profile and returns a weak pointer.
+  // |factories| contains BCKSs to use with the newly created profile.
+  TestingProfile* CreateTestingProfileWithFactories(
+      const std::string& profile_name,
+      scoped_ptr<PrefServiceSyncable> prefs,
+      const string16& user_name,
+      int avatar_id,
+      const std::string& managed_user_id,
+      const TestingProfile::TestingFactories& factories);
+
+  // Same as above, with no testing factories.
   TestingProfile* CreateTestingProfile(const std::string& profile_name,

[Paweł Hajdan Jr., 2013/11/18 19:59:33]

Can we delete this and convert callers if there aren't too many?

Having a method with 1 arg vs. 6 args makes sense, but 6 vs. 5 vs. 1 much less.

                                        scoped_ptr<PrefServiceSyncable> prefs,
                                        const string16& user_name,