Added 'starred' to EntryProperty in FileManagerPrivateAPI.

components/drive/resource_metadata_storage.cc

 // Copyright 2013 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 "components/drive/resource_metadata_storage.h"
 
 #include <stddef.h>
 
 #include <map>
 #include <set>
@@ skipping 612 matching lines @@
       } else {
         init_result = DB_INIT_FAILED;
         resource_map_.reset();
       }
     } else {
       LOG(ERROR) << "Failed to create resource map DB: " << status.ToString();
       init_result = LevelDBStatusToDBInitStatus(status);
     }
   }
 
+  // Get 'Starred' property if it hasn't been loaded.
+  if (resource_map_) {

[harukam, 2016/09/29 05:42:18]

if (resource_map_) { ... } is needed to work correctly even in the case such as
ResourceMetadataStorageTest.WrongPath
Is this an appropriate way?

+    bool has_starred = false;
+    ResourceMetadataHeader header;
+    if (GetHeader(&header) == FILE_ERROR_OK)
+      has_starred = header.has_starred();
+
+    if (!has_starred) {
+      FileError error = SetLargestChangestamp(0);
+      if (error == FILE_ERROR_OK && GetHeader(&header) == FILE_ERROR_OK) {
+          header.set_has_starred(true);
+          PutHeader(header);
+      }
+    }
+  }
+
   UMA_HISTOGRAM_ENUMERATION("Drive.MetadataDBInitResult",
                             init_result,
                             DB_INIT_MAX_VALUE);
   return !!resource_map_;
 }
 
 void ResourceMetadataStorage::RecoverCacheInfoFromTrashedResourceMap(
     RecoveredCacheInfoMap* out_info) {
   const base::FilePath trashed_resource_map_path =
       directory_path_.Append(kTrashedResourceMapDBName);
@@ skipping 418 matching lines @@
     RecordCheckValidityFailure(
         CHECK_VALIDITY_FAILURE_CHILD_ENTRY_COUNT_MISMATCH);
     return false;
   }
 
   return true;
 }
 
 }  // namespace internal
 }  // namespace drive