Added 'starred' to EntryProperty in FileManagerPrivateAPI.

components/drive/resource_metadata_storage.cc

Index: components/drive/resource_metadata_storage.cc
diff --git a/components/drive/resource_metadata_storage.cc b/components/drive/resource_metadata_storage.cc
index 8a0eb35bef7530689e1f077f321e57c3c44ffc43..e263badce9c4d084d5f80384a350f6b6af22716e 100644
--- a/components/drive/resource_metadata_storage.cc
+++ b/components/drive/resource_metadata_storage.cc
@@ -630,6 +630,22 @@ bool ResourceMetadataStorage::Initialize() {
     }
   }
 
+  // Update local resouces if 'starred' property has not been initialized.
+  if (resource_map_) {
+    bool update_needed = false;
+    ResourceMetadataHeader header;
+    if (GetHeader(&header) == FILE_ERROR_OK)

[hashimoto, 2016/09/29 08:43:43]

GetHeader() shouldn't fail here.
If it fails, it's better to return false to abort.

[harukam, 2016/09/29 09:39:45]

Acknowledged.

+      update_needed = !header.starred_property_initialized();
+
+    if (update_needed) {
+      FileError error = SetLargestChangestamp(0);

[hashimoto, 2016/09/29 08:43:43]

Instead of SetLargestChangestamp() & PutHeader(), you can do the same thing with
a single PutHeader() (SetLargestChangestamp is just a wrapper of PutHeader).

header.largest_changestamp(0);
header.set_starred_proeprty_initialized(true);
error = PutHeader(header);

[harukam, 2016/09/29 09:39:46]

Thanks.

+      if (error == FILE_ERROR_OK && GetHeader(&header) == FILE_ERROR_OK) {
+        header.set_starred_property_initialized(true);
+        PutHeader(header);

[hashimoto, 2016/09/29 08:43:43]

Please check for an error here.

[harukam, 2016/09/29 09:39:46]

Done.

+      }
+    }
+  }
+
   UMA_HISTOGRAM_ENUMERATION("Drive.MetadataDBInitResult",
                             init_result,
                             DB_INIT_MAX_VALUE);