Check for CloseHandle failures even when not debugging

base/files/file_util_win.cc

Index: base/files/file_util_win.cc
diff --git a/base/files/file_util_win.cc b/base/files/file_util_win.cc
index e254232f66305910e0bd64d0f2c7ab71b5e3fac2..071ff1f73940328e81caa1515984b623fcb37dd0 100644
--- a/base/files/file_util_win.cc
+++ b/base/files/file_util_win.cc
@@ -231,14 +231,13 @@ bool PathExists(const FilePath& path) {
 
 bool PathIsWritable(const FilePath& path) {
   ThreadRestrictions::AssertIOAllowed();
-  HANDLE dir =
+  win::ScopedHandle dir(
       CreateFile(path.value().c_str(), FILE_ADD_FILE, kFileShareAll,
-                 NULL, OPEN_EXISTING, FILE_FLAG_BACKUP_SEMANTICS, NULL);
+                 NULL, OPEN_EXISTING, FILE_FLAG_BACKUP_SEMANTICS, NULL));
 
-  if (dir == INVALID_HANDLE_VALUE)
+  if (!dir.IsValid())

[rvargas (doing something else), 2015/09/16 02:09:12]

nit: return dir.IsValid();

[brucedawson, 2015/09/16 18:29:00]

D'oh! Of course.

     return false;
 
-  CloseHandle(dir);
   return true;
 }