win: porting for scoped_temp_dir_test

util/test/scoped_temp_dir_test.cc

 // Copyright 2015 The Crashpad Authors. All rights reserved.
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
 //     http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
 #include "util/test/scoped_temp_dir.h"
 
 #include <errno.h>
 #include <fcntl.h>
 #include <string.h>
 #include <sys/stat.h>
-#include <unistd.h>
 
 #include "base/posix/eintr_wrapper.h"
+#include "build/build_config.h"
 #include "gtest/gtest.h"
 #include "util/test/errors.h"
 
+#if defined(OS_POSIX)

[Robert Sesek, 2015/01/07 23:38:25]

Should these come before the project #includes?

[scottmg, 2015/01/08 00:16:11]

They require build/build_config.h, so must go after that. I could put
build_config.h first, then the #if block, then more project #includes, but I
don't think that's common usage, and isn't mentioned in
http://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Names_and_Ord...
.

+#include <unistd.h>
+#elif defined(OS_WIN)
+#include <direct.h>
+#include <io.h>
+#endif  // OS_POSIX
+
 namespace crashpad {
 namespace test {
 namespace {
 
 bool FileExists(const base::FilePath& path) {
 #if defined(OS_POSIX)
   struct stat st;
   int rv = lstat(path.value().c_str(), &st);
+#elif defined(OS_WIN)
+  struct _stat st;
+  int rv = _wstat(path.value().c_str(), &st);
+#else
+#error "Not implemented"
+#endif
   if (rv < 0) {
     EXPECT_EQ(ENOENT, errno) << ErrnoMessage("lstat") << " " << path.value();

[Mark Mentovai, 2015/01/08 17:22:20]

The string isn’t right for Windows. :(

[scottmg, 2015/01/08 17:39:28]

Oops, thanks: https://codereview.chromium.org/836833003

     return false;
   }
   return true;
-#else
-#error "Not implemented"
-#endif
 }
 
 void CreateFile(const base::FilePath& path) {
 #if defined(OS_POSIX)
   int fd = HANDLE_EINTR(creat(path.value().c_str(), 0644));
   ASSERT_GE(fd, 0) << ErrnoMessage("creat") << " " << path.value();
   ASSERT_EQ(0, IGNORE_EINTR(close(fd)))
       << ErrnoMessage("close") << " " << path.value();
+#elif defined(OS_WIN)
+  int fd = _wcreat(path.value().c_str(), 0644);
+  ASSERT_GE(fd, 0) << ErrnoMessage("_wcreat") << " " << path.value();
+  ASSERT_EQ(0, _close(fd)) << ErrnoMessage("_close") << " " << path.value();
 #else
 #error "Not implemented"
 #endif
   EXPECT_TRUE(FileExists(path));
 }
 
 void CreateDirectory(const base::FilePath& path) {
 #if defined(OS_POSIX)
   ASSERT_EQ(0, mkdir(path.value().c_str(), 0755))
       << ErrnoMessage("mkdir") << " " << path.value();
+#elif defined(OS_WIN)
+  ASSERT_EQ(0, _wmkdir(path.value().c_str()))
+      << ErrnoMessage("_wmkdir") << " " << path.value();
 #else
 #error "Not implemented"
 #endif
   ASSERT_TRUE(FileExists(path));
 }
 
 TEST(ScopedTempDir, Empty) {
   base::FilePath path;
   {
     ScopedTempDir dir;
     path = dir.path();
     EXPECT_TRUE(FileExists(path));
   }
   EXPECT_FALSE(FileExists(path));
 }
 
 TEST(ScopedTempDir, WithTwoFiles) {
   base::FilePath parent, file1, file2;
 
   {
     ScopedTempDir dir;
     parent = dir.path();
     ASSERT_TRUE(FileExists(parent));
 
-    file1 = parent.Append("test1");
+    file1 = parent.Append(FILE_PATH_LITERAL("test1"));
     CreateFile(file1);
 
-    file2 = parent.Append("test 2");
+    file2 = parent.Append(FILE_PATH_LITERAL("test 2"));
     CreateFile(file2);
   }
 
   EXPECT_FALSE(FileExists(file1));
   EXPECT_FALSE(FileExists(file2));
   EXPECT_FALSE(FileExists(parent));
 }
 
 TEST(ScopedTempDir, WithRecursiveDirectory) {
   base::FilePath parent, file1, child_dir, file2;
 
   {
     ScopedTempDir dir;
     parent = dir.path();
     ASSERT_TRUE(FileExists(parent));
 
-    file1 = parent.Append(".first-level file");
+    file1 = parent.Append(FILE_PATH_LITERAL(".first-level file"));
     CreateFile(file1);
 
-    child_dir = parent.Append("subdir");
+    child_dir = parent.Append(FILE_PATH_LITERAL("subdir"));
     CreateDirectory(child_dir);
 
-    file2 = child_dir.Append("second level file");
+    file2 = child_dir.Append(FILE_PATH_LITERAL("second level file"));
     CreateFile(file2);
   }
 
   EXPECT_FALSE(FileExists(file1));
   EXPECT_FALSE(FileExists(file2));
   EXPECT_FALSE(FileExists(child_dir));
   EXPECT_FALSE(FileExists(parent));
 }
 
 }  // namespace
 }  // namespace test
 }  // namespace crashpad