Fix an issue where themes would sporadically fail to install.

Trying to send decoded images over IPC didn't work too well.  Instead, we'll
write them to a file and have the browser slurp them in from there.  My first
instinct was to use SharedMemory, but that would require us to impose a limit
on the size of the decoded image data.

Also made sure that the undecoded images are deleted when we install.

BUG=13455
TEST=Try the repro steps in bug 13455 several times and make sure it works
every time.
Review URL: http://codereview.chromium.org/119255

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17797 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 1 insertions, 5 deletions

diff --git a/chrome/browser/utility_process_host.h b/chrome/browser/utility_process_host.h
index 0efaa56..03d98c8 100644
--- a/chrome/browser/utility_process_host.h
+++ b/chrome/browser/utility_process_host.h
@@ -6,7 +6,6 @@
 #define CHROME_BROWSER_UTILITY_PROCESS_HOST_H_
 
 #include <string>
-#include <vector>
 
 #include "base/basictypes.h"
 #include "base/ref_counted.h"
@@ -17,7 +16,6 @@
 class CommandLine;
 class DictionaryValue;
 class MessageLoop;
-class SkBitmap;
 
 // This class acts as the browser-side host to a utility child process.  A
 // utility process is a short-lived sandboxed process that is created to run
@@ -38,9 +36,7 @@ class UtilityProcessHost : public ChildProcessHost {
     // Called when the extension has unpacked successfully.  |manifest| is the
     // parsed manifest.json file.  |images| contains a list of decoded images
     // and the associated paths where those images live on disk.
-    virtual void OnUnpackExtensionSucceeded(
-        const DictionaryValue& manifest,
-        const std::vector< Tuple2<SkBitmap, FilePath> >& images) {}
+    virtual void OnUnpackExtensionSucceeded(const DictionaryValue& manifest) {}
 
     // Called when an error occurred while unpacking the extension.
     // |error_message| contains a description of the problem.