Make all the files mapped in when running base_unittests read only.

base/file_util_win.cc

Index: base/file_util_win.cc
diff --git a/base/file_util_win.cc b/base/file_util_win.cc
index 6ff1820a0bc3555ed8c4baeb259ead5b9b228c01..edcec535a472ff3161e81a7ed82b20734732d55e 100644
--- a/base/file_util_win.cc
+++ b/base/file_util_win.cc
@@ -728,8 +728,24 @@ bool CopyFileUnsafe(const FilePath& from_path, const FilePath& to_path) {
       to_path.value().length() >= MAX_PATH) {
     return false;
   }
-  return (::CopyFile(from_path.value().c_str(), to_path.value().c_str(),
-                     false) != 0);
+
+  // Unlike the posix implementation that copies the file manually and discards
+  // the ACL bits, CopyFile() copies the complete SECURITY_DESCRIPTOR and access
+  // bits, which is usually not what we want. We can't do much about the
+  // SECURITY_DESCRIPTOR but at least remove the read only bit.
+  const wchar_t* dest = to_path.value().c_str();
+  if (0 == ::CopyFile(from_path.value().c_str(), dest, false)) {

[Nico, 2014/01/14 02:30:33]

nit: `::CopyFile(...) == 0`; no yoda conditions in chrome

[M-A Ruel, 2014/01/14 14:59:39]

Done.

+    // Copy failed.
+    return false;
+  }
+  DWORD attrs = GetFileAttributes(dest);
+  if (attrs==INVALID_FILE_ATTRIBUTES) {

[Nico, 2014/01/14 02:30:33]

nit: spaces around ==

[M-A Ruel, 2014/01/14 14:59:39]

Done.

+    return false;
+  }
+  if (attrs & FILE_ATTRIBUTE_READONLY) {
+    SetFileAttributes(dest, attrs & ~FILE_ATTRIBUTE_READONLY);
+  }
+  return true;
 }
 
 bool CopyAndDeleteDirectory(const FilePath& from_path,