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

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

Issue 537883002: [SyncFS] Fix unexpected delay on running sync task (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@peria
Patch Set: Created 6 years, 3 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 side-by-side diff with in-line comments
Download patch
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 320e6f6604a0101caa012ed577ada6b33bcf01d9..29e45f3a9523d7dd85b6ed2ea01b99abed948ffe 100644
--- a/chrome/browser/sync_file_system/drive_backend/sync_engine.cc
+++ b/chrome/browser/sync_file_system/drive_backend/sync_engine.cc
@@ -358,7 +358,6 @@ void SyncEngine::InitializeInternal(
drive_service_->AddObserver(this);
service_state_ = REMOTE_SERVICE_TEMPORARY_UNAVAILABLE;
- SetSyncEnabled(sync_enabled_);
OnNetworkChanged(net::NetworkChangeNotifier::GetConnectionType());
if (drive_service_->HasRefreshToken())
OnReadyToSendRequests();
@@ -665,7 +664,7 @@ void SyncEngine::OnNotificationReceived() {
"Got push notification for Drive"));
}
-void SyncEngine::OnPushNotificationEnabled(bool) {}
+void SyncEngine::OnPushNotificationEnabled(bool enabled) {}
peria 2014/09/04 09:47:48 |enabled| seems not to be used?
tzik 2014/09/04 10:57:07 Commented out. Our style guide (and linter) seems
void SyncEngine::OnReadyToSendRequests() {
has_refresh_token_ = true;

Powered by Google App Engine
This is Rietveld 408576698