Delete temporary application file as soon as the application is launched.

shell/dynamic_application_loader.cc

Index: shell/dynamic_application_loader.cc
diff --git a/shell/dynamic_application_loader.cc b/shell/dynamic_application_loader.cc
index 374484531eaba8ca9b5b7f9f5703bfdeeaf22309..71fb47e25e53361705c744f0d127eda57dfd2d1f 100644
--- a/shell/dynamic_application_loader.cc
+++ b/shell/dynamic_application_loader.cc
@@ -81,6 +81,8 @@ class DynamicApplicationLoader::Loader {
 
   virtual bool PeekFirstLine(std::string* line) = 0;
 
+  virtual bool CleanPath() = 0;

[Aaron Boodman, 2015/01/22 19:51:30]

This doesn't appear to ever be answered based on runtime state. It only depends
on what the subclass is. In that case, it seems like the subclass should just
pass a constant to the superclass' constructor.

[qsr, 2015/01/26 09:12:13]

Done.

+
   void Load() {
     // If the response begins with a #!mojo <content-handler-url>, use it.
     GURL url;
@@ -139,7 +141,7 @@ class DynamicApplicationLoader::Loader {
 
     runner_ = runner_factory_->Create(context_);
     runner_->Start(
-        path, shell_handle_.Pass(),
+        path, CleanPath(), shell_handle_.Pass(),
         base::Bind(&Loader::ReportComplete, weak_ptr_factory_.GetWeakPtr()));
   }
 
@@ -234,6 +236,8 @@ class DynamicApplicationLoader::LocalLoader : public Loader {
     return true;
   }
 
+  bool CleanPath() override { return false; }
+
   GURL url_;
   base::FilePath path_;
 
@@ -339,6 +343,8 @@ class DynamicApplicationLoader::NetworkLoader : public Loader {
                             kPeekTimeout);
   }
 
+  bool CleanPath() override { return true; }
+
   void StartNetworkRequest(const GURL& url, NetworkService* network_service) {
     URLRequestPtr request(URLRequest::New());
     request->url = String::From(url);