Added new ReadFileToString API with a max_size argument

base/file_util.h

Index: base/file_util.h
diff --git a/base/file_util.h b/base/file_util.h
index f8418318a6a7ecd92255e6d708624b7f1ae6bcf7..1b67d6a231b868dbfbf09a7beb7bca10f58b1770 100644
--- a/base/file_util.h
+++ b/base/file_util.h
@@ -137,6 +137,14 @@ BASE_EXPORT bool TextContentsEqual(const FilePath& filename1,
 // Useful for unit tests.
 BASE_EXPORT bool ReadFileToString(const FilePath& path, std::string* contents);
 
+// Read the file at |path| into |contents|, returning true on success.
+// This function has an additional check on the maximum size of the file.
+// When the file size is greater than |maxsize|, the function returns false
+// with |contents| pointing to empty string.

[bartfab (slow), 2014/02/11 12:51:12]

Nit: s/to empty/to an empty/

[kaliamoorthi, 2014/02/11 16:29:16]

Done.

+BASE_EXPORT bool ReadFileToString(const FilePath& path,
+                                  std::string* contents,
+                                  size_t maxsize);

[bartfab (slow), 2014/02/11 12:51:12]

Nit: s/maxsize/max_size/ to be more consistent with other variable names in
Chrome code

[kaliamoorthi, 2014/02/11 16:29:16]

Done.

+
 #if defined(OS_POSIX)
 
 // Read exactly |bytes| bytes from file descriptor |fd|, storing the result