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

Unified Diff: chrome/browser/sync/sessions/sync_session.h

Issue 7621085: Server directed error handling backend code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: For review. Created 9 years, 4 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/sessions/sync_session.h
diff --git a/chrome/browser/sync/sessions/sync_session.h b/chrome/browser/sync/sessions/sync_session.h
index 9520f9dcfe65515245cde396d429a24049bb4eb4..68abb76150231d4d9785a41cc2b98bf9776e5187 100644
--- a/chrome/browser/sync/sessions/sync_session.h
+++ b/chrome/browser/sync/sessions/sync_session.h
@@ -79,8 +79,16 @@ class SyncSession {
// the Syncer detects that the backend store has fundamentally changed or
// is a different instance altogether (e.g. swapping from a test instance
// to production, or a global stop syncing operation has wiped the store).
+ // TODO(lipalani) : Replace this function with the one below. This function
+ // stops the current sync cycle and purges the client. In the new model
+ // the former would be done by the |SyncProtocolError| and
+ // the latter(which is an action) would be done in ProfileSyncService
+ // along with the rest of the actions.
virtual void OnShouldStopSyncingPermanently() = 0;
+ // Called for the syncer to respond to the error sent by the server.
+ virtual void OnSyncProtocolError(sessions::SyncSession* session) = 0;
+
protected:
virtual ~Delegate() {}
};

Powered by Google App Engine
This is Rietveld 408576698