drive: Pass through AboutResouce from server among callbacks in ChangeListLoader.

chrome/browser/chromeos/drive/drive_test_util.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/drive_test_util.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 200 matching lines @@
     return false;
 
   scoped_ptr<google_apis::ResourceList> document_feed(
       google_apis::ResourceList::ExtractAndParse(*document));
   if (!document_feed.get())
     return false;
 
   ScopedVector<google_apis::ResourceList> feed_list;
   feed_list.push_back(document_feed.release());
 
+  scoped_ptr<google_apis::AboutResource> about_resource(
+      new google_apis::AboutResource);
+  about_resource->set_largest_change_id(root_feed_changestamp);
+  about_resource->set_root_folder_id("fake_root");

[hidehiko, 2013/03/18 06:22:27]

Let's avoid hard-coded root id of the fake server.
Passing resource_id or AboutResource instance should be better.

[Haruki Sato, 2013/03/18 07:06:41]

Done. Thanks. This is what you meant, right?

+
   change_list_loader->UpdateFromFeed(
+      about_resource.Pass(),
       feed_list,
       is_delta_feed,
-      root_feed_changestamp,
       base::Bind(&base::DoNothing));
   // ChangeListLoader::UpdateFromFeed is asynchronous, so wait for it to finish.
   google_apis::test_util::RunBlockingPoolTask();
 
   return true;
 }
 
 void DeleteDriveCache(DriveCache* drive_cache) {
   DCHECK(drive_cache);
   drive_cache->Destroy();
   // The cache destruction requires to post a task to the blocking pool.
   google_apis::test_util::RunBlockingPoolTask();
 }
 
 }  // namespace test_util
 }  // namespace drive