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

Unified Diff: chrome/browser/sync_file_system/drive_backend/sync_engine.cc

Issue 98633003: SyncFS v2: Add modest logging for ApplyLocalChange, ApplyRemoteChange (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 years 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/sync_file_system/local/local_file_sync_service.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync_file_system/drive_backend/sync_engine.cc
diff --git a/chrome/browser/sync_file_system/drive_backend/sync_engine.cc b/chrome/browser/sync_file_system/drive_backend/sync_engine.cc
index 74c6138783a5ef20cedb638406fec0421c3bbc27..f18fdc222071d2fd65ee8804f170f22500ed28c1 100644
--- a/chrome/browser/sync_file_system/drive_backend/sync_engine.cc
+++ b/chrome/browser/sync_file_system/drive_backend/sync_engine.cc
@@ -284,9 +284,13 @@ void SyncEngine::ApplyLocalChange(
const SyncFileMetadata& /* unused */,
const fileapi::FileSystemURL& url,
const SyncStatusCallback& callback) {
+ util::Log(logging::LOG_INFO, FROM_HERE,
+ "[Local->Remote] ApplyLocalChange: %s on %s",
+ local_change.DebugString().c_str(),
+ url.DebugString().c_str());
+
LocalToRemoteSyncer* syncer = new LocalToRemoteSyncer(
this, local_change, local_path, url);
-
task_manager_->ScheduleSyncTask(
scoped_ptr<SyncTask>(syncer),
base::Bind(&SyncEngine::DidApplyLocalChange,
@@ -434,6 +438,10 @@ void SyncEngine::DidProcessRemoteChange(RemoteToLocalSyncer* syncer,
void SyncEngine::DidApplyLocalChange(LocalToRemoteSyncer* syncer,
const SyncStatusCallback& callback,
SyncStatusCode status) {
+ util::Log(logging::LOG_INFO, FROM_HERE,
+ "[Local->Remote] ApplyLocalChange finished --> %s",
+ SyncStatusCodeToString(status));
+
if ((status == SYNC_STATUS_OK || status == SYNC_STATUS_RETRY) &&
syncer->url().is_valid() &&
syncer->sync_action() != SYNC_ACTION_NONE) {
« no previous file with comments | « no previous file | chrome/browser/sync_file_system/local/local_file_sync_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698