Add missing fields in Drive V2 API parser.

chrome/browser/chromeos/gdata/drive_api_parser.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/gdata/drive_api_parser.h"
 
 #include <algorithm>
 
 #include "base/basictypes.h"
 #include "base/file_path.h"
@@ skipping 20 matching lines @@
 }
 
 // Drive v2 API JSON names.
 
 // Common
 const char kKind[] = "kind";
 const char kId[] = "id";
 const char kETag[] = "etag";
 const char kItems[] = "items";
 const char kLargestChangeId[] = "largestChangeId";
+const char kSelfLink[] = "selfLink";
 
 // About Resource:
 const char kAboutKind[] = "drive#about";
 const char kRootFolderId[] = "rootFolderId";
 const char kQuotaBytesTotal[] = "quotaBytesTotal";
 const char kQuotaBytesUsed[] = "quotaBytesUsed";
 
 // App Icon:
 const char kCategory[] = "category";
 const char kSize[] = "size";
@@ skipping 12 matching lines @@
 const char kSecondaryMimeTypes[] = "secondaryMimeTypes";
 const char kPrimaryFileExtensions[] = "primaryFileExtensions";
 const char kSecondaryFileExtensions[] = "secondaryFileExtensions";
 const char kIcons[] = "icons";
 
 // Apps List:
 const char kAppListKind[] = "drive#appList";
 
 // Parent Resource:
 const char kParentReferenceKind[] = "drive#parentReference";
+const char kParentLink[] = "parentLink";
 const char kIsRoot[] = "isRoot";
 
 // File Resource:
 const char kFileKind[] = "drive#file";
 const char kMimeType[] = "mimeType";
 const char kTitle[] = "title";
+const char kCreatedDate[] = "createdDate";
 const char kModifiedByMeDate[] = "modifiedByMeDate";
 const char kParents[] = "parents";
 const char kDownloadUrl[] = "downloadUrl";
 const char kFileExtension[] = "fileExtension";
 const char kMd5Checksum[] = "md5Checksum";
 const char kFileSize[] = "fileSize";
+const char kAlternateLink[] = "alternateLink";

[hashimoto, 2012/08/07 04:19:02]

Is there a policy to order these values?
If so, please add a comment.
If not, please order them in a reasonable manner.

[kochi, 2012/08/07 05:24:37]

Basically this order follows the order in Drive V2 API
web document.

Added a comment about the order and links for each structure.

+const char kEmbedLink[] = "embedLink";
+const char kThumbnailLink[] = "thumbnailLink";
+const char kWebContentLink[] = "webContentLink";
+const char kExplicitlyTrashed[] = "explicitlyTrashed";

[hashimoto, 2012/08/07 04:19:02]

What is this for?

[kochi, 2012/08/07 05:24:37]

Removed.

 
 const char kDriveFolderMimeType[] = "application/vnd.google-apps.folder";
 
 // Files List:
 const char kFileListKind[] = "drive#fileList";
 const char kNextPageToken[] = "nextPageToken";
 const char kNextLink[] = "nextLink";
 
 // Change Resource:
 const char kChangeKind[] = "drive#change";
@@ skipping 217 matching lines @@
 // ParentReference implementation
 
 ParentReference::ParentReference() : is_root_(false) {}
 
 ParentReference::~ParentReference() {}
 
 // static
 void ParentReference::RegisterJSONConverter(
     base::JSONValueConverter<ParentReference>* converter) {
   converter->RegisterStringField(kId, &ParentReference::file_id_);
+  converter->RegisterCustomField<GURL>(kParentLink,
+                                       &ParentReference::parent_link_,
+                                       GetGURLFromString);
   converter->RegisterBoolField(kIsRoot, &ParentReference::is_root_);
 }
 
 // static
 scoped_ptr<ParentReference>
 ParentReference::CreateFrom(const base::Value& value) {
   scoped_ptr<ParentReference> reference(new ParentReference());
   if (!IsResourceKindExpected(value, kParentReferenceKind) ||
       !reference->Parse(value)) {
     LOG(ERROR) << "Unable to create: Invalid ParentRefernce JSON!";
@@ skipping 19 matching lines @@
 FileResource::~FileResource() {}
 
 // static
 void FileResource::RegisterJSONConverter(
     base::JSONValueConverter<FileResource>* converter) {
   converter->RegisterStringField(kId, &FileResource::file_id_);
   converter->RegisterStringField(kETag, &FileResource::etag_);
   converter->RegisterStringField(kMimeType, &FileResource::mime_type_);
   converter->RegisterStringField(kTitle, &FileResource::title_);
   converter->RegisterCustomField<base::Time>(
+      kCreatedDate,
+      &FileResource::created_date_,
+      &gdata::util::GetTimeFromString);
+  converter->RegisterCustomField<base::Time>(
       kModifiedByMeDate,
       &FileResource::modified_by_me_date_,
       &gdata::util::GetTimeFromString);
   converter->RegisterRepeatedMessage<ParentReference>(kParents,
                                                       &FileResource::parents_);
-  converter->RegisterCustomField<GURL>(kDownloadUrl,
-                                       &FileResource::download_url_,
-                                       GetGURLFromString);
   converter->RegisterStringField(kFileExtension,
                                  &FileResource::file_extension_);
   converter->RegisterStringField(kMd5Checksum, &FileResource::md5_checksum_);
   converter->RegisterCustomField<int64>(kFileSize,
                                         &FileResource::file_size_,
                                         &base::StringToInt64);
+  converter->RegisterCustomField<GURL>(kDownloadUrl,
+                                       &FileResource::download_url_,
+                                       GetGURLFromString);
+  converter->RegisterCustomField<GURL>(kSelfLink,
+                                       &FileResource::self_link_,
+                                       GetGURLFromString);
+  converter->RegisterCustomField<GURL>(kAlternateLink,
+                                       &FileResource::alternate_link_,
+                                       GetGURLFromString);
+  converter->RegisterCustomField<GURL>(kEmbedLink,
+                                       &FileResource::embed_link_,
+                                       GetGURLFromString);
+  converter->RegisterCustomField<GURL>(kThumbnailLink,
+                                       &FileResource::thumbnail_link_,
+                                       GetGURLFromString);
+  converter->RegisterCustomField<GURL>(kWebContentLink,
+                                       &FileResource::web_content_link_,
+                                       GetGURLFromString);
 }
 
 // static
 scoped_ptr<FileResource> FileResource::CreateFrom(const base::Value& value) {
   scoped_ptr<FileResource> resource(new FileResource());
   if (!IsResourceKindExpected(value, kFileKind) || !resource->Parse(value)) {
     LOG(ERROR) << "Unable to create: Invalid FileResource JSON!";
     return scoped_ptr<FileResource>(NULL);
   }
   return resource.Pass();
@@ skipping 124 matching lines @@
 bool ChangeList::Parse(const base::Value& value) {
   base::JSONValueConverter<ChangeList> converter;
   if (!converter.Convert(value, this)) {
     LOG(ERROR) << "Unable to parse: Invalid ChangeList";
     return false;
   }
   return true;
 }
 
 }  // namespace gdata