Make ScopedTempDir::Delete return a bool so that its functionality can be te...

base/scoped_temp_dir_unittest.cc

 // Copyright (c) 2009 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/file_util.h"
+#include "base/platform_file.h"
 #include "base/scoped_temp_dir.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 TEST(ScopedTempDir, FullPath) {
   FilePath test_path;
   file_util::CreateNewTempDirectory(FILE_PATH_LITERAL("scoped_temp_dir"),
                                     &test_path);
 
   // Against an existing dir, it should get destroyed when leaving scope.
   EXPECT_TRUE(file_util::DirectoryExists(test_path));
@@ skipping 64 matching lines @@
   EXPECT_FALSE(dir.CreateUniqueTempDir());
   dir.Delete();
   EXPECT_TRUE(dir.CreateUniqueTempDir());
   EXPECT_FALSE(dir.CreateUniqueTempDir());
   ScopedTempDir other_dir;
   other_dir.Set(dir.Take());
   EXPECT_TRUE(dir.CreateUniqueTempDir());
   EXPECT_FALSE(dir.CreateUniqueTempDir());
   EXPECT_FALSE(other_dir.CreateUniqueTempDir());
 }
+
+#if defined(OS_WIN)
+

[Paweł Hajdan Jr., 2010/11/27 14:57:07]

nit: Remove empty line.

[tommi (sloooow) - chröme, 2010/11/28 02:37:03]

Done.

+TEST(ScopedTempDir, LockedTempDir) {
+  ScopedTempDir dir;
+  EXPECT_TRUE(dir.CreateUniqueTempDir());
+  int file_flags = base::PLATFORM_FILE_CREATE_ALWAYS |
+                   base::PLATFORM_FILE_WRITE;
+  base::PlatformFileError error_code = base::PLATFORM_FILE_OK;
+  FilePath file_path(dir.path().Append(FILE_PATH_LITERAL("temp")));
+  base::PlatformFile file = base::CreatePlatformFile(file_path, file_flags,
+                                                     NULL, &error_code);
+  EXPECT_NE(base::kInvalidPlatformFileValue, file);
+  EXPECT_EQ(base::PLATFORM_FILE_OK, error_code);
+  EXPECT_FALSE(dir.Delete());  // We should not be able to delete.
+  EXPECT_FALSE(dir.path().empty());  // We should still have a valid path.
+  EXPECT_TRUE(base::ClosePlatformFile(file));
+  // Now, we should be able to delete.
+  EXPECT_TRUE(dir.Delete());
+}
+

[Paweł Hajdan Jr., 2010/11/27 14:57:07]

nit: Remove empty line.

[tommi (sloooow) - chröme, 2010/11/28 02:37:03]

Done.

+#endif

[Paweł Hajdan Jr., 2010/11/27 14:57:07]

nit:  // defined(OS_WIN)

[tommi (sloooow) - chröme, 2010/11/28 02:37:03]

Done.