ExtensionSystem cleanup part 2

extensions/browser/extension_system_provider.h

Index: extensions/browser/extension_system_provider.h
diff --git a/extensions/browser/extension_system_provider.h b/extensions/browser/extension_system_provider.h
new file mode 100644
index 0000000000000000000000000000000000000000..45ca56c185cfd9ee876d39337e9ea7d037c590be
--- /dev/null
+++ b/extensions/browser/extension_system_provider.h
@@ -0,0 +1,35 @@
+// 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.
+
+#ifndef EXTENSIONS_BROWSER_EXTENSION_SYSTEM_PROVIDER_H_
+#define EXTENSIONS_BROWSER_EXTENSION_SYSTEM_PROVIDER_H_
+
+#include "components/browser_context_keyed_service/browser_context_keyed_service_factory.h"
+
+class BrowserContextDependencyManager;
+
+namespace content {
+class BrowserContext;
+}
+
+namespace extensions {
+class ExtensionSystem;
+
+// An ExtensionSystemProvider maps a BrowserContext to its ExtensionSystem.
+// Different applications may use this to provide differing implementations
+// of ExtensionSystem.
+// TODO(yoz): Rename to ExtensionSystemFactory.

[James Cook, 2014/02/12 17:18:12]

I presume these renames are for step 3 or 4 or whatever?

[Yoyo Zhou, 2014/02/12 23:32:10]

Yeah, step 3 is to give things more correct names. Not sure if there is a step 4
- I was going to have one, but see note.

+class ExtensionSystemProvider : public BrowserContextKeyedServiceFactory {
+ public:
+  ExtensionSystemProvider(const char* name,
+                          BrowserContextDependencyManager* manager);
+  virtual ~ExtensionSystemProvider();
+
+  virtual ExtensionSystem* GetForBrowserContext(
+      content::BrowserContext* context) = 0;
+};
+
+}  // namespace extensions
+
+#endif  // EXTENSIONS_BROWSER_EXTENSION_SYSTEM_PROVIDER_H_