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

Unified Diff: ceee/ie/broker/window_events_funnel_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
« no previous file with comments | « ceee/ie/broker/window_events_funnel.cc ('k') | ceee/ie/common/ceee_module_util.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ceee/ie/broker/window_events_funnel_unittest.cc
===================================================================
--- ceee/ie/broker/window_events_funnel_unittest.cc (revision 66617)
+++ ceee/ie/broker/window_events_funnel_unittest.cc (working copy)
@@ -65,14 +65,14 @@
class MockChromePostman : public ChromePostman {
public:
- MOCK_METHOD2(FireEvent, void(BSTR, BSTR));
+ MOCK_METHOD2(FireEvent, void(const char*, const char*));
};
// Simply instantiating the postman will register it as the
// one and only singleton to use all the time.
CComObjectStackEx<testing::StrictMock<MockChromePostman>> postman;
EXPECT_CALL(postman, FireEvent(
- StrEq(CComBSTR(kEventName).m_str),
- StrEq(CComBSTR(event_args_str.c_str()).m_str))).Times(1);
+ StrEq(kEventName),
+ StrEq(event_args_str.c_str()))).Times(1);
EXPECT_HRESULT_SUCCEEDED(events_funnel.CallSendEvent(kEventName, event_args));
}
« no previous file with comments | « ceee/ie/broker/window_events_funnel.cc ('k') | ceee/ie/common/ceee_module_util.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698