Removing, opening and creating files in platform_file and file

webrtc/base/platform_file.h

Index: webrtc/base/platform_file.h
diff --git a/webrtc/base/platform_file.h b/webrtc/base/platform_file.h
index 12e08e9acc2d278ab2474f4144be4244f402f6ee..3bd1b1c035a2168916f451e609670cd186c3a668 100644
--- a/webrtc/base/platform_file.h
+++ b/webrtc/base/platform_file.h
@@ -13,8 +13,10 @@
 
 #include <stdio.h>
 

[perkj_webrtc, 2016/09/15 13:35:30]

nit: remove extra line after stdio.h

+#include <string>
+
 #if defined(WEBRTC_WIN)
-#include <windows.h>
+#include "webrtc/base/win32.h"
 #endif
 
 namespace rtc {
@@ -39,6 +41,19 @@ FILE* FdopenPlatformFileForWriting(PlatformFile file);
 // Use fclose instead.
 bool ClosePlatformFile(PlatformFile file);
 
+// Removes a file in the filesystem.
+bool RemoveFile(const std::string& path);
+
+// Opens a file for reading and writing. You might want to use base/file.h
+// instead, this function is largely used for tests that require exactly a

[perkj_webrtc, 2016/09/15 13:35:30]

dito as comment below.

+// PlatformFile.
+PlatformFile OpenPlatformFile(const std::string& path);
+
+// Creates a new file for reading and writing. If the file already exists it
+// will be overwritten. You might want to use base/file.h instead, this

[perkj_webrtc, 2016/09/15 13:35:30]

instead. 
Suggest you remove the sentence about how this is used since it might change in
the future.

+// function is largely used for tests that require exactly a PlatformFile.
+PlatformFile CreatePlatformFile(const std::string& path);
+
 }  // namespace rtc
 
 #endif  // WEBRTC_BASE_PLATFORM_FILE_H_