summaryrefslogtreecommitdiffstats
path: root/chrome/browser/sync/profile_sync_service_unittest.cc
diff options
context:
space:
mode:
Diffstat (limited to 'chrome/browser/sync/profile_sync_service_unittest.cc')
-rw-r--r--chrome/browser/sync/profile_sync_service_unittest.cc19
1 files changed, 11 insertions, 8 deletions
diff --git a/chrome/browser/sync/profile_sync_service_unittest.cc b/chrome/browser/sync/profile_sync_service_unittest.cc
index 80ac89a..c7a335b 100644
--- a/chrome/browser/sync/profile_sync_service_unittest.cc
+++ b/chrome/browser/sync/profile_sync_service_unittest.cc
@@ -10,6 +10,7 @@
#include "chrome/browser/sync/glue/bookmark_data_type_controller.h"
#include "chrome/browser/sync/glue/data_type_controller.h"
#include "chrome/browser/sync/js_arg_list.h"
+#include "chrome/browser/sync/js_event_details.h"
#include "chrome/browser/sync/js_test_util.h"
#include "chrome/browser/sync/profile_sync_factory_mock.h"
#include "chrome/browser/sync/test_profile_sync_service.h"
@@ -171,8 +172,8 @@ TEST_F(ProfileSyncServiceTest,
StrictMock<MockJsEventHandler> event_handler;
EXPECT_CALL(event_handler,
- HandleJsEvent("onSyncServiceStateChanged",
- HasArgs(JsArgList()))).Times(AtLeast(3));
+ HandleJsEvent("onServiceStateChanged",
+ HasDetails(JsEventDetails()))).Times(AtLeast(3));
// For some reason, these events may or may not fire.
//
// TODO(akalin): Figure out exactly why there's non-determinism
@@ -181,7 +182,7 @@ TEST_F(ProfileSyncServiceTest,
.Times(AtMost(1));
EXPECT_CALL(event_handler, HandleJsEvent("onChangesComplete", _))
.Times(AtMost(1));
- EXPECT_CALL(event_handler, HandleJsEvent("onSyncNotificationStateChange", _))
+ EXPECT_CALL(event_handler, HandleJsEvent("onNotificationStateChange", _))
.Times(AtMost(1));
EXPECT_EQ(NULL, service_->GetBackendForTest());
@@ -215,7 +216,7 @@ TEST_F(ProfileSyncServiceTest, JsFrontendProcessMessageBasic) {
.Times(AtMost(1));
EXPECT_CALL(event_handler, HandleJsEvent("onChangesComplete", _))
.Times(AtMost(1));
- EXPECT_CALL(event_handler, HandleJsEvent("onSyncNotificationStateChange", _))
+ EXPECT_CALL(event_handler, HandleJsEvent("onNotificationStateChange", _))
.Times(AtMost(1));
ListValue arg_list1;
@@ -274,7 +275,7 @@ TEST_F(ProfileSyncServiceTest,
.Times(AtMost(1));
EXPECT_CALL(event_handler, HandleJsEvent("onChangesComplete", _))
.Times(AtMost(1));
- EXPECT_CALL(event_handler, HandleJsEvent("onSyncNotificationStateChange", _))
+ EXPECT_CALL(event_handler, HandleJsEvent("onNotificationStateChange", _))
.Times(AtMost(1));
ListValue arg_list1;
@@ -298,10 +299,10 @@ TEST_F(ProfileSyncServiceTest,
EXPECT_CALL(event_handler,
HandleJsMessageReply("delayTestMessage3", HasArgs(args3)));
- const JsArgList kNoArgs;
+ const JsEventDetails kNoDetails;
- EXPECT_CALL(event_handler, HandleJsEvent("onSyncServiceStateChanged",
- HasArgs(kNoArgs))).Times(AtLeast(3));
+ EXPECT_CALL(event_handler, HandleJsEvent("onServiceStateChanged",
+ HasDetails(kNoDetails))).Times(AtLeast(3));
JsFrontend* js_backend = service_->GetJsFrontend();
@@ -321,6 +322,8 @@ TEST_F(ProfileSyncServiceTest,
// Fires delayTestMessage3.
ui_loop_.RunAllPending();
+ const JsArgList kNoArgs;
+
js_backend->ProcessMessage("delayNotRepliedTo", kNoArgs, &event_handler);
js_backend->RemoveHandler(&event_handler);