Added 'starred' to EntryProperty in FileManagerPrivateAPI.

components/drive/resource_metadata_storage_unittest.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 <stdint.h>
 
 #include <algorithm>
@@ skipping 25 matching lines @@
   }
 
   // Overwrites |storage_|'s version.
   void SetDBVersion(int version) {
     ResourceMetadataHeader header;
     ASSERT_EQ(FILE_ERROR_OK, storage_->GetHeader(&header));
     header.set_version(version);
     EXPECT_EQ(FILE_ERROR_OK, storage_->PutHeader(header));
   }
 
+  // Overwrites |storage_|'s starred_property_initialized.
+  void SetStarredPropertyInitialized(bool value) {
+    ResourceMetadataHeader header;
+    ASSERT_EQ(FILE_ERROR_OK, storage_->GetHeader(&header));
+    header.set_starred_property_initialized(value);
+    EXPECT_EQ(FILE_ERROR_OK, storage_->PutHeader(header));
+  }
+
+  // Get |storage_|'s starred_property_initialized.
+  void GetStarredPropertyInitialized(bool* value) {

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

How about returning the value?
  bool GetStarredPropertyInitialized()

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

I chose passe-by-pointer like as GetLargestChangestamp(int64_t* val), but
passing-by-value seems to be simpler in this case. Acknowledged.

+    ResourceMetadataHeader header;
+    EXPECT_EQ(FILE_ERROR_OK, storage_->GetHeader(&header));
+    *value = header.starred_property_initialized();
+  }
+
   bool CheckValidity() {
     return storage_->CheckValidity();
   }
 
   leveldb::DB* resource_map() { return storage_->resource_map_.get(); }
 
   // Puts a child entry.
   void PutChild(const std::string& parent_id,
                 const std::string& child_base_name,
                 const std::string& child_id) {
@@ skipping 571 matching lines @@
   RemoveChild(key2, name3);
   EXPECT_FALSE(CheckValidity());
   EXPECT_EQ(FILE_ERROR_OK, storage_->RemoveEntry(key3));
   EXPECT_TRUE(CheckValidity());
 
   // Remove key1.
   EXPECT_EQ(FILE_ERROR_OK, storage_->RemoveEntry(key1));
   EXPECT_TRUE(CheckValidity());
 }
 
+TEST_F(ResourceMetadataStorageTest, ChangeStarredPropertyInitialized) {
+  // Suppose 'Starred' property has not loaded.
+  bool starred_property_initialized = false;
+  SetStarredPropertyInitialized(starred_property_initialized);
+
+  // Close DB and reopen.
+  storage_.reset(new ResourceMetadataStorage(
+      temp_dir_.GetPath(), base::ThreadTaskRunnerHandle::Get().get()));
+  ASSERT_TRUE(storage_->Initialize());
+
+  GetStarredPropertyInitialized(&starred_property_initialized);
+  EXPECT_TRUE(starred_property_initialized);

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

Please check that the largest changestamp becomes 0.

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

Done.

+}
+
 }  // namespace internal
 }  // namespace drive