Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(70)

Side by Side Diff: components/drive/resource_metadata_storage.cc

Issue 2346873004: Added 'starred' to EntryProperty in FileManagerPrivateAPI. (Closed)
Patch Set: Added error check. Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/drive/resource_metadata_storage.h" 5 #include "components/drive/resource_metadata_storage.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <map> 9 #include <map>
10 #include <set> 10 #include <set>
(...skipping 612 matching lines...) Expand 10 before | Expand all | Expand 10 after
623 } else { 623 } else {
624 init_result = DB_INIT_FAILED; 624 init_result = DB_INIT_FAILED;
625 resource_map_.reset(); 625 resource_map_.reset();
626 } 626 }
627 } else { 627 } else {
628 LOG(ERROR) << "Failed to create resource map DB: " << status.ToString(); 628 LOG(ERROR) << "Failed to create resource map DB: " << status.ToString();
629 init_result = LevelDBStatusToDBInitStatus(status); 629 init_result = LevelDBStatusToDBInitStatus(status);
630 } 630 }
631 } 631 }
632 632
633 // Update local resouces if 'starred' property has not been initialized.
634 if (resource_map_) {
635 ResourceMetadataHeader header;
636 if (GetHeader(&header) != FILE_ERROR_OK)
637 return false;
638
639 if (!header.starred_property_initialized()) {
640 header.set_largest_changestamp(0);
hashimoto 2016/09/29 09:46:06 Please add a comment to describe how it works (i.e
harukam 2016/09/29 11:08:22 Done.
641 header.set_starred_property_initialized(true);
642 FileError error = PutHeader(header);
643
644 if (error != FILE_ERROR_OK)
645 return false;
646 }
647 }
648
633 UMA_HISTOGRAM_ENUMERATION("Drive.MetadataDBInitResult", 649 UMA_HISTOGRAM_ENUMERATION("Drive.MetadataDBInitResult",
634 init_result, 650 init_result,
635 DB_INIT_MAX_VALUE); 651 DB_INIT_MAX_VALUE);
636 return !!resource_map_; 652 return !!resource_map_;
637 } 653 }
638 654
639 void ResourceMetadataStorage::RecoverCacheInfoFromTrashedResourceMap( 655 void ResourceMetadataStorage::RecoverCacheInfoFromTrashedResourceMap(
640 RecoveredCacheInfoMap* out_info) { 656 RecoveredCacheInfoMap* out_info) {
641 const base::FilePath trashed_resource_map_path = 657 const base::FilePath trashed_resource_map_path =
642 directory_path_.Append(kTrashedResourceMapDBName); 658 directory_path_.Append(kTrashedResourceMapDBName);
(...skipping 418 matching lines...) Expand 10 before | Expand all | Expand 10 after
1061 RecordCheckValidityFailure( 1077 RecordCheckValidityFailure(
1062 CHECK_VALIDITY_FAILURE_CHILD_ENTRY_COUNT_MISMATCH); 1078 CHECK_VALIDITY_FAILURE_CHILD_ENTRY_COUNT_MISMATCH);
1063 return false; 1079 return false;
1064 } 1080 }
1065 1081
1066 return true; 1082 return true;
1067 } 1083 }
1068 1084
1069 } // namespace internal 1085 } // namespace internal
1070 } // namespace drive 1086 } // namespace drive
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698