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

Unified Diff: ceee/ie/broker/api_dispatcher_unittest.cc

Issue 4989002: Firing event to broker without worker thread. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 10 years, 1 month 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: ceee/ie/broker/api_dispatcher_unittest.cc
===================================================================
--- ceee/ie/broker/api_dispatcher_unittest.cc (revision 66617)
+++ ceee/ie/broker/api_dispatcher_unittest.cc (working copy)
@@ -255,7 +255,7 @@
CComBSTR request_name(keys::kAutomationBrowserEventRequestTarget);
EXPECT_CALL(postman, PostMessage(StrEq(CComBSTR(message1_str.c_str()).m_str),
StrEq(request_name.m_str))).Times(1);
- dispatcher.FireEvent(CComBSTR("Event1"), CComBSTR("EventHandler1Args"));
+ dispatcher.FireEvent("Event1", "EventHandler1Args");
ListValue message2;
message2.Append(Value::CreateStringValue("Event2"));
@@ -264,10 +264,10 @@
base::JSONWriter::Write(&message2, false, &message2_str);
EXPECT_CALL(postman, PostMessage(StrEq(CComBSTR(message2_str.c_str()).m_str),
StrEq(request_name.m_str))).Times(1);
- dispatcher.FireEvent(CComBSTR("Event2"), CComBSTR("EventHandler2Args"));
+ dispatcher.FireEvent("Event2", "EventHandler2Args");
// There shouldn't be a post when the event handler returns false.
- dispatcher.FireEvent(CComBSTR("Event3"), CComBSTR(""));
+ dispatcher.FireEvent("Event3", "");
}
// TODO(mad@chromium.org): Add tests for the EphemeralEventHandlers.

Powered by Google App Engine
This is Rietveld 408576698