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

Unified Diff: chrome/browser/ui/tab_helpers.cc

Issue 872433006: Disconnect one click sign in code. The code itself will be removed in followup (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix merge Created 5 years, 10 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/browser/ui/sync/one_click_signin_sync_starter.cc ('k') | chrome/chrome_tests.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/tab_helpers.cc
diff --git a/chrome/browser/ui/tab_helpers.cc b/chrome/browser/ui/tab_helpers.cc
index 30d4da1d26e4473f1b74a425108c75df75048bea..19fb99696300c63e76e5d275950502831aec8dc3 100644
--- a/chrome/browser/ui/tab_helpers.cc
+++ b/chrome/browser/ui/tab_helpers.cc
@@ -36,7 +36,6 @@
#include "components/autofill/core/browser/autofill_manager.h"
#include "components/dom_distiller/content/web_contents_main_frame_observer.h"
#include "components/password_manager/core/browser/password_manager.h"
-#include "components/signin/core/common/profile_management_switches.h"
#include "content/public/browser/web_contents.h"
#if defined(OS_ANDROID)
@@ -87,10 +86,6 @@
#endif // defined(ENABLE_PRINT_PREVIEW)
#endif // defined(ENABLE_PRINTING)
-#if defined(ENABLE_ONE_CLICK_SIGNIN)
-#include "chrome/browser/ui/sync/one_click_signin_helper.h"
-#endif
-
using content::WebContents;
namespace {
@@ -221,21 +216,6 @@ void TabHelpers::AttachTabHelpers(WebContents* web_contents) {
web_contents);
}
-#if defined(ENABLE_ONE_CLICK_SIGNIN)
- // If this is not an incognito window, setup to handle one-click login.
- // We don't want to check that the profile is already connected at this time
- // because the connected state may change while this tab is open. Having a
- // one-click signin helper attached does not cause problems if the profile
- // happens to be already connected.
- if (switches::IsEnableWebBasedSignin() &&
- OneClickSigninHelper::CanOffer(web_contents,
- OneClickSigninHelper::CAN_OFFER_FOR_ALL,
- std::string(),
- NULL)) {
- OneClickSigninHelper::CreateForWebContents(web_contents);
- }
-#endif
-
if (predictors::ResourcePrefetchPredictorFactory::GetForProfile(
web_contents->GetBrowserContext())) {
predictors::ResourcePrefetchPredictorTabHelper::CreateForWebContents(
« no previous file with comments | « chrome/browser/ui/sync/one_click_signin_sync_starter.cc ('k') | chrome/chrome_tests.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698