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

Unified Diff: sync/internal_api/sync_manager.h

Issue 10545170: [Sync] Propagate XMPP auth errors to SyncNotifierObservers (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix compile error Created 8 years, 6 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: sync/internal_api/sync_manager.h
diff --git a/sync/internal_api/sync_manager.h b/sync/internal_api/sync_manager.h
index 80cda3f53c37f3b33bdbd0b20c0798a8ef3d884f..2179f6b219c55f52bf314148d70070b80fa8e4f1 100644
--- a/sync/internal_api/sync_manager.h
+++ b/sync/internal_api/sync_manager.h
@@ -19,6 +19,7 @@
#include "sync/internal_api/configure_reason.h"
#include "sync/internal_api/public/engine/model_safe_worker.h"
#include "sync/internal_api/public/syncable/model_type.h"
+#include "sync/notifier/sync_notifier_state.h"
#include "sync/protocol/sync_protocol_error.h"
#include "sync/util/report_unrecoverable_error_function.h"
#include "sync/util/unrecoverable_error_handler.h"
@@ -95,6 +96,7 @@ class SyncManager {
Status();
~Status();
+ // TODO(akalin): Change this to a SyncNotifierState.
bool notifications_enabled; // True only if subscribed for notifications.
// Notifications counters updated by the actions in synapi.
@@ -598,8 +600,8 @@ class SyncManager {
// Functions used for testing.
- void TriggerOnNotificationStateChangeForTest(
- bool notifications_enabled);
+ void TriggerOnSyncNotifierStateChangeForTest(
+ sync_notifier::SyncNotifierState sync_notifier_state);
void TriggerOnIncomingNotificationForTest(
syncable::ModelTypeSet model_types);

Powered by Google App Engine
This is Rietveld 408576698