Refactor AO2TS to make it easier to componentize.

chrome/browser/extensions/updater/extension_updater_unittest.cc

Index: chrome/browser/extensions/updater/extension_updater_unittest.cc
diff --git a/chrome/browser/extensions/updater/extension_updater_unittest.cc b/chrome/browser/extensions/updater/extension_updater_unittest.cc
index e4b7b7bd7e8ba315c41f05c7f68f80a79f857f4b..52ce95d25cf120ecc7cfe6b7822108b3c1f9ff0d 100644
--- a/chrome/browser/extensions/updater/extension_updater_unittest.cc
+++ b/chrome/browser/extensions/updater/extension_updater_unittest.cc
@@ -387,7 +387,8 @@ class MockService : public TestExtensionService {
     fake_identity_provider.reset(new FakeIdentityProvider(
           fake_token_service_.get()));
     fake_identity_provider->LogIn(kFakeAccountId);
-    fake_token_service_->AddAccount(kFakeAccountId);
+    fake_token_service_->GetDelegate()->UpdateCredentials(kFakeAccountId,

[Marijn Kruisselbrink, 2015/06/29 17:21:39]

Similar to some of the other comments it seems a bit weird to have to call a
method on GetDelegate() here. But in this case it seems especially weird since
ProfileOAuth2TokenService already adds an UpdateCredentials method that just
calls the same method on the delegate; why not move those pass-through methods
to OAuth2TokenService and get rid of the GetDelegate() here?

+                                                          "refresh_token");
 
     scoped_ptr<ExtensionDownloader> downloader(
         CreateExtensionDownloader(delegate));