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

Unified Diff: chrome/browser/sync/glue/sync_backend_host.h

Issue 7861013: Fix the false-positive detection of commit errors (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Another attempt at detecting errors Created 9 years, 2 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
« no previous file with comments | « chrome/browser/sync/engine/syncer_proto_util.cc ('k') | chrome/browser/sync/glue/sync_backend_host.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/glue/sync_backend_host.h
diff --git a/chrome/browser/sync/glue/sync_backend_host.h b/chrome/browser/sync/glue/sync_backend_host.h
index f0d17be21ed0c8329be2277809c5f460dc92a8b2..833af1d944f5d59a4cbd498a355fdd14203498b0 100644
--- a/chrome/browser/sync/glue/sync_backend_host.h
+++ b/chrome/browser/sync/glue/sync_backend_host.h
@@ -100,7 +100,7 @@ class SyncFrontend {
// Called when the sync cycle returns there is an user actionable error.
virtual void OnActionableError(
- const browser_sync::SyncProtocolError& error) = 0;
+ const browser_sync::SyncOperationResult& result) = 0;
protected:
// Don't delete through SyncFrontend interface.
@@ -276,7 +276,7 @@ class SyncBackendHost {
virtual void OnEncryptionComplete(
const syncable::ModelTypeSet& encrypted_types);
virtual void OnActionableError(
- const browser_sync::SyncProtocolError& sync_error);
+ const browser_sync::SyncOperationResult& sync_error);
struct DoInitializeOptions {
DoInitializeOptions(
@@ -407,7 +407,7 @@ class SyncBackendHost {
// Let the front end handle the actionable error event.
void HandleActionableErrorEventOnFrontendLoop(
- const browser_sync::SyncProtocolError& sync_error);
+ const browser_sync::SyncOperationResult& sync_error);
// Dispatched to from OnAuthError to handle updating frontend UI
// components.
« no previous file with comments | « chrome/browser/sync/engine/syncer_proto_util.cc ('k') | chrome/browser/sync/glue/sync_backend_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698