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

Side by Side 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 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « ceee/ie/broker/api_dispatcher.cc ('k') | ceee/ie/broker/broker.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 // 4 //
5 // Unit tests for ApiDispatcher. 5 // Unit tests for ApiDispatcher.
6 6
7 #include "base/json/json_writer.h" 7 #include "base/json/json_writer.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "ceee/ie/broker/api_dispatcher.h" 9 #include "ceee/ie/broker/api_dispatcher.h"
10 #include "ceee/ie/broker/chrome_postman.h" 10 #include "ceee/ie/broker/chrome_postman.h"
(...skipping 237 matching lines...) Expand 10 before | Expand all | Expand 10 after
248 dispatcher.RegisterPermanentEventHandler("Event3", EventHandler3); 248 dispatcher.RegisterPermanentEventHandler("Event3", EventHandler3);
249 249
250 ListValue message1; 250 ListValue message1;
251 message1.Append(Value::CreateStringValue("Event1")); 251 message1.Append(Value::CreateStringValue("Event1"));
252 message1.Append(Value::CreateStringValue("EventHandler1ConvertedArgs")); 252 message1.Append(Value::CreateStringValue("EventHandler1ConvertedArgs"));
253 std::string message1_str; 253 std::string message1_str;
254 base::JSONWriter::Write(&message1, false, &message1_str); 254 base::JSONWriter::Write(&message1, false, &message1_str);
255 CComBSTR request_name(keys::kAutomationBrowserEventRequestTarget); 255 CComBSTR request_name(keys::kAutomationBrowserEventRequestTarget);
256 EXPECT_CALL(postman, PostMessage(StrEq(CComBSTR(message1_str.c_str()).m_str), 256 EXPECT_CALL(postman, PostMessage(StrEq(CComBSTR(message1_str.c_str()).m_str),
257 StrEq(request_name.m_str))).Times(1); 257 StrEq(request_name.m_str))).Times(1);
258 dispatcher.FireEvent(CComBSTR("Event1"), CComBSTR("EventHandler1Args")); 258 dispatcher.FireEvent("Event1", "EventHandler1Args");
259 259
260 ListValue message2; 260 ListValue message2;
261 message2.Append(Value::CreateStringValue("Event2")); 261 message2.Append(Value::CreateStringValue("Event2"));
262 message2.Append(Value::CreateStringValue("EventHandler2ConvertedArgs")); 262 message2.Append(Value::CreateStringValue("EventHandler2ConvertedArgs"));
263 std::string message2_str; 263 std::string message2_str;
264 base::JSONWriter::Write(&message2, false, &message2_str); 264 base::JSONWriter::Write(&message2, false, &message2_str);
265 EXPECT_CALL(postman, PostMessage(StrEq(CComBSTR(message2_str.c_str()).m_str), 265 EXPECT_CALL(postman, PostMessage(StrEq(CComBSTR(message2_str.c_str()).m_str),
266 StrEq(request_name.m_str))).Times(1); 266 StrEq(request_name.m_str))).Times(1);
267 dispatcher.FireEvent(CComBSTR("Event2"), CComBSTR("EventHandler2Args")); 267 dispatcher.FireEvent("Event2", "EventHandler2Args");
268 268
269 // There shouldn't be a post when the event handler returns false. 269 // There shouldn't be a post when the event handler returns false.
270 dispatcher.FireEvent(CComBSTR("Event3"), CComBSTR("")); 270 dispatcher.FireEvent("Event3", "");
271 } 271 }
272 272
273 // TODO(mad@chromium.org): Add tests for the EphemeralEventHandlers. 273 // TODO(mad@chromium.org): Add tests for the EphemeralEventHandlers.
274 274
275 } // namespace 275 } // namespace
OLDNEW
« no previous file with comments | « ceee/ie/broker/api_dispatcher.cc ('k') | ceee/ie/broker/broker.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698