Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(289)

Unified Diff: chrome/common/extensions/extension_file_util.cc

Issue 9374009: Install platform apps into a separate data directory (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix test Created 8 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/common/extensions/extension_file_util.cc
diff --git a/chrome/common/extensions/extension_file_util.cc b/chrome/common/extensions/extension_file_util.cc
index ebc064443aab7b62950233e387af8cb957350514..df524e5289220f819f85824438ea9e65067749bb 100644
--- a/chrome/common/extensions/extension_file_util.cc
+++ b/chrome/common/extensions/extension_file_util.cc
@@ -46,7 +46,8 @@ const char kInstallDirectoryName[] = "Extensions";
FilePath InstallExtension(const FilePath& unpacked_source_dir,
const std::string& id,
const std::string& version,
- const FilePath& all_extensions_dir) {
+ const FilePath& all_extensions_dir,
+ DeleteSource should_delete_source) {
FilePath extension_dir = all_extensions_dir.AppendASCII(id);
FilePath version_dir;
@@ -81,7 +82,8 @@ FilePath InstallExtension(const FilePath& unpacked_source_dir,
<< " to : " << extension_temp_dir.path().value() << " failed.";
return FilePath();
}
- file_util::Delete(unpacked_source_dir, true);
+ if (should_delete_source == DELETE_SOURCE_YES)
+ file_util::Delete(unpacked_source_dir, true);
FilePath crx_temp_source =
extension_temp_dir.path().Append(unpacked_source_dir.BaseName());

Powered by Google App Engine
This is Rietveld 408576698