Added WARN_UNUSED_RESULT to ScopedTempDir methods.

webkit/fileapi/file_system_path_manager_unittest.cc

 // Copyright (c) 2010 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 "webkit/fileapi/file_system_path_manager.h"
 
 #include "base/basictypes.h"
 #include "base/file_util.h"
 #include "base/message_loop.h"
 #include "base/message_loop_proxy.h"
@@ skipping 147 matching lines @@
 }  // namespace
 
 class FileSystemPathManagerTest : public testing::Test {
  public:
   FileSystemPathManagerTest()
       : callback_factory_(ALLOW_THIS_IN_INITIALIZER_LIST(this)) {
   }
 
   void SetUp() {
     data_dir_.reset(new ScopedTempDir);
-    data_dir_->CreateUniqueTempDir();
+    ASSERT_TRUE(data_dir_->CreateUniqueTempDir());
     ASSERT_TRUE(data_dir_->IsValid());

[Scott Hess - ex-Googler, 2011/01/06 19:17:26]

Second ASSERT_TRUE() is probably now removable.

[cbentzel, 2011/01/06 19:20:33]

Thanks. Done.

     root_path_callback_status_ = false;
     root_path_.clear();
     file_system_name_.clear();
   }
 
  protected:
   FileSystemPathManager* NewPathManager(
       bool incognito,
       bool allow_file_access) {
     return new FileSystemPathManager(
@@ skipping 238 matching lines @@
 TEST_F(FileSystemPathManagerTest, IsRestrictedName) {
   scoped_ptr<FileSystemPathManager> manager(NewPathManager(false, false));
   for (size_t i = 0; i < ARRAYSIZE_UNSAFE(kIsRestrictedNameTestCases); ++i) {
     SCOPED_TRACE(testing::Message() << "IsRestrictedName #" << i << " "
                  << kIsRestrictedNameTestCases[i].name);
     FilePath name(kIsRestrictedNameTestCases[i].name);
     EXPECT_EQ(kIsRestrictedNameTestCases[i].expected_dangerous,
               manager->IsRestrictedFileName(name));
   }
 }