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..63527c6e14d362dcda0b24c0b62b4cbf18850aec 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.
+BASE_EXPORT bool ReadFileToString(const FilePath& path,

[Andrew T Wilson (Slow), 2014/02/11 10:27:12]

One nit: put | around variable names in contents.

So, for example:

"When the file size is greater than |maxsize|, the function returns false with
|contents| set to an empty string".

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

Done.

+                                  std::string* contents,
+                                  size_t maxsize);
+
 #if defined(OS_POSIX)
 
 // Read exactly |bytes| bytes from file descriptor |fd|, storing the result