Clean up empty directories after install and on errors

chrome/browser/component_updater/component_updater_utils.cc

 // 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.
 
 #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"
 #include "base/win/windows_version.h"
 #include "chrome/common/chrome_version_info.h"
 #include "chrome/common/omaha_query_params/omaha_query_params.h"
 #include "net/base/load_flags.h"
 #include "net/url_request/url_fetcher.h"
 #include "net/url_request/url_request_context_getter.h"
 #include "net/url_request/url_request_status.h"
@@ skipping 79 matching lines @@
       // Network errors are small negative numbers.
       const int error = fetcher.GetStatus().error();
       return error ? error : -1;
     }
 
     default:
       return -1;
   }
 }
 
-
 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