Remove ClosePlatformFile from media galleries.

chrome/utility/media_galleries/iapps_xml_utils.cc

 // Copyright 2013 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 "chrome/utility/media_galleries/iapps_xml_utils.h"
 
 #include "base/logging.h"
 #include "base/stl_util.h"
 #include "base/strings/string_number_conversions.h"
 #include "third_party/libxml/chromium/libxml_utils.h"
@@ skipping 59 matching lines @@
   return ReadSimpleValue(reader, "string", result);
 }
 
 bool ReadInteger(XmlReader* reader, uint64* result) {
   std::string value;
   if (!ReadSimpleValue(reader, "integer", &value))
     return false;
   return base::StringToUint64(value, result);
 }
 
-std::string ReadPlatformFileAsString(const base::PlatformFile file) {
+std::string ReadFileAsString(base::File file) {
   std::string result;
-  if (file == base::kInvalidPlatformFileValue)
+  if (!file.IsValid())
     return result;
 
   // A "reasonable" artificial limit.
   // TODO(vandebo): Add a UMA to figure out what common values are.
   const int64 kMaxLibraryFileSize = 150 * 1024 * 1024;
-  base::PlatformFileInfo file_info;
-  if (!base::GetPlatformFileInfo(file, &file_info) ||
-      file_info.size > kMaxLibraryFileSize) {
-    base::ClosePlatformFile(file);
+  base::File::Info file_info;
+  if (!file.GetInfo(&file_info) ||  file_info.size > kMaxLibraryFileSize)

[Lei Zhang, 2014/04/28 19:47:00]

nit: only 1 space after ||

[rvargas (doing something else), 2014/04/28 19:53:15]

Done.

     return result;
-  }
 
   result.resize(file_info.size);
-  int bytes_read =
-      base::ReadPlatformFile(file, 0, string_as_array(&result), file_info.size);
+  int bytes_read = file.Read(0, string_as_array(&result), file_info.size);
   if (bytes_read != file_info.size)
     result.clear();
 
-  base::ClosePlatformFile(file);
   return result;
 }
 
 }  // namespace iapps