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

Unified Diff: chrome_frame/test/automation_client_mock.cc

Issue 5998006: Clean up Automation and Chrome Frame IPC code. (Closed) Base URL: svn://chrome-svn/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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome_frame/test/automation_client_mock.h ('k') | chrome_frame/test/chrome_frame_automation_mock.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome_frame/test/automation_client_mock.cc
===================================================================
--- chrome_frame/test/automation_client_mock.cc (revision 69966)
+++ chrome_frame/test/automation_client_mock.cc (working copy)
@@ -170,18 +170,18 @@
EXPECT_CALL(cfd, GetBounds(_)).Times(testing::AnyNumber());
- EXPECT_CALL(cfd, OnNavigationStateChanged(_, _))
+ EXPECT_CALL(cfd, OnNavigationStateChanged(_))
.Times(testing::AnyNumber());
{
testing::InSequence s;
- EXPECT_CALL(cfd, OnDidNavigate(_, EqNavigationInfoUrl(GURL())))
+ EXPECT_CALL(cfd, OnDidNavigate(EqNavigationInfoUrl(GURL())))
.Times(1);
- EXPECT_CALL(cfd, OnUpdateTargetUrl(_, _)).Times(testing::AtMost(1));
+ EXPECT_CALL(cfd, OnUpdateTargetUrl(_)).Times(testing::AtMost(1));
- EXPECT_CALL(cfd, OnLoad(_, _))
+ EXPECT_CALL(cfd, OnLoad(_))
.Times(1)
.WillOnce(QUIT_LOOP(loop));
}
@@ -218,18 +218,18 @@
url, std::string(), &navigation_constraints))));
EXPECT_CALL(cfd, GetBounds(_)).Times(testing::AnyNumber());
- EXPECT_CALL(cfd, OnNavigationStateChanged(_, _)).Times(testing::AnyNumber());
+ EXPECT_CALL(cfd, OnNavigationStateChanged(_)).Times(testing::AnyNumber());
- EXPECT_CALL(cfd, OnRequestStart(_, _, _))
+ EXPECT_CALL(cfd, OnRequestStart(_, _))
// Often there's another request for the error page
.Times(testing::Between(1, 2))
- .WillRepeatedly(testing::WithArgs<1>(testing::Invoke(CreateFunctor(&cfd,
+ .WillRepeatedly(testing::WithArgs<0>(testing::Invoke(CreateFunctor(&cfd,
&MockCFDelegate::Reply, connection_failed))));
- EXPECT_CALL(cfd, OnUpdateTargetUrl(_, _)).Times(testing::AnyNumber());
- EXPECT_CALL(cfd, OnLoad(_, _)).Times(testing::AtMost(1));
+ EXPECT_CALL(cfd, OnUpdateTargetUrl(_)).Times(testing::AnyNumber());
+ EXPECT_CALL(cfd, OnLoad(_)).Times(testing::AtMost(1));
- EXPECT_CALL(cfd, OnNavigationFailed(_, _, GURL(url)))
+ EXPECT_CALL(cfd, OnNavigationFailed(_, GURL(url)))
.Times(1)
.WillOnce(QUIT_LOOP_SOON(loop, 2));
« no previous file with comments | « chrome_frame/test/automation_client_mock.h ('k') | chrome_frame/test/chrome_frame_automation_mock.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698