Use permissions 0666 on POSIX platforms except Android

base/files/file_util_posix.cc

 // Copyright (c) 2012 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 "base/files/file_util.h"
 
 #include <dirent.h>
 #include <errno.h>
 #include <fcntl.h>
 #include <libgen.h>
@@ skipping 656 matching lines @@
     return -1;
 
   ssize_t bytes_read = HANDLE_EINTR(read(fd, data, max_size));
   if (IGNORE_EINTR(close(fd)) < 0)
     return -1;
   return bytes_read;
 }
 
 int WriteFile(const FilePath& filename, const char* data, int size) {
   ThreadRestrictions::AssertIOAllowed();
-  int fd = HANDLE_EINTR(creat(filename.value().c_str(), 0640));
+#if !defined(OS_ANDROID)

[Lei Zhang, 2015/10/13 08:46:40]

nit: how about being positive with if-else defines (and statements) ? (See ToTT
387)

[hashimoto, 2015/10/14 03:18:14]

Acknowledged.

+  const mode_t mode = 0666;
+#else
+  const mode_t mode = 0640;  // Be strict on Android. crbug.com/362887
+#endif
+  int fd = HANDLE_EINTR(creat(filename.value().c_str(), mode));
   if (fd < 0)
     return -1;
 
   int bytes_written = WriteFileDescriptor(fd, data, size) ? size : -1;
   if (IGNORE_EINTR(close(fd)) < 0)
     return -1;
   return bytes_written;
 }
 
 bool WriteFileDescriptor(const int fd, const char* data, int size) {
@@ skipping 226 matching lines @@
     return false;
 
   DeleteFile(from_path, true);
   return true;
 }
 
 }  // namespace internal
 
 #endif  // !defined(OS_NACL_NONSFI)
 }  // namespace base