diff options
| author | Ben Murdoch <benm@google.com> | 2010-11-18 18:32:45 +0000 |
|---|---|---|
| committer | Ben Murdoch <benm@google.com> | 2010-11-18 18:38:07 +0000 |
| commit | 513209b27ff55e2841eac0e4120199c23acce758 (patch) | |
| tree | aeba30bb08c5f47c57003544e378a377c297eee6 /chrome/browser/sync/engine/syncer_proto_util.cc | |
| parent | 164f7496de0fbee436b385a79ead9e3cb81a50c1 (diff) | |
| download | external_chromium-513209b27ff55e2841eac0e4120199c23acce758.zip external_chromium-513209b27ff55e2841eac0e4120199c23acce758.tar.gz external_chromium-513209b27ff55e2841eac0e4120199c23acce758.tar.bz2 | |
Merge Chromium at r65505: Initial merge by git.
Change-Id: I31d8f1d8cd33caaf7f47ffa7350aef42d5fbdb45
Diffstat (limited to 'chrome/browser/sync/engine/syncer_proto_util.cc')
| -rw-r--r-- | chrome/browser/sync/engine/syncer_proto_util.cc | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/chrome/browser/sync/engine/syncer_proto_util.cc b/chrome/browser/sync/engine/syncer_proto_util.cc index ffc00d8..2603b0a 100644 --- a/chrome/browser/sync/engine/syncer_proto_util.cc +++ b/chrome/browser/sync/engine/syncer_proto_util.cc @@ -195,6 +195,10 @@ bool SyncerProtoUtil::PostClientToServerMessage( } switch (response->error_code()) { + case ClientToServerResponse::UNKNOWN: + LOG(WARNING) << "Sync protocol out-of-date. The server is using a more " + << "recent version."; + return false; case ClientToServerResponse::SUCCESS: LogResponseProfilingData(*response); return true; @@ -203,11 +207,14 @@ bool SyncerProtoUtil::PostClientToServerMessage( session->delegate()->OnSilencedUntil(base::TimeTicks::Now() + base::TimeDelta::FromSeconds(kSyncDelayAfterThrottled)); return false; + case ClientToServerResponse::TRANSIENT_ERROR: + return false; case ClientToServerResponse::USER_NOT_ACTIVATED: case ClientToServerResponse::AUTH_INVALID: case ClientToServerResponse::ACCESS_DENIED: // WARNING: PostAndProcessHeaders contains a hack for this case. LOG(WARNING) << "SyncerProtoUtil: Authentication expired."; + // TODO(sync): Was this meant to be a fall-through? default: NOTREACHED(); return false; |
