base: Stop overloading ReadFileToString()

chrome/test/ppapi/ppapi_filechooser_browsertest.cc

 // Copyright 2015 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/bind.h"
 #include "base/command_line.h"
 #include "base/files/file_util.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/thread_task_runner_handle.h"
 #include "chrome/common/chrome_switches.h"
@@ skipping 221 matching lines @@
   file_info_list.push_back(
       ui::SelectedFileInfo(suggested_filename, suggested_filename));
   TestSelectFileDialogFactory test_dialog_factory(
       TestSelectFileDialogFactory::REPLACE_BASENAME, file_info_list);
 
   RunTestViaHTTP("FileChooser_SaveAsSafeDefaultName");
   base::FilePath actual_filename = temp_dir.path().AppendASCII("innocuous.txt");
 
   ASSERT_TRUE(base::PathExists(actual_filename));
   std::string file_contents;
-  ASSERT_TRUE(base::ReadFileToString(actual_filename, &file_contents, 100));
+  ASSERT_TRUE(base::ReadFileToString(actual_filename, &file_contents));
   EXPECT_EQ("Hello from PPAPI", file_contents);
 }
 
 IN_PROC_BROWSER_TEST_F(PPAPIFileChooserTest,
                        FileChooser_SaveAs_UnsafeDefaultName) {
   base::ScopedTempDir temp_dir;
   ASSERT_TRUE(temp_dir.CreateUniqueTempDir());
   base::FilePath suggested_filename = temp_dir.path().AppendASCII("foo");
 
   TestSelectFileDialogFactory::SelectedFileInfoList file_info_list;
   file_info_list.push_back(
       ui::SelectedFileInfo(suggested_filename, suggested_filename));
   TestSelectFileDialogFactory test_dialog_factory(
       TestSelectFileDialogFactory::REPLACE_BASENAME, file_info_list);
 
   RunTestViaHTTP("FileChooser_SaveAsUnsafeDefaultName");
   base::FilePath actual_filename = temp_dir.path().AppendASCII("unsafe.txt-");
 
   ASSERT_TRUE(base::PathExists(actual_filename));
   std::string file_contents;
-  ASSERT_TRUE(base::ReadFileToString(actual_filename, &file_contents, 100));
+  ASSERT_TRUE(base::ReadFileToString(actual_filename, &file_contents));
   EXPECT_EQ("Hello from PPAPI", file_contents);
 }
 
 IN_PROC_BROWSER_TEST_F(PPAPIFileChooserTest, FileChooser_SaveAs_Cancel) {
   TestSelectFileDialogFactory test_dialog_factory(
       TestSelectFileDialogFactory::CANCEL,
       TestSelectFileDialogFactory::SelectedFileInfoList());
   RunTestViaHTTP("FileChooser_SaveAsCancel");
 }
 
@@ skipping 14 matching lines @@
   file_info_list.push_back(
       ui::SelectedFileInfo(suggested_filename, suggested_filename));
   TestSelectFileDialogFactory test_dialog_factory(
       TestSelectFileDialogFactory::REPLACE_BASENAME, file_info_list);
 
   RunTestViaHTTP("FileChooser_SaveAsDangerousExecutableAllowed");
   base::FilePath actual_filename = temp_dir.path().AppendASCII("dangerous.exe");
 
   ASSERT_TRUE(base::PathExists(actual_filename));
   std::string file_contents;
-  ASSERT_TRUE(base::ReadFileToString(actual_filename, &file_contents, 100));
+  ASSERT_TRUE(base::ReadFileToString(actual_filename, &file_contents));
   EXPECT_EQ("Hello from PPAPI", file_contents);
 }
 
 IN_PROC_BROWSER_TEST_F(PPAPIFileChooserTest,
                        FileChooser_SaveAs_DangerousExecutable_Disallowed) {
   base::CommandLine::ForCurrentProcess()->AppendSwitch(
       switches::kDisallowUncheckedDangerousDownloads);
   TestSelectFileDialogFactory test_dialog_factory(
       TestSelectFileDialogFactory::NOT_REACHED,
       TestSelectFileDialogFactory::SelectedFileInfoList());
@@ skipping 27 matching lines @@
   file_info_list.push_back(
       ui::SelectedFileInfo(suggested_filename, suggested_filename));
   TestSelectFileDialogFactory test_dialog_factory(
       TestSelectFileDialogFactory::REPLACE_BASENAME, file_info_list);
 
   RunTestViaHTTP("FileChooser_SaveAsDangerousExecutableAllowed");
   base::FilePath actual_filename = temp_dir.path().AppendASCII("dangerous.exe");
 
   ASSERT_TRUE(base::PathExists(actual_filename));
   std::string file_contents;
-  ASSERT_TRUE(base::ReadFileToString(actual_filename, &file_contents, 100));
+  ASSERT_TRUE(base::ReadFileToString(actual_filename, &file_contents));
   EXPECT_EQ("Hello from PPAPI", file_contents);
 }
 
 #endif  // FULL_SAFE_BROWSING