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

Side by Side Diff: chrome/browser/sync_file_system/drive_backend/metadata_database_index.cc

Issue 377463002: [SyncFS] Handle dirty tracker IDs in MetadataDatabaseIndexOnDisk (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Work for comments Created 6 years, 5 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 | Annotate | Revision Log
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "chrome/browser/sync_file_system/drive_backend/metadata_database_index. h" 5 #include "chrome/browser/sync_file_system/drive_backend/metadata_database_index. h"
6 6
7 #include "base/metrics/histogram.h" 7 #include "base/metrics/histogram.h"
8 #include "base/strings/string_number_conversions.h" 8 #include "base/strings/string_number_conversions.h"
9 #include "base/strings/string_util.h" 9 #include "base/strings/string_util.h"
10 #include "chrome/browser/sync_file_system/drive_backend/drive_backend_constants. h" 10 #include "chrome/browser/sync_file_system/drive_backend/drive_backend_constants. h"
(...skipping 365 matching lines...) Expand 10 before | Expand all | Expand 10 after
376 return ParentIDAndTitle(kInvalidTrackerID, std::string()); 376 return ParentIDAndTitle(kInvalidTrackerID, std::string());
377 return *multi_backing_file_paths_.begin(); 377 return *multi_backing_file_paths_.begin();
378 } 378 }
379 379
380 int64 MetadataDatabaseIndex::PickDirtyTracker() const { 380 int64 MetadataDatabaseIndex::PickDirtyTracker() const {
381 if (dirty_trackers_.empty()) 381 if (dirty_trackers_.empty())
382 return kInvalidTrackerID; 382 return kInvalidTrackerID;
383 return *dirty_trackers_.begin(); 383 return *dirty_trackers_.begin();
384 } 384 }
385 385
386 void MetadataDatabaseIndex::DemoteDirtyTracker(int64 tracker_id) { 386 void MetadataDatabaseIndex::DemoteDirtyTracker(
387 int64 tracker_id, leveldb::WriteBatch* /*unused_batch*/) {
nhiroki 2014/07/08 04:17:20 "/*unused_batch*/" -> "/* unused_batch */"
peria 2014/07/08 04:58:32 Done.
387 if (dirty_trackers_.erase(tracker_id)) 388 if (dirty_trackers_.erase(tracker_id))
388 demoted_dirty_trackers_.insert(tracker_id); 389 demoted_dirty_trackers_.insert(tracker_id);
389 } 390 }
390 391
391 bool MetadataDatabaseIndex::HasDemotedDirtyTracker() const { 392 bool MetadataDatabaseIndex::HasDemotedDirtyTracker() const {
392 return !demoted_dirty_trackers_.empty(); 393 return !demoted_dirty_trackers_.empty();
393 } 394 }
394 395
395 void MetadataDatabaseIndex::PromoteDemotedDirtyTrackers() { 396 void MetadataDatabaseIndex::PromoteDemotedDirtyTrackers(
397 leveldb::WriteBatch* /*unused_batch*/) {
nhiroki 2014/07/08 04:17:20 ditto.
peria 2014/07/08 04:58:32 Done.
396 dirty_trackers_.insert(demoted_dirty_trackers_.begin(), 398 dirty_trackers_.insert(demoted_dirty_trackers_.begin(),
397 demoted_dirty_trackers_.end()); 399 demoted_dirty_trackers_.end());
398 demoted_dirty_trackers_.clear(); 400 demoted_dirty_trackers_.clear();
399 } 401 }
400 402
401 size_t MetadataDatabaseIndex::CountDirtyTracker() const { 403 size_t MetadataDatabaseIndex::CountDirtyTracker() const {
402 return dirty_trackers_.size() + demoted_dirty_trackers_.size(); 404 return dirty_trackers_.size() + demoted_dirty_trackers_.size();
403 } 405 }
404 406
405 size_t MetadataDatabaseIndex::CountFileMetadata() const { 407 size_t MetadataDatabaseIndex::CountFileMetadata() const {
(...skipping 282 matching lines...) Expand 10 before | Expand all | Expand 10 after
688 690
689 DVLOG(3) << " Remove from dirty_trackers_: " << tracker_id; 691 DVLOG(3) << " Remove from dirty_trackers_: " << tracker_id;
690 dirty_trackers_.erase(tracker_id); 692 dirty_trackers_.erase(tracker_id);
691 693
692 demoted_dirty_trackers_.erase(tracker_id); 694 demoted_dirty_trackers_.erase(tracker_id);
693 } 695 }
694 } 696 }
695 697
696 } // namespace drive_backend 698 } // namespace drive_backend
697 } // namespace sync_file_system 699 } // namespace sync_file_system
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698