Pass parsed ChangeList to GDataFileSystem.

chrome/browser/chromeos/gdata/drive_api_parser.h

Index: chrome/browser/chromeos/gdata/drive_api_parser.h
diff --git a/chrome/browser/chromeos/gdata/drive_api_parser.h b/chrome/browser/chromeos/gdata/drive_api_parser.h
index 7472677d09dce551cc434dba01a06d0c63dfcb8c..59cf2e7dc4ef80e5c683696b1684a79ff014518f 100644
--- a/chrome/browser/chromeos/gdata/drive_api_parser.h
+++ b/chrome/browser/chromeos/gdata/drive_api_parser.h
@@ -15,6 +15,7 @@
 #include "base/string_piece.h"
 #include "base/time.h"
 #include "googleurl/src/gurl.h"
+#include "chrome/browser/chromeos/gdata/gdata_wapi_parser.h"

[satorux1, 2012/08/10 18:14:34]

This dependency is worrisome. This is for DocumentEntry::EntryKind, right? This
sounds bad, as DocumentEntry will eventually be gone once the migration is done.

We should probably move EntryKind out of DocumentEntry, and put it in a separate
header file like parser_constants.h

[kochi, 2012/08/13 09:08:39]

Once migration is done, EntryKind will not necessary and
parser_constants.h will be empty.  Drive API parser is the
only parser in this directory, drive_api_parser.h is enough
and no reason to have a separate constant definition header.

Even before complete migration to Drive API, once we directly convert to proto,
this isn't necessary.

So I added TODO to remove this dependency.  The same
as the comment for GetKind().

 
 namespace base {
 class Value;
@@ -315,6 +316,8 @@ class FileResource {
   // but outside this file we use "directory" to match HTML5 filesystem API.
   bool IsDirectory() const;
 
+  DocumentEntry::EntryKind GetKind() const;
+
   // Returns file ID.  This is unique in all files in Google Drive.
   const std::string& file_id() const { return file_id_; }