Eliminate ProfileOAuth2TokenService being a BCKS.

chrome/browser/signin/fake_profile_oauth2_token_service.h

Index: chrome/browser/signin/fake_profile_oauth2_token_service.h
diff --git a/chrome/browser/signin/fake_profile_oauth2_token_service.h b/chrome/browser/signin/fake_profile_oauth2_token_service.h
index cc5cbc9a784936a9ca9744af3bb81b619a69e25d..8153df060e831ca0e7cd48b6cade285072eab23c 100644
--- a/chrome/browser/signin/fake_profile_oauth2_token_service.h
+++ b/chrome/browser/signin/fake_profile_oauth2_token_service.h
@@ -110,14 +110,14 @@ class FakeProfileOAuth2TokenService
 
   // Helper function to be used with
   // BrowserContextKeyedService::SetTestingFactory().
-  static BrowserContextKeyedService* Build(content::BrowserContext* profile);
+  static BrowserContextKeyedService* Build(content::BrowserContext* context);

[blundell, 2014/02/10 16:19:16]

These functions should move to be on the FakePO2TSWrapper. Should I do that in
this CL do you think?

[Roger Tawa OOO till Jul 10th, 2014/02/10 19:49:13]

Probably makes sense to do it as part of this CL, since its about adding the new
layer of indirection.  Otherwise its a TODO that we'll need schedule later.

[blundell, 2014/02/13 16:07:06]

Done.

 
   // Helper function to be used with
   // BrowserContextKeyedService::SetTestingFactory() that creates a
   // FakeProfileOAuth2TokenService object that posts fetch responses on the
   // current message loop.
   static BrowserContextKeyedService* BuildAutoIssuingTokenService(
-      content::BrowserContext* profile);
+      content::BrowserContext* context);
 
  protected:
   // OAuth2TokenService overrides.