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

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

Issue 1659203002: Remove HostDesktopType from FindLastActive[WithProfile] (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: nullptr Created 4 years, 11 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
Index: chrome/browser/ui/webui/signin/inline_login_handler_impl.cc
diff --git a/chrome/browser/ui/webui/signin/inline_login_handler_impl.cc b/chrome/browser/ui/webui/signin/inline_login_handler_impl.cc
index ab6886853de94cc54cdc386544ea898b4e7e70fe..c97de483f5ebf1f06ac009c7a21b3694d0a86548 100644
--- a/chrome/browser/ui/webui/signin/inline_login_handler_impl.cc
+++ b/chrome/browser/ui/webui/signin/inline_login_handler_impl.cc
@@ -386,8 +386,7 @@ bool InlineSigninHelper::HandleCrossAccountError(
return false;
}
- Browser* browser = chrome::FindLastActiveWithProfile(
- profile_, chrome::GetActiveDesktop());
+ Browser* browser = chrome::FindLastActiveWithProfile(profile_);
content::WebContents* web_contents =
browser->tab_strip_model()->GetActiveWebContents();
@@ -410,8 +409,7 @@ void InlineSigninHelper::ConfirmEmailAction(
OneClickSigninSyncStarter::ConfirmationRequired confirmation_required,
OneClickSigninSyncStarter::StartSyncMode start_mode,
InlineSigninHelper::Action action) {
- Browser* browser = chrome::FindLastActiveWithProfile(
- profile_, chrome::GetActiveDesktop());
+ Browser* browser = chrome::FindLastActiveWithProfile(profile_);
switch (action) {
case InlineSigninHelper::CREATE_NEW_USER:
content::RecordAction(
@@ -859,10 +857,8 @@ void InlineLoginHandlerImpl::HandleLoginError(const std::string& error_msg) {
Browser* InlineLoginHandlerImpl::GetDesktopBrowser() {
Browser* browser = chrome::FindBrowserWithWebContents(
web_ui()->GetWebContents());
- if (!browser) {
- browser = chrome::FindLastActiveWithProfile(
- Profile::FromWebUI(web_ui()), chrome::GetActiveDesktop());
- }
+ if (!browser)
+ browser = chrome::FindLastActiveWithProfile(Profile::FromWebUI(web_ui()));
return browser;
}
« no previous file with comments | « chrome/browser/ui/views/select_file_dialog_extension.cc ('k') | chrome/browser/ui/webui/signin/sync_confirmation_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698