Move ThemeHandler from c/c/e/api; move GetBrowserImages() out of Extension class

chrome/common/extensions/extension_file_util.h

 // 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.
 
 #ifndef CHROME_COMMON_EXTENSIONS_EXTENSION_FILE_UTIL_H_
 #define CHROME_COMMON_EXTENSIONS_EXTENSION_FILE_UTIL_H_
 
 #include <string>
 #include <map>
 
@@ skipping 64 matching lines @@
 // Returns true if the given extension object is valid and consistent.
 // May also append a series of warning messages to |warnings|, but they
 // should not prevent the extension from running.
 //
 // Otherwise, returns false, and a description of the error is
 // returned in |error|.
 bool ValidateExtension(const extensions::Extension* extension,
                        std::string* error,
                        std::vector<extensions::InstallWarning>* warnings);
 
+// Returns a list of paths (relative to the extension dir) for images that
+// the browser might load (like themes and page action icons) for the given
+// extension.
+std::set<base::FilePath> GetExtensionBrowserImages(

[Yoyo Zhou, 2013/04/10 21:44:58]

While we're at it, can we give this a better name? How about
GetBrowserImagePaths
(Extension is implied since it's extension_file_util.)

[Devlin, 2013/04/12 02:17:06]

Done.

+    const extensions::Extension* extension);
+
+
 // Returns a list of files that contain private keys inside |extension_dir|.
-std::vector<base::FilePath> FindPrivateKeyFiles(const base::FilePath& extension_dir);
+std::vector<base::FilePath> FindPrivateKeyFiles(
+    const base::FilePath& extension_dir);
 
 // Cleans up the extension install directory. It can end up with garbage in it
 // if extensions can't initially be removed when they are uninstalled (eg if a
 // file is in use).
 //
 // |extensions_dir| is the install directory to look in. |extension_paths| is a
 // map from extension id to full installation path.
 //
 // Obsolete version directories are removed, as are directories that aren't
 // found in |extension_paths|.
@@ skipping 38 matching lines @@
 base::FilePath GetInstallTempDir(const base::FilePath& extensions_dir);
 
 // Helper function to delete files. This is used to avoid ugly casts which
 // would be necessary with PostMessage since file_util::Delete is overloaded.
 // TODO(skerner): Make a version of Delete that is not overloaded in file_util.
 void DeleteFile(const base::FilePath& path, bool recursive);
 
 }  // namespace extension_file_util
 
 #endif  // CHROME_COMMON_EXTENSIONS_EXTENSION_FILE_UTIL_H_