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

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

Issue 443793003: [SyncFS] Clear dirty flag on changelist application phase (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 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
« no previous file with comments | « chrome/browser/sync_file_system/drive_backend/metadata_database.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 "chrome/browser/sync_file_system/drive_backend/metadata_database.h" 5 #include "chrome/browser/sync_file_system/drive_backend/metadata_database.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <stack> 8 #include <stack>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 1357 matching lines...) Expand 10 before | Expand all | Expand 10 after
1368 std::swap(*trackers_out, trackers); 1368 std::swap(*trackers_out, trackers);
1369 return true; 1369 return true;
1370 } 1370 }
1371 1371
1372 void MetadataDatabase::GetRegisteredAppIDs(std::vector<std::string>* app_ids) { 1372 void MetadataDatabase::GetRegisteredAppIDs(std::vector<std::string>* app_ids) {
1373 DCHECK(worker_sequence_checker_.CalledOnValidSequencedThread()); 1373 DCHECK(worker_sequence_checker_.CalledOnValidSequencedThread());
1374 DCHECK(app_ids); 1374 DCHECK(app_ids);
1375 *app_ids = index_->GetRegisteredAppIDs(); 1375 *app_ids = index_->GetRegisteredAppIDs();
1376 } 1376 }
1377 1377
1378 void MetadataDatabase::SweepDirtyTrackers(
1379 const std::vector<std::string>& file_ids,
1380 const SyncStatusCallback& callback) {
1381 DCHECK(worker_sequence_checker_.CalledOnValidSequencedThread());
1382
1383 std::set<int64> tracker_ids;
1384 for (size_t i = 0; i < file_ids.size(); ++i) {
1385 TrackerIDSet trackers_for_file_id =
1386 index_->GetFileTrackerIDsByFileID(file_ids[i]);
1387 for (TrackerIDSet::iterator itr = trackers_for_file_id.begin();
1388 itr != trackers_for_file_id.end(); ++itr)
1389 tracker_ids.insert(*itr);
1390 }
1391
1392 for (std::set<int64>::iterator itr = tracker_ids.begin();
1393 itr != tracker_ids.end(); ++itr) {
1394 scoped_ptr<FileTracker> tracker(new FileTracker);
1395 if (!index_->GetFileTracker(*itr, tracker.get()) ||
1396 !CanClearDirty(*tracker))
1397 continue;
1398 tracker->set_dirty(false);
1399 index_->StoreFileTracker(tracker.Pass());
1400 }
1401
1402 WriteToDatabase(callback);
1403 }
1404
1378 MetadataDatabase::MetadataDatabase( 1405 MetadataDatabase::MetadataDatabase(
1379 base::SequencedTaskRunner* worker_task_runner, 1406 base::SequencedTaskRunner* worker_task_runner,
1380 const base::FilePath& database_path, 1407 const base::FilePath& database_path,
1381 leveldb::Env* env_override) 1408 leveldb::Env* env_override)
1382 : worker_task_runner_(worker_task_runner), 1409 : worker_task_runner_(worker_task_runner),
1383 database_path_(database_path), 1410 database_path_(database_path),
1384 env_override_(env_override), 1411 env_override_(env_override),
1385 largest_known_change_id_(0), 1412 largest_known_change_id_(0),
1386 weak_ptr_factory_(this) { 1413 weak_ptr_factory_(this) {
1387 DCHECK(worker_task_runner); 1414 DCHECK(worker_task_runner);
(...skipping 506 matching lines...) Expand 10 before | Expand all | Expand 10 after
1894 *app_root_metadata); 1921 *app_root_metadata);
1895 1922
1896 index_->StoreFileMetadata(app_root_metadata.Pass()); 1923 index_->StoreFileMetadata(app_root_metadata.Pass());
1897 index_->StoreFileTracker(app_root_tracker.Pass()); 1924 index_->StoreFileTracker(app_root_tracker.Pass());
1898 } 1925 }
1899 1926
1900 void MetadataDatabase::DetachFromSequence() { 1927 void MetadataDatabase::DetachFromSequence() {
1901 worker_sequence_checker_.DetachFromSequence(); 1928 worker_sequence_checker_.DetachFromSequence();
1902 } 1929 }
1903 1930
1931 bool MetadataDatabase::CanClearDirty(const FileTracker& tracker) {
1932 DCHECK(worker_sequence_checker_.CalledOnValidSequencedThread());
1933
1934 FileMetadata metadata;
1935 if (!index_->GetFileMetadata(tracker.file_id(), &metadata) ||
1936 !tracker.active() || !tracker.dirty() ||
1937 !tracker.has_synced_details() ||
1938 tracker.needs_folder_listing())
1939 return false;
1940
1941 const FileDetails& remote_details = metadata.details();
1942 const FileDetails& synced_details = tracker.synced_details();
1943 if (remote_details.title() != synced_details.title() ||
1944 remote_details.md5() != synced_details.md5())
1945 return false;
1946
1947 std::set<std::string> parents;
1948 for (int i = 0; i < remote_details.parent_folder_ids_size(); ++i)
1949 parents.insert(remote_details.parent_folder_ids(i));
1950
1951 for (int i = 0; i < synced_details.parent_folder_ids_size(); ++i)
1952 if (parents.erase(synced_details.parent_folder_ids(i)) != 1)
1953 return false;
1954
1955 if (!parents.empty())
1956 return false;
1957
1958 return true;
1959 }
1960
1904 } // namespace drive_backend 1961 } // namespace drive_backend
1905 } // namespace sync_file_system 1962 } // namespace sync_file_system
OLDNEW
« no previous file with comments | « chrome/browser/sync_file_system/drive_backend/metadata_database.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698