Clean up empty directories after install and on errors

chrome/browser/component_updater/component_updater_utils.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_COMPONENT_UPDATER_COMPONENT_UPDATER_UTILS_H_
 #define CHROME_BROWSER_COMPONENT_UPDATER_COMPONENT_UPDATER_UTILS_H_
 
 #include <string>
 
 class GURL;
 
+namespace base {
+class FilePath;
+}
+
 namespace net {
 class URLFetcher;
 class URLFetcherDelegate;
 class URLRequestContextGetter;
 }
 
 namespace component_updater {
 
 // An update protocol request starts with a common preamble which includes
 // version and platform information for Chrome and the operating system,
@@ skipping 26 matching lines @@
 
 // Returns the error code which occured during the fetch. The function returns 0
 // if the fetch was successful. If errors happen, the function could return a
 // network error, an http response code, or the status of the fetch, if the
 // fetch is pending or canceled.
 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_