[components] Switch {RefCounted}ProfileKeyedService to use BrowserContext

chrome/browser/extensions/api/identity/identity_apitest.cc

 // 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.
 
 #include "base/string_util.h"
 #include "base/stringprintf.h"
 #include "base/values.h"
 #include "chrome/browser/extensions/api/identity/identity_api.h"
 #include "chrome/browser/extensions/api/identity/web_auth_flow.h"
 #include "chrome/browser/extensions/extension_apitest.h"
@@ skipping 162 matching lines @@
         break;
       }
     }
   }
 
  private:
   ResultType result_;
   OAuth2MintTokenFlow::Delegate* delegate_;
 };
 
-ProfileKeyedService* IdentityAPITestFactory(Profile* profile) {
-  return new IdentityAPI(profile);
+ProfileKeyedService* IdentityAPITestFactory(content::BrowserContext* profile) {

[Jói, 2013/04/23 22:45:22]

nit: Might be nice to rename |profile| to |context|, here and elsewhere. I don't
mind if that's done as a separate follow-up change.

[Paweł Hajdan Jr., 2013/04/23 22:58:11]

I think I'll be changing that later, also including ProfileKeyedService ->
BrowserContextKeyedService etc.

[Jói, 2013/04/24 09:09:10]

OK, fine by me.

+  return new IdentityAPI(static_cast<Profile*>(profile));
 }
 
 }  // namespace
 
 class MockGetAuthTokenFunction : public IdentityGetAuthTokenFunction {
  public:
   MockGetAuthTokenFunction() : login_ui_result_(true),
                                install_ui_result_(false),
                                login_ui_shown_(false),
                                install_ui_shown_(false) {
@@ skipping 666 matching lines @@
       "https://abcdefghij.chromiumapp.org/callback#test')</script>\"}]",
       browser()));
 
   std::string url;
   EXPECT_TRUE(value->GetAsString(&url));
   EXPECT_EQ(std::string("https://abcdefghij.chromiumapp.org/callback#test"),
             url);
 }
 
 }  // namespace extensions