Add ExtensionsTestBrowserContext class for test.

extensions/browser/extensions_test_browser_context.cc

Index: extensions/browser/extensions_test_browser_context.cc
diff --git a/extensions/browser/extensions_test_browser_context.cc b/extensions/browser/extensions_test_browser_context.cc
new file mode 100644
index 0000000000000000000000000000000000000000..eae98fb09dbf4283375f6c514770894bbebbbe55
--- /dev/null
+++ b/extensions/browser/extensions_test_browser_context.cc
@@ -0,0 +1,25 @@
+// Copyright 2014 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 "extensions/browser/extensions_test_browser_context.h"
+
+#include "components/keyed_service/content/browser_context_dependency_manager.h"
+#include "extensions/browser/extensions_browser_client.h"
+#include "extensions/browser/test_extensions_browser_client.h"
+
+namespace extensions {
+
+ExtensionsTestBrowserContext::ExtensionsTestBrowserContext() {
+  BrowserContextDependencyManager::GetInstance()
+      ->CreateBrowserContextServicesForTest(this);
+  ExtensionsBrowserClient::Set(new TestExtensionsBrowserClient(this));

[Yoyo Zhou, 2014/07/18 01:47:13]

It looks like this is leaked. Can you mean the TestExtensionsBrowserClient a
member of this class?

[limasdf, 2014/07/18 02:22:25]

This is necessary because from the
ChromeExtensionsBrowserClient::GetPrefServiceForContext,
ChromeExtensionsBrowserClient::GetOriginalContext,
ChromeExtensionsBrowserClient::GetOfftheRecordContext do 'static_cast' to
Profile. and call Profile::GetPrefs(), GetOriginalProfile(),
HasOfftheRecordProfile(). but BrowserContext doesn't have those methods. So
there's crash.

Yeah, we eventually have to fix ChromeExtensionsBrowserClinet to handle pure
BrowserContext. I think.

[Yoyo Zhou, 2014/07/18 02:38:45]

Sorry, I meant to write "make" instead of "mean" above.

I mean the part:
new TestExtensionsBrowserClient...
This TestExtensionsBrowserClient never gets deleted. But if we make it a member
of ExtensionsTestBrowserContext, and we call
ExtensionsBrowserClient::Set(&test_extensions_browser_client_);
that would fix it.

[limasdf, 2014/07/18 06:48:12]

Done.

Yes exactly, there was a leak.
Fixed.

+}
+
+ExtensionsTestBrowserContext::~ExtensionsTestBrowserContext() {
+  BrowserContextDependencyManager::GetInstance()
+      ->DestroyBrowserContextServices(this);
+  ExtensionsBrowserClient::Set(NULL);
+}
+
+}  // namespace extensions