Clean up empty directories after install and on errors

chrome/browser/component_updater/component_updater_utils.h

Index: chrome/browser/component_updater/component_updater_utils.h
diff --git a/chrome/browser/component_updater/component_updater_utils.h b/chrome/browser/component_updater/component_updater_utils.h
index f4b3fdddf0b616b60490a723b303ac7f05c90ce0..f1c29a1c251081080f1fc2762f5e0c5e71eb4575 100644
--- a/chrome/browser/component_updater/component_updater_utils.h
+++ b/chrome/browser/component_updater/component_updater_utils.h
@@ -9,6 +9,10 @@
 
 class GURL;
 
+namespace base {
+class FilePath;
+}
+
 namespace net {
 class URLFetcher;
 class URLFetcherDelegate;
@@ -55,6 +59,9 @@ int GetFetchError(const net::URLFetcher& fetcher);
 // Returns true if the |status_code| represents a server error 5xx.
 bool IsHttpServerError(int status_code);
 
+// Deletes the file and its directory, if the directory is empty.

[waffles, 2013/12/16 17:42:39]

Maybe make a note about what is returned here.

Should the function return "false" if the parent directoy is not empty?

[Sorin Jianu, 2013/12/16 18:06:17]

Changed the name and made the function return true in the case the file was
deleted but the directory was not because the directory was not empty.

+bool DeleteFileAndParentDirectoryIfEmpty(const base::FilePath& filepath);
+
 }  // namespace component_updater
 
 #endif  // CHROME_BROWSER_COMPONENT_UPDATER_COMPONENT_UPDATER_UTILS_H_