drive: Enable recursive fast-fetch.

chrome/browser/chromeos/drive/file_system_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/file_system.h"
 
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 362 matching lines @@
   // After "fast fetch" is done, full resource list is fetched.
   EXPECT_EQ(1, fake_drive_service_->resource_list_load_count());
 
   // "Fast fetch" will fire an OnirectoryChanged event.
   ASSERT_EQ(1u, mock_directory_observer_->changed_directories().size());
   EXPECT_EQ(base::FilePath(FILE_PATH_LITERAL("drive")),
             mock_directory_observer_->changed_directories()[0]);
 }
 
 TEST_F(FileSystemTest, GetExistingFile) {
+  // Simulate the situation that full feed fetching takes very long time,
+  // to test the "fast fetch" feature is properly working.

[satorux1, 2013/09/06 08:32:25]

"fast fetch" -> recursive "fast fetch"

[kinaba, 2013/09/06 10:12:14]

Done.

+  fake_drive_service_->set_never_return_all_resource_list(true);
+
   const base::FilePath kFilePath(FILE_PATH_LITERAL("drive/root/File 1.txt"));

[satorux1, 2013/09/06 08:32:25]

could you make this one level deeper, to make it more obvious that the recursive
fast fetch is working.

[kinaba, 2013/09/06 10:12:14]

Done.

   scoped_ptr<ResourceEntry> entry = GetResourceEntryByPathSync(kFilePath);
   ASSERT_TRUE(entry);
   EXPECT_EQ("file:2_file_resource_id", entry->resource_id());
 
-  EXPECT_EQ(1, fake_drive_service_->about_resource_load_count());
-  EXPECT_EQ(1, fake_drive_service_->resource_list_load_count());
+  // Directory load for "drive" and "drive/root" that loads about_resource,
+  // and one background full resource list loading.
+  EXPECT_EQ(2, fake_drive_service_->about_resource_load_count());
+  EXPECT_EQ(1, fake_drive_service_->blocked_resource_list_load_count());
 }
 
 TEST_F(FileSystemTest, GetExistingDocument) {
   const base::FilePath kFilePath(
       FILE_PATH_LITERAL("drive/root/Document 1 excludeDir-test.gdoc"));
   scoped_ptr<ResourceEntry> entry = GetResourceEntryByPathSync(kFilePath);
   ASSERT_TRUE(entry);
   EXPECT_EQ("document:5_document_resource_id", entry->resource_id());
 }
 
@@ skipping 60 matching lines @@
 TEST_F(FileSystemTest, GetInSubSubdir) {
   const base::FilePath kFilePath(
       FILE_PATH_LITERAL("drive/root/Directory 1/Sub Directory Folder/"
                         "Sub Sub Directory Folder"));
   scoped_ptr<ResourceEntry> entry = GetResourceEntryByPathSync(kFilePath);
   ASSERT_TRUE(entry);
   ASSERT_EQ("folder:sub_sub_directory_folder_id", entry->resource_id());
 }
 
 TEST_F(FileSystemTest, GetOrphanFile) {
+  ASSERT_TRUE(LoadFullResourceList());
+
+  // Entry without parents are placed under "drive/other".
   const base::FilePath kFilePath(
       FILE_PATH_LITERAL("drive/other/Orphan File 1.txt"));
   scoped_ptr<ResourceEntry> entry = GetResourceEntryByPathSync(kFilePath);
   ASSERT_TRUE(entry);
   EXPECT_EQ("file:1_orphanfile_resource_id", entry->resource_id());
 }
 
 TEST_F(FileSystemTest, ReadDirectoryByPath_Root) {
   // ReadDirectoryByPath() should kick off the resource list loading.
   scoped_ptr<ResourceEntryVector> entries(
@@ skipping 387 matching lines @@
       kEmbedOrigin,
       google_apis::test_util::CreateCopyResultCallback(&error, &share_url));
   test_util::RunBlockingPoolTask();
 
   // Verify the error and the share url, which should be empty.
   EXPECT_EQ(FILE_ERROR_FAILED, error);
   EXPECT_TRUE(share_url.is_empty());
 }
 
 }   // namespace drive