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

Unified Diff: chrome_frame/test/automation_client_mock.cc

Issue 1205001: Revert 42366 - Allow TabContentsDelegate classes to specify whether InfoBars ... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 10 years, 9 months 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/chrome_frame_plugin.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 42377)
+++ chrome_frame/test/automation_client_mock.cc (working copy)
@@ -100,17 +100,7 @@
.Times(1)
.WillOnce(QUIT_LOOP(loop));
- ChromeFrameLaunchParams cfp = {
- timeout,
- GURL(),
- GURL(),
- profile,
- L"",
- false,
- false,
- false
- };
- EXPECT_TRUE(client->Initialize(&cfd, cfp));
+ EXPECT_TRUE(client->Initialize(&cfd, timeout, false, profile, L"", false));
loop.RunFor(10);
client->Uninitialize();
}
@@ -135,17 +125,7 @@
EXPECT_CALL(cfd, OnAutomationServerLaunchFailed(_, _))
.Times(0);
- ChromeFrameLaunchParams cfp = {
- timeout,
- GURL(),
- GURL(),
- profile,
- L"",
- false,
- false,
- false
- };
- EXPECT_TRUE(client->Initialize(&cfd, cfp));
+ EXPECT_TRUE(client->Initialize(&cfd, timeout, false, profile, L"", false));
loop.RunFor(11);
client->Uninitialize();
@@ -185,17 +165,7 @@
.WillOnce(QUIT_LOOP(loop));
}
- ChromeFrameLaunchParams cfp = {
- timeout,
- GURL(),
- GURL(),
- profile,
- L"",
- false,
- false,
- false
- };
- EXPECT_TRUE(client->Initialize(&cfd, cfp));
+ EXPECT_TRUE(client->Initialize(&cfd, timeout, false, profile, L"", false));
loop.RunFor(10);
client->Uninitialize();
client = NULL;
@@ -234,17 +204,7 @@
.Times(1)
.WillOnce(QUIT_LOOP_SOON(loop, 2));
- ChromeFrameLaunchParams cfp = {
- 10000,
- GURL(),
- GURL(),
- profile,
- L"",
- false,
- false,
- false
- };
- EXPECT_TRUE(client->Initialize(&cfd, cfp));
+ EXPECT_TRUE(client->Initialize(&cfd, 10000, false, profile, L"", false));
loop.RunFor(10);
client->Uninitialize();
@@ -277,17 +237,7 @@
EXPECT_CALL(proxy_, CancelAsync(_)).Times(testing::AnyNumber());
// Here we go!
- ChromeFrameLaunchParams cfp = {
- timeout,
- GURL(),
- GURL(),
- profile_,
- L"",
- false,
- false,
- false
- };
- EXPECT_TRUE(client_->Initialize(&cfd_, cfp));
+ EXPECT_TRUE(client_->Initialize(&cfd_, timeout, false, profile_, L"", false));
loop_.RunFor(10);
client_->Uninitialize();
}
@@ -312,17 +262,8 @@
Set_CFD_LaunchFailed(AUTOMATION_CREATE_TAB_FAILED);
// Here we go!
- ChromeFrameLaunchParams cfp = {
- timeout_,
- GURL(),
- GURL(),
- profile_,
- L"",
- false,
- false,
- false
- };
- EXPECT_TRUE(client_->Initialize(&cfd_, cfp));
+ EXPECT_TRUE(client_->Initialize(&cfd_, timeout_, false, profile_, L"",
+ false));
loop_.RunFor(4);
client_->Uninitialize();
}
« no previous file with comments | « chrome_frame/chrome_frame_plugin.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