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

Unified Diff: chrome/browser/ui/webui/signin/inline_login_ui_browsertest.cc

Issue 2594703002: Create a new profile and sign in the user. (Closed)
Patch Set: Rebase Created 4 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
Index: chrome/browser/ui/webui/signin/inline_login_ui_browsertest.cc
diff --git a/chrome/browser/ui/webui/signin/inline_login_ui_browsertest.cc b/chrome/browser/ui/webui/signin/inline_login_ui_browsertest.cc
index d8fb3755234a71aec82f5f5551f6d7cdf085a6f6..57bd3f0fa65e388481e77c10dc6211bf1301e35a 100644
--- a/chrome/browser/ui/webui/signin/inline_login_ui_browsertest.cc
+++ b/chrome/browser/ui/webui/signin/inline_login_ui_browsertest.cc
@@ -151,12 +151,13 @@ class MockInlineSigninHelper : public InlineSigninHelper {
MOCK_METHOD1(OnClientOAuthSuccess, void(const ClientOAuthResult& result));
MOCK_METHOD1(OnClientOAuthFailure, void(const GoogleServiceAuthError& error));
- MOCK_METHOD7(CreateSyncStarter,
+ MOCK_METHOD8(CreateSyncStarter,
void(Browser*,
content::WebContents*,
const GURL&,
const GURL&,
const std::string&,
+ OneClickSigninSyncStarter::ProfileMode,
OneClickSigninSyncStarter::StartSyncMode,
OneClickSigninSyncStarter::ConfirmationRequired));
@@ -209,12 +210,13 @@ class MockSyncStarterInlineSigninHelper : public InlineSigninHelper {
bool choose_what_to_sync,
bool confirm_untrusted_signin);
- MOCK_METHOD7(CreateSyncStarter,
+ MOCK_METHOD8(CreateSyncStarter,
void(Browser*,
content::WebContents*,
const GURL&,
const GURL&,
const std::string&,
+ OneClickSigninSyncStarter::ProfileMode,
OneClickSigninSyncStarter::StartSyncMode,
OneClickSigninSyncStarter::ConfirmationRequired));
@@ -593,6 +595,7 @@ IN_PROC_BROWSER_TEST_F(InlineLoginHelperBrowserTest,
EXPECT_CALL(
*helper,
CreateSyncStarter(_, _, _, _, "refresh_token",
+ OneClickSigninSyncStarter::CURRENT_PROFILE,
OneClickSigninSyncStarter::CONFIRM_SYNC_SETTINGS_FIRST,
OneClickSigninSyncStarter::CONFIRM_AFTER_SIGNIN));
@@ -627,6 +630,7 @@ IN_PROC_BROWSER_TEST_F(InlineLoginHelperBrowserTest,
false); // confirm untrusted signin
EXPECT_CALL(*helper, CreateSyncStarter(
_, _, _, _, "refresh_token",
+ OneClickSigninSyncStarter::CURRENT_PROFILE,
OneClickSigninSyncStarter::CONFIGURE_SYNC_FIRST,
OneClickSigninSyncStarter::CONFIRM_AFTER_SIGNIN));
@@ -662,6 +666,7 @@ IN_PROC_BROWSER_TEST_F(InlineLoginHelperBrowserTest,
EXPECT_CALL(
*helper,
CreateSyncStarter(_, _, _, _, "refresh_token",
+ OneClickSigninSyncStarter::CURRENT_PROFILE,
OneClickSigninSyncStarter::CONFIRM_SYNC_SETTINGS_FIRST,
OneClickSigninSyncStarter::CONFIRM_UNTRUSTED_SIGNIN));
@@ -699,6 +704,7 @@ IN_PROC_BROWSER_TEST_F(InlineLoginHelperBrowserTest,
// settings, which means the user wants to CONFIGURE_SYNC_FIRST.
EXPECT_CALL(*helper, CreateSyncStarter(
_, _, _, _, "refresh_token",
+ OneClickSigninSyncStarter::CURRENT_PROFILE,
OneClickSigninSyncStarter::CONFIGURE_SYNC_FIRST,
OneClickSigninSyncStarter::CONFIRM_AFTER_SIGNIN));

Powered by Google App Engine
This is Rietveld 408576698