base::ReadFile() should return the number of read bytes on Windows.

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 94b545a5f21888f286528c6633046ff64812992b..eb9c6dc23dcf3f3eff8177d6c5083f7a9318385d 100644
--- a/base/file_util_win.cc
+++ b/base/file_util_win.cc
@@ -571,7 +571,7 @@ FILE* OpenFile(const FilePath& filename, const char* mode) {
   return _wfsopen(filename.value().c_str(), w_mode.c_str(), _SH_DENYNO);
 }
 
-int ReadFile(const FilePath& filename, char* data, int size) {
+int ReadFile(const FilePath& filename, char* data, int max_size) {
   ThreadRestrictions::AssertIOAllowed();
   base::win::ScopedHandle file(CreateFile(filename.value().c_str(),
                                           GENERIC_READ,
@@ -584,9 +584,9 @@ int ReadFile(const FilePath& filename, char* data, int size) {
     return -1;
 
   DWORD read;
-  if (::ReadFile(file, data, size, &read, NULL) &&
-      static_cast<int>(read) == size)
+  if (::ReadFile(file, data, max_size, &read, NULL))

[Mark Mentovai, 2014/04/17 16:21:57]

The same concern about short non-EOF reads that I mentioned in the POSIX file
applies here.

[fukino, 2014/04/18 02:22:12]

Yes. filed as http://crbug.com/364762.

     return read;
+
   return -1;
 }