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

Unified Diff: chrome/browser/sync/engine/sync_scheduler.h

Issue 9348036: Trim code from sync's ServerConnectionManager (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Back off some changes Created 8 years, 10 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/engine/sync_scheduler.h
diff --git a/chrome/browser/sync/engine/sync_scheduler.h b/chrome/browser/sync/engine/sync_scheduler.h
index ef8636495121d628deebd0fe21f34ddb453c16f9..72774e74623e0104967f5af9af71954901bd2ad0 100644
--- a/chrome/browser/sync/engine/sync_scheduler.h
+++ b/chrome/browser/sync/engine/sync_scheduler.h
@@ -343,7 +343,7 @@ class SyncScheduler : public sessions::SyncSession::Delegate {
SyncerStep* start,
SyncerStep* end);
- // Used to update |server_connection_ok_|, see below.
+ // Used to update |connection_code_|, see below.
void UpdateServerConnectionManagerStatus(
HttpResponse::ServerConnectionCode code);
@@ -391,9 +391,6 @@ class SyncScheduler : public sessions::SyncSession::Delegate {
// since the nudges could be for different types. Current impl doesn't care.
base::TimeTicks last_sync_session_end_time_;
- // Have we observed a valid server connection?
- bool server_connection_ok_;
-
// The latest connection code we got while trying to connect.
HttpResponse::ServerConnectionCode connection_code_;

Powered by Google App Engine
This is Rietveld 408576698