Clean up empty directories after install and on errors

chrome/browser/component_updater/component_updater_utils.cc

Index: chrome/browser/component_updater/component_updater_utils.cc
diff --git a/chrome/browser/component_updater/component_updater_utils.cc b/chrome/browser/component_updater/component_updater_utils.cc
index e86f72c8a998807080881fc2d0254850bad66545..035d4afd519891fa651185389f87d8a14e79fadb 100644
--- a/chrome/browser/component_updater/component_updater_utils.cc
+++ b/chrome/browser/component_updater/component_updater_utils.cc
@@ -3,6 +3,8 @@
 // found in the LICENSE file.
 
 #include "chrome/browser/component_updater/component_updater_utils.h"
+#include "base/file_util.h"
+#include "base/files/file_path.h"
 #include "base/guid.h"
 #include "base/strings/stringprintf.h"
 #include "base/sys_info.h"
@@ -102,10 +104,20 @@ int GetFetchError(const net::URLFetcher& fetcher) {
   }
 }
 
-
 bool IsHttpServerError(int status_code) {
   return 500 <= status_code && status_code < 600;
 }
 
+bool DeleteFileAndEmptyParentDirectory(const base::FilePath& filepath) {
+  if (!base::DeleteFile(filepath, false))
+    return false;
+
+  const base::FilePath dirname(filepath.DirName());
+  if (!base::IsDirectoryEmpty(dirname))
+    return true;
+
+  return base::DeleteFile(dirname, false);
+}
+

[cpu_(ooo_6.6-7.5), 2013/12/17 00:48:38]

I hope we don't end up deleting %temp% but i guess the chances of that being
empty are of 1 in a trillion.

[Sorin Jianu, 2013/12/17 01:19:40]

I thought about it but what's the fix? My naive thinking was that if we end up
deleting temp, then somebody else will recreate it.

 }  // namespace component_updater