Update webkit/ to use scoped_refptr<T>::get() rather than implicit "operator T*"

webkit/browser/fileapi/syncable/local_file_sync_context_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 "webkit/browser/fileapi/syncable/local_file_sync_context.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/file_util.h"
@@ skipping 189 matching lines @@
 
   scoped_refptr<LocalFileSyncContext> sync_context_;
 
   SyncStatusCode status_;
   base::PlatformFileError file_error_;
   bool async_modify_finished_;
   bool has_inflight_prepare_for_sync_;
 };
 
 TEST_F(LocalFileSyncContextTest, ConstructAndDestruct) {
-  sync_context_ = new LocalFileSyncContext(
-      ui_task_runner_, io_task_runner_);
+  sync_context_ =
+      new LocalFileSyncContext(ui_task_runner_.get(), io_task_runner_.get());
   sync_context_->ShutdownOnUIThread();
 }
 
 TEST_F(LocalFileSyncContextTest, InitializeFileSystemContext) {
-  CannedSyncableFileSystem file_system(GURL(kOrigin1), kServiceName,
-                                       io_task_runner_, file_task_runner_);
+  CannedSyncableFileSystem file_system(GURL(kOrigin1),
+                                       kServiceName,
+                                       io_task_runner_.get(),
+                                       file_task_runner_.get());
   file_system.SetUp();
 
-  sync_context_ = new LocalFileSyncContext(ui_task_runner_, io_task_runner_);
+  sync_context_ =
+      new LocalFileSyncContext(ui_task_runner_.get(), io_task_runner_.get());
 
   // Initializes file_system using |sync_context_|.
   EXPECT_EQ(SYNC_STATUS_OK,
-            file_system.MaybeInitializeFileSystemContext(sync_context_));
+            file_system.MaybeInitializeFileSystemContext(sync_context_.get()));
 
   // Make sure everything's set up for file_system to be able to handle
   // syncable file system operations.
   EXPECT_TRUE(file_system.file_system_context()->sync_context() != NULL);
   EXPECT_TRUE(file_system.file_system_context()->change_tracker() != NULL);
   EXPECT_EQ(sync_context_.get(),
             file_system.file_system_context()->sync_context());
 
   // Calling MaybeInitialize for the same context multiple times must be ok.
   EXPECT_EQ(SYNC_STATUS_OK,
-            file_system.MaybeInitializeFileSystemContext(sync_context_));
+            file_system.MaybeInitializeFileSystemContext(sync_context_.get()));
   EXPECT_EQ(sync_context_.get(),
             file_system.file_system_context()->sync_context());
 
   // Opens the file_system, perform some operation and see if the change tracker
   // correctly captures the change.
   EXPECT_EQ(base::PLATFORM_FILE_OK, file_system.OpenFileSystem());
 
   const FileSystemURL kURL(file_system.URL("foo"));
   EXPECT_EQ(base::PLATFORM_FILE_OK, file_system.CreateFile(kURL));
 
   FileSystemURLSet urls;
   file_system.GetChangedURLsInTracker(&urls);
   ASSERT_EQ(1U, urls.size());
   EXPECT_TRUE(ContainsKey(urls, kURL));
 
   // Finishing the test.
   sync_context_->ShutdownOnUIThread();
   file_system.TearDown();
 }
 
 TEST_F(LocalFileSyncContextTest, MultipleFileSystemContexts) {
-  CannedSyncableFileSystem file_system1(GURL(kOrigin1), kServiceName,
-                                        io_task_runner_, file_task_runner_);
-  CannedSyncableFileSystem file_system2(GURL(kOrigin2), kServiceName,
-                                        io_task_runner_, file_task_runner_);
+  CannedSyncableFileSystem file_system1(GURL(kOrigin1),
+                                        kServiceName,
+                                        io_task_runner_.get(),
+                                        file_task_runner_.get());
+  CannedSyncableFileSystem file_system2(GURL(kOrigin2),
+                                        kServiceName,
+                                        io_task_runner_.get(),
+                                        file_task_runner_.get());
   file_system1.SetUp();
   file_system2.SetUp();
 
-  sync_context_ = new LocalFileSyncContext(ui_task_runner_, io_task_runner_);
+  sync_context_ =
+      new LocalFileSyncContext(ui_task_runner_.get(), io_task_runner_.get());
 
   // Initializes file_system1 and file_system2.
   EXPECT_EQ(SYNC_STATUS_OK,
-            file_system1.MaybeInitializeFileSystemContext(sync_context_));
+            file_system1.MaybeInitializeFileSystemContext(sync_context_.get()));
   EXPECT_EQ(SYNC_STATUS_OK,
-            file_system2.MaybeInitializeFileSystemContext(sync_context_));
+            file_system2.MaybeInitializeFileSystemContext(sync_context_.get()));
 
   EXPECT_EQ(base::PLATFORM_FILE_OK, file_system1.OpenFileSystem());
   EXPECT_EQ(base::PLATFORM_FILE_OK, file_system2.OpenFileSystem());
 
   const FileSystemURL kURL1(file_system1.URL("foo"));
   const FileSystemURL kURL2(file_system2.URL("bar"));
 
   // Creates a file in file_system1.
   EXPECT_EQ(base::PLATFORM_FILE_OK, file_system1.CreateFile(kURL1));
 
@@ skipping 52 matching lines @@
 }
 
 // LocalFileSyncContextTest.PrepareSyncWhileWriting is flaky on android.
 // http://crbug.com/239793
 #if defined(OS_ANDROID)
 #define MAYBE_PrepareSyncWhileWriting DISABLED_PrepareSyncWhileWriting
 #else
 #define MAYBE_PrepareSyncWhileWriting PrepareSyncWhileWriting
 #endif
 TEST_F(LocalFileSyncContextTest, MAYBE_PrepareSyncWhileWriting) {
-  CannedSyncableFileSystem file_system(GURL(kOrigin1), kServiceName,
-                                       io_task_runner_, file_task_runner_);
+  CannedSyncableFileSystem file_system(GURL(kOrigin1),
+                                       kServiceName,
+                                       io_task_runner_.get(),
+                                       file_task_runner_.get());
   file_system.SetUp();
-  sync_context_ = new LocalFileSyncContext(ui_task_runner_, io_task_runner_);
+  sync_context_ =
+      new LocalFileSyncContext(ui_task_runner_.get(), io_task_runner_.get());
   EXPECT_EQ(SYNC_STATUS_OK,
-            file_system.MaybeInitializeFileSystemContext(sync_context_));
+            file_system.MaybeInitializeFileSystemContext(sync_context_.get()));
 
   EXPECT_EQ(base::PLATFORM_FILE_OK, file_system.OpenFileSystem());
 
   const FileSystemURL kURL1(file_system.URL("foo"));
 
   // Creates a file in file_system.
   EXPECT_EQ(base::PLATFORM_FILE_OK, file_system.CreateFile(kURL1));
 
   // Kick file write on IO thread.
   StartModifyFileOnIOThread(&file_system, kURL1);
@@ skipping 31 matching lines @@
   EXPECT_TRUE(changes.list().back().IsAddOrUpdate());
   EXPECT_EQ(SYNC_FILE_TYPE_FILE, metadata.file_type);
   EXPECT_EQ(1, metadata.size);
 
   sync_context_->ShutdownOnUIThread();
   sync_context_ = NULL;
   file_system.TearDown();
 }
 
 TEST_F(LocalFileSyncContextTest, ApplyRemoteChangeForDeletion) {
-  CannedSyncableFileSystem file_system(GURL(kOrigin1), kServiceName,
-                                       io_task_runner_, file_task_runner_);
+  CannedSyncableFileSystem file_system(GURL(kOrigin1),
+                                       kServiceName,
+                                       io_task_runner_.get(),
+                                       file_task_runner_.get());
   file_system.SetUp();
 
-  sync_context_ = new LocalFileSyncContext(ui_task_runner_, io_task_runner_);
+  sync_context_ =
+      new LocalFileSyncContext(ui_task_runner_.get(), io_task_runner_.get());
   ASSERT_EQ(SYNC_STATUS_OK,
-            file_system.MaybeInitializeFileSystemContext(sync_context_));
+            file_system.MaybeInitializeFileSystemContext(sync_context_.get()));
   ASSERT_EQ(base::PLATFORM_FILE_OK, file_system.OpenFileSystem());
 
   // Record the initial usage (likely 0).
   int64 initial_usage = -1;
   int64 quota = -1;
   EXPECT_EQ(quota::kQuotaStatusOk,
             file_system.GetUsageAndQuota(&initial_usage, &quota));
 
   // Create a file and directory in the file_system.
   const FileSystemURL kFile(file_system.URL("file"));
@@ skipping 60 matching lines @@
 
   sync_context_->ShutdownOnUIThread();
   sync_context_ = NULL;
   file_system.TearDown();
 }
 
 TEST_F(LocalFileSyncContextTest, ApplyRemoteChangeForAddOrUpdate) {
   base::ScopedTempDir temp_dir;
   ASSERT_TRUE(temp_dir.CreateUniqueTempDir());
 
-  CannedSyncableFileSystem file_system(GURL(kOrigin1), kServiceName,
-                                       io_task_runner_, file_task_runner_);
+  CannedSyncableFileSystem file_system(GURL(kOrigin1),
+                                       kServiceName,
+                                       io_task_runner_.get(),
+                                       file_task_runner_.get());
   file_system.SetUp();
 
-  sync_context_ = new LocalFileSyncContext(ui_task_runner_, io_task_runner_);
+  sync_context_ =
+      new LocalFileSyncContext(ui_task_runner_.get(), io_task_runner_.get());
   ASSERT_EQ(SYNC_STATUS_OK,
-            file_system.MaybeInitializeFileSystemContext(sync_context_));
+            file_system.MaybeInitializeFileSystemContext(sync_context_.get()));
   ASSERT_EQ(base::PLATFORM_FILE_OK, file_system.OpenFileSystem());
 
   const FileSystemURL kFile1(file_system.URL("file1"));
   const FileSystemURL kFile2(file_system.URL("file2"));
   const FileSystemURL kDir(file_system.URL("dir"));
 
   const char kTestFileData0[] = "0123456789";
   const char kTestFileData1[] = "Lorem ipsum!";
   const char kTestFileData2[] = "This is sample test data.";
 
@@ skipping 120 matching lines @@
   EXPECT_EQ(base::PLATFORM_FILE_OK, file_system.DirectoryExists(kDir));
 
   sync_context_->ShutdownOnUIThread();
   file_system.TearDown();
 }
 
 TEST_F(LocalFileSyncContextTest, ApplyRemoteChangeForAddOrUpdate_NoParent) {
   base::ScopedTempDir temp_dir;
   ASSERT_TRUE(temp_dir.CreateUniqueTempDir());
 
-  CannedSyncableFileSystem file_system(GURL(kOrigin1), kServiceName,
-                                       io_task_runner_, file_task_runner_);
+  CannedSyncableFileSystem file_system(GURL(kOrigin1),
+                                       kServiceName,
+                                       io_task_runner_.get(),
+                                       file_task_runner_.get());
   file_system.SetUp();
 
-  sync_context_ = new LocalFileSyncContext(ui_task_runner_, io_task_runner_);
+  sync_context_ =
+      new LocalFileSyncContext(ui_task_runner_.get(), io_task_runner_.get());
   ASSERT_EQ(SYNC_STATUS_OK,
-            file_system.MaybeInitializeFileSystemContext(sync_context_));
+            file_system.MaybeInitializeFileSystemContext(sync_context_.get()));
   ASSERT_EQ(base::PLATFORM_FILE_OK, file_system.OpenFileSystem());
 
   const char kTestFileData[] = "Lorem ipsum!";
   const FileSystemURL kDir(file_system.URL("dir"));
   const FileSystemURL kFile(file_system.URL("dir/file"));
 
   // Either kDir or kFile not exist yet.
   EXPECT_EQ(base::PLATFORM_FILE_ERROR_NOT_FOUND, file_system.FileExists(kDir));
   EXPECT_EQ(base::PLATFORM_FILE_ERROR_NOT_FOUND, file_system.FileExists(kFile));
 
@@ skipping 21 matching lines @@
 
   // Make sure kDir and kFile are created by ApplyRemoteChange.
   EXPECT_EQ(base::PLATFORM_FILE_OK, file_system.FileExists(kFile));
   EXPECT_EQ(base::PLATFORM_FILE_OK, file_system.DirectoryExists(kDir));
 
   sync_context_->ShutdownOnUIThread();
   file_system.TearDown();
 }
 
 }  // namespace sync_file_system