Remove PlatformFile from webrtc_log_uploader

chrome/browser/media/webrtc_log_uploader_unittest.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 <string>
 
 #include "base/file_util.h"
+#include "base/files/file.h"
 #include "base/files/file_path.h"
 #include "base/logging.h"
-#include "base/platform_file.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_split.h"
 #include "base/time/time.h"
 #include "chrome/browser/media/webrtc_log_uploader.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 const char kTestTime[] = "time";
 const char kTestReportId[] = "report-id";
 const char kTestLocalId[] = "local-id";
 
@@ skipping 48 matching lines @@
     EXPECT_EQ(3u, line_parts.size());
     if (3u != line_parts.size())
       return false;
     EXPECT_FALSE(line_parts[0].empty());
     EXPECT_STREQ(kTestReportId, line_parts[1].c_str());
     EXPECT_TRUE(line_parts[2].empty());
     return true;
   }
 
   bool AddLinesToTestFile(int number_of_lines) {
-    int flags = base::PLATFORM_FILE_OPEN |
-                base::PLATFORM_FILE_APPEND;
-    base::PlatformFileError error = base::PLATFORM_FILE_OK;
-    base::PlatformFile test_list_file =
-        base::CreatePlatformFile(test_list_path_, flags, NULL, &error);
-    EXPECT_EQ(base::PLATFORM_FILE_OK, error);
-    EXPECT_NE(base::kInvalidPlatformFileValue, test_list_file);
-    if (base::PLATFORM_FILE_OK != error ||
-        base::kInvalidPlatformFileValue == test_list_file) {
+    base::File test_list_file(test_list_path_,
+                              base::File::FLAG_OPEN | base::File::FLAG_APPEND);
+    EXPECT_TRUE(test_list_file.IsValid());
+    if (!test_list_file.IsValid())
       return false;
-    }
 
     for (int i = 0; i < number_of_lines; ++i) {
       EXPECT_EQ(static_cast<int>(sizeof(kTestTime)) - 1,
-                base::WritePlatformFileAtCurrentPos(test_list_file,
-                                                    kTestTime,
-                                                    sizeof(kTestTime) - 1));
-      EXPECT_EQ(1, base::WritePlatformFileAtCurrentPos(test_list_file, ",", 1));
+                test_list_file.WriteAtCurrentPos(kTestTime,
+                                                 sizeof(kTestTime) - 1));
+      EXPECT_EQ(1, test_list_file.WriteAtCurrentPos(",", 1));
       EXPECT_EQ(static_cast<int>(sizeof(kTestReportId)) - 1,
-                base::WritePlatformFileAtCurrentPos(test_list_file,
-                                                    kTestReportId,
-                                                    sizeof(kTestReportId) - 1));
-      EXPECT_EQ(1, base::WritePlatformFileAtCurrentPos(test_list_file, ",", 1));
+                test_list_file.WriteAtCurrentPos(kTestReportId,
+                                                 sizeof(kTestReportId) - 1));
+      EXPECT_EQ(1, test_list_file.WriteAtCurrentPos(",", 1));
       EXPECT_EQ(static_cast<int>(sizeof(kTestLocalId)) - 1,
-                base::WritePlatformFileAtCurrentPos(
-                    test_list_file, kTestLocalId, sizeof(kTestLocalId) - 1));
-      EXPECT_EQ(1, base::WritePlatformFileAtCurrentPos(test_list_file,
-                                                       "\n", 1));
+                test_list_file.WriteAtCurrentPos(kTestLocalId,
+                                                 sizeof(kTestLocalId) - 1));
+      EXPECT_EQ(1, test_list_file.WriteAtCurrentPos("\n", 1));
     }
-    EXPECT_TRUE(base::ClosePlatformFile(test_list_file));
-
     return true;
   }
 
   std::vector<std::string> GetLinesFromListFile() {
     std::string contents;
     int read = base::ReadFileToString(test_list_path_, &contents);
     EXPECT_GT(read, 0);
     if (read == 0)
       return std::vector<std::string>();
     // Since every line should end with '\n', the last line should be empty. So
@@ skipping 73 matching lines @@
       test_list_path_, kTestLocalId, kTestReportId);
   ASSERT_TRUE(VerifyNumberOfLines(1));
   ASSERT_TRUE(VerifyLastLineHasAllInfo());
 
   // Use a local ID that should not be found in the list.
   webrtc_log_uploader_->AddUploadedLogInfoToUploadListFile(
       test_list_path_, "dummy id", kTestReportId);
   ASSERT_TRUE(VerifyNumberOfLines(2));
   ASSERT_TRUE(VerifyLastLineHasUploadTimeAndIdOnly());
 }