Remove root resource id aliasing from DriveResourceMetadata.

chrome/browser/chromeos/drive/change_list_loader.h

Index: chrome/browser/chromeos/drive/change_list_loader.h
diff --git a/chrome/browser/chromeos/drive/change_list_loader.h b/chrome/browser/chromeos/drive/change_list_loader.h
index debf7ae073593c063bb6d2e2ede2581e20531029..b5646e85c8616323197a7ab2bdd5094674d6cc6c 100644
--- a/chrome/browser/chromeos/drive/change_list_loader.h
+++ b/chrome/browser/chromeos/drive/change_list_loader.h
@@ -155,6 +155,24 @@ class ChangeListLoader {
   void DoLoadDirectoryFromServer(const DirectoryFetchInfo& directory_fetch_info,
                                  const FileOperationCallback& callback);
 
+  // Part of DoLoadDirectoryFromServer. Called when GetEntryInfoByPath for
+  // mydrive creation is completed.
+  // |callback| must not be null.
+  void DoLoadDirectoryFromServerAfterGetEntryInfoByPath(

[kinaba, 2013/04/10 04:34:44]

How about naming these methods in a way that it explicitly tells this is
handling the special case of reading "/drive"?
And regard them as sub-routines of DoLoadDirectoryFromServer
(i.e., normal case uses LoadFromServer and special case uses the new ones).

+      const DirectoryFetchInfo& directory_fetch_info,
+      const FileOperationCallback& callback,
+      DriveFileError error,
+      scoped_ptr<DriveEntryProto> entry_proto);
+
+  // Part of DoLoadDIrectoryFromServer. Called when GetAboutResource for mydrive

[kinaba, 2013/04/10 04:34:44]

nit: DoLoadD<i>rectory

[hidehiko, 2013/04/10 06:11:19]

Done.

+  // root creation is completed.
+  // |callback must not be null.

[Haruki Sato, 2013/04/10 03:44:35]

|callback| ? missing "|".

[hidehiko, 2013/04/10 06:11:19]

Done.

[hidehiko, 2013/04/10 06:11:19]

Done.

+  void DoLoadDirectoryFromServerAfterGetAboutResource(
+      const DirectoryFetchInfo& directory_fetch_info,
+      const FileOperationCallback& callback,
+      google_apis::GDataErrorCode status,
+      scoped_ptr<google_apis::AboutResource> about_resource);
+
   // Part of DoLoadDirectoryFromServer(). Called after
   // LoadFromServer() is complete.
   void DoLoadDirectoryFromServerAfterLoad(