Change GetProfileToUse method from Profile to content::BrowserContext

chrome/browser/profiles/incognito_helpers.cc

Index: chrome/browser/profiles/incognito_helpers.cc
diff --git a/chrome/browser/profiles/incognito_helpers.cc b/chrome/browser/profiles/incognito_helpers.cc
new file mode 100644
index 0000000000000000000000000000000000000000..6155e9606d67ca9f167fd7568f6fa858b3b166c5
--- /dev/null
+++ b/chrome/browser/profiles/incognito_helpers.cc
@@ -0,0 +1,21 @@
+// Copyright 2013 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 "chrome/browser/profiles/incognito_helpers.h"
+
+#include "chrome/browser/profiles/profile.h"
+
+namespace chrome {
+
+content::BrowserContext* GetBrowserContextRedirectedInIncognito(
+    content::BrowserContext* context) {
+  return static_cast<Profile*>(context)->GetOriginalProfile();

[Elliot Glaysher, 2013/04/30 00:11:12]

The only thing that I worry about is the possibility of something not a Profile
being passed in here.

[Paweł Hajdan Jr., 2013/04/30 18:07:56]

Right, this is a reasonable concern. Note however, that this is the part
intended to stay in chrome, while most of other files will be pushed down to
content.

In chrome, Profile is the only subclass of BrowserContext:

$ git grep 'public BrowserContext'
content/browser/download/download_manager_impl_unittest.cc:class
MockBrowserContext : public BrowserContext {
content/public/test/test_browser_context.h:class TestBrowserContext : public
BrowserContext {
content/shell/shell_browser_context.h:class ShellBrowserContext : public
BrowserContext {
$ git grep 'public content::BrowserContext'
android_webview/browser/aw_browser_context.h:class AwBrowserContext : public
content::BrowserContext,
chrome/browser/profiles/profile.h:class Profile : public content::BrowserContext
{

Now the publicly exposed TestBrowserContext in content is a bit suspicious - I'm
going to give it a try to make it non-public.

Note that TestBrowserContext used not to be in content/public, but got moved in
https://chromiumcodereview.appspot.com/10492009 (and I didn't fully realize the
long-term consequences of exposing another subclass of BrowserContext which is
not Profile).

[Paweł Hajdan Jr., 2013/04/30 18:21:09]

Actually it's not going to be easy because of usage out of content:

ash/test/test_shell_delegate.cc:#include
"content/public/test/test_browser_context.h"
chrome/browser/extensions/event_listener_map_unittest.cc:#include
"content/public/test/test_browser_context.h"
components/visitedlink/test/visitedlink_unittest.cc:#include
"content/public/test/test_browser_context.h"

+}
+
+content::BrowserContext* GetBrowserContextOwnInstanceInIncognito(
+    content::BrowserContext* context) {
+  return context;
+}
+
+}  // namespace chrome