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

Unified Diff: sync/internal_api/syncapi_unittest.cc

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/syncapi_unittest.cc
diff --git a/sync/internal_api/syncapi_unittest.cc b/sync/internal_api/syncapi_unittest.cc
index 991aed6b0a6f82dcc38a483c8f23e35a11e18a7f..aaeb28329f167ad552ca94a541355134911d7b54 100644
--- a/sync/internal_api/syncapi_unittest.cc
+++ b/sync/internal_api/syncapi_unittest.cc
@@ -1264,16 +1264,22 @@ TEST_F(SyncManagerTest, OnNotificationStateChange) {
HandleJsEvent("onNotificationStateChange",
HasDetailsAsDictionary(false_details)));
- sync_manager_.TriggerOnNotificationStateChangeForTest(true);
- sync_manager_.TriggerOnNotificationStateChangeForTest(false);
+ sync_manager_.TriggerOnSyncNotifierStateChangeForTest(
+ sync_notifier::NOTIFICATIONS_ON);
+ sync_manager_.TriggerOnSyncNotifierStateChangeForTest(
+ sync_notifier::NOTIFICATIONS_OFF);
SetJsEventHandler(event_handler.AsWeakHandle());
- sync_manager_.TriggerOnNotificationStateChangeForTest(true);
- sync_manager_.TriggerOnNotificationStateChangeForTest(false);
+ sync_manager_.TriggerOnSyncNotifierStateChangeForTest(
+ sync_notifier::NOTIFICATIONS_ON);
+ sync_manager_.TriggerOnSyncNotifierStateChangeForTest(
+ sync_notifier::NOTIFICATIONS_OFF);
SetJsEventHandler(WeakHandle<JsEventHandler>());
- sync_manager_.TriggerOnNotificationStateChangeForTest(true);
- sync_manager_.TriggerOnNotificationStateChangeForTest(false);
+ sync_manager_.TriggerOnSyncNotifierStateChangeForTest(
+ sync_notifier::NOTIFICATIONS_ON);
+ sync_manager_.TriggerOnSyncNotifierStateChangeForTest(
+ sync_notifier::NOTIFICATIONS_OFF);
// Should trigger the replies.
PumpLoop();

Powered by Google App Engine
This is Rietveld 408576698