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

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: rebased 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
Index: chrome/browser/ui/tab_helpers.cc
diff --git a/chrome/browser/ui/tab_helpers.cc b/chrome/browser/ui/tab_helpers.cc
index 283933334bfea0d86ccab58da339abfdc8a515ca..cea5e7d57592aac1c2f1ea979ae2b2abc70f4738 100644
--- a/chrome/browser/ui/tab_helpers.cc
+++ b/chrome/browser/ui/tab_helpers.cc
@@ -34,7 +34,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(

Powered by Google App Engine
This is Rietveld 408576698