Replace GDataDirectory::TakeEntry with GDataDirectoryService::AddEntryToDirectory.

chrome/browser/chromeos/gdata/gdata_files_unittest.cc

Index: chrome/browser/chromeos/gdata/gdata_files_unittest.cc
===================================================================
--- chrome/browser/chromeos/gdata/gdata_files_unittest.cc	(revision 150441)
+++ chrome/browser/chromeos/gdata/gdata_files_unittest.cc	(working copy)
@@ -26,24 +26,28 @@
 const char kResumableEditMediaUrl[] = "http://resumable-edit-media/";
 const char kResumableCreateMediaUrl[] = "http://resumable-create-media/";
 
+void OnAddEntryToDirectory(GDataFileError expected_error,
+                           GDataFileError actual_error,
+                           const FilePath& updated_file_path) {
+  EXPECT_EQ(expected_error, actual_error);
+}
+
 // Add a directory to |parent| and return that directory. The name and
 // resource_id are determined by the incrementing counter |sequence_id|.
 GDataDirectory* AddDirectory(GDataDirectory* parent,
                              GDataDirectoryService* directory_service,
                              int sequence_id) {
-  GDataDirectory* dir = new GDataDirectory(parent, directory_service);
+  GDataDirectory* dir = new GDataDirectory(NULL, directory_service);

[satorux1, 2012/08/08 14:09:06]

can you remove the first parameter?

[achuithb, 2012/08/08 22:08:41]

In the next CL?

   const std::string dir_name = "dir" + base::IntToString(sequence_id);
   const std::string resource_id = std::string("dir_resource_id:") +
                                   dir_name;
   dir->set_title(dir_name);
   dir->set_resource_id(resource_id);
-  GDataFileError error = GDATA_FILE_ERROR_FAILED;
   directory_service->AddEntryToDirectory(
       parent->GetFilePath(),
       dir,
-      base::Bind(&test_util::CopyErrorCodeFromFileOperationCallback, &error));
+      base::Bind(&OnAddEntryToDirectory, GDATA_FILE_OK));

[satorux1, 2012/08/08 14:09:06]

Hmm, this looks trickier than the original code. callback is hard to follow for
me. More importantly, with the original code, it was easy to detect where the
test failed from the line number. With this new code, the line number is always
the same (line 32). This is not good.

[achuithb, 2012/08/08 22:08:41]

Good point. I can't use CopyErrorCodeFromFileOperationCallback, so I'd need to
create something like CopyErrorCodeFromFilePathUpdateCallback. Is that what
you'd prefer?

[achuithb, 2012/08/08 22:38:00]

I did this since I'm guessing you prefer this solution.

   test_util::RunBlockingPoolTask();
-  EXPECT_EQ(GDATA_FILE_OK, error);
   return dir;
 }
 
@@ -52,20 +56,18 @@
 GDataFile* AddFile(GDataDirectory* parent,
                    GDataDirectoryService* directory_service,
                    int sequence_id) {
-  GDataFile* file = new GDataFile(parent, directory_service);
+  GDataFile* file = new GDataFile(NULL, directory_service);
   const std::string title = "file" + base::IntToString(sequence_id);
   const std::string resource_id = std::string("file_resource_id:") +
                                   title;
   file->set_title(title);
   file->set_resource_id(resource_id);
   file->set_file_md5(std::string("file_md5:") + title);
-  GDataFileError error = GDATA_FILE_ERROR_FAILED;
   directory_service->AddEntryToDirectory(
       parent->GetFilePath(),
       file,
-      base::Bind(&test_util::CopyErrorCodeFromFileOperationCallback, &error));
+      base::Bind(&OnAddEntryToDirectory, GDATA_FILE_OK));
   test_util::RunBlockingPoolTask();
-  EXPECT_EQ(GDATA_FILE_OK, error);
   return file;
 }
 
@@ -373,21 +375,21 @@
 
 TEST(GDataRootDirectoryTest, RefreshFile) {
   MessageLoopForUI message_loop;
+  content::TestBrowserThread ui_thread(content::BrowserThread::UI,
+                                       &message_loop);
+
   GDataDirectoryService directory_service;
-  GDataDirectory* root(directory_service.root());
   // Add a directory to the file system.
-  GDataDirectory* directory_entry = new GDataDirectory(root,
+  GDataDirectory* directory_entry = new GDataDirectory(NULL,
                                                        &directory_service);
   directory_entry->set_resource_id("folder:directory_resource_id");
   directory_entry->set_title("directory");
   directory_entry->SetBaseNameFromTitle();
-  GDataFileError error = GDATA_FILE_ERROR_FAILED;
   directory_service.AddEntryToDirectory(
       FilePath(kGDataRootDirectory),
       directory_entry,
-      base::Bind(&test_util::CopyErrorCodeFromFileOperationCallback, &error));
+      base::Bind(&OnAddEntryToDirectory, GDATA_FILE_OK));
   test_util::RunBlockingPoolTask();
-  ASSERT_EQ(GDATA_FILE_OK, error);
 
   // Add a new file to the directory.
   GDataFile* initial_file_entry = new GDataFile(NULL, &directory_service);
@@ -397,9 +399,8 @@
   directory_service.AddEntryToDirectory(
       directory_entry->GetFilePath(),
       initial_file_entry,
-      base::Bind(&test_util::CopyErrorCodeFromFileOperationCallback, &error));
+      base::Bind(&OnAddEntryToDirectory, GDATA_FILE_OK));
   test_util::RunBlockingPoolTask();
-  ASSERT_EQ(GDATA_FILE_OK, error);
 
   ASSERT_EQ(directory_entry, initial_file_entry->parent());