Moves extension_icon_image and image_loader to extensions/browser

extensions/browser/component_extension_resource_manager.h

Index: extensions/browser/component_extension_resource_manager.h
diff --git a/extensions/browser/component_extension_resource_manager.h b/extensions/browser/component_extension_resource_manager.h
new file mode 100644
index 0000000000000000000000000000000000000000..d475f8192068289fcf7b45df931115d62c3d723f
--- /dev/null
+++ b/extensions/browser/component_extension_resource_manager.h
@@ -0,0 +1,29 @@
+// 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_COMPONENT_EXTENSION_RESOURCE_MANAGER_H_
+#define EXTENSIONS_BROWSER_COMPONENT_EXTENSION_RESOURCE_MANAGER_H_
+
+#include "base/files/file_path.h"
+
+namespace extensions {
+
+class ComponentExtensionResourceManager {
+ public:
+  // Sets/gets the singleton instance of ComponentExtensionResourceManager.
+  static void Set(ComponentExtensionResourceManager* manager);

[James Cook, 2014/06/17 22:01:04]

Rather than using a singleton I would prefer that you get a
ComponentExtensionResourceManager* via the ExtensionsClient or
ExtensionsBrowserClient interface. It's OK if it returns NULL.  That way the
extension system embedder knows that they must consider the issue of component
resources.

[Jun Mukai, 2014/06/17 23:53:02]

Sounds reasonable. Rather than returning an instance, I added a method itself to
ExtensionsBrowserClient.

+  static ComponentExtensionResourceManager* Get();
+  static bool HasInstance();
+
+  virtual ~ComponentExtensionResourceManager() {}
+
+  virtual bool IsComponentExtensionResource(
+    const base::FilePath& extension_path,
+    const base::FilePath& resource_path,
+    int* resource_id) = 0;
+};
+
+}  // namespace extensions
+
+#endif  // EXTENSIONS_BROWSER_COMPONENT_EXTENSION_RESOURCE_MANAGER_H_