Keep sync tasks alive as long as it's not finished

chrome/browser/chromeos/drive/sync_client_unittest.cc

 // Copyright (c) 2012 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 "chrome/browser/chromeos/drive/sync_client.h"
 
 #include "base/file_util.h"
 #include "base/files/file_path.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 459 matching lines @@
 
   ResourceEntry entry2;
   entry2.set_parent_local_id(local_id1);
   entry2.set_title(kPath2.BaseName().AsUTF8Unsafe());
   entry2.mutable_file_info()->set_is_directory(true);
   entry2.set_metadata_edit_state(ResourceEntry::DIRTY);
   std::string local_id2;
   EXPECT_EQ(FILE_ERROR_OK, metadata_->AddEntry(entry2, &local_id2));
 
   // Start syncing the child first.
   sync_client_->AddUpdateTask(ClientContext(USER_INITIATED), local_id2);

[hashimoto, 2014/07/16 11:05:16]

In the new code, RunUntilIdle() is not called here because AddDependentTask()
requires the parent's task to be present.

In the production environment, AddUpdateTask should be called in parent-to-child
order.
We are reversing the order here just for convenience.

-  base::RunLoop().RunUntilIdle();
   // Start syncing the parent later.
   sync_client_->AddUpdateTask(ClientContext(USER_INITIATED), local_id1);
   base::RunLoop().RunUntilIdle();
 
   // Both entries are synced.
   EXPECT_EQ(FILE_ERROR_OK, metadata_->GetResourceEntryById(local_id1, &entry1));
   EXPECT_EQ(ResourceEntry::CLEAN, entry1.metadata_edit_state());
   EXPECT_EQ(FILE_ERROR_OK, metadata_->GetResourceEntryById(local_id2, &entry2));
   EXPECT_EQ(ResourceEntry::CLEAN, entry2.metadata_edit_state());
 }
 
 }  // namespace internal
 }  // namespace drive