Commit 288d6fcd authored by Bettina's avatar Bettina Committed by Commit Bot

Remove showing the change password card completely.

The change password card in the settings is not triggered
from password protection so it should be removed completely.
This is part 1 of the removal of the change password handler.
The next CL will remove the files completely.

Bug: 991764
Change-Id: I8e435d9ff517325cbca75eb41bc3a678fb282368
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1834950
Commit-Queue: Bettina Dea <bdea@chromium.org>
Reviewed-by: default avatarTommy Li <tommycli@chromium.org>
Reviewed-by: default avatarNathan Parker <nparker@chromium.org>
Reviewed-by: default avatarVarun Khaneja <vakh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#702516}
parent b64da89f
......@@ -153,11 +153,6 @@ Polymer({
this.showChangePassword = visibility;
});
if (loadTimeData.getBoolean('passwordProtectionAvailable')) {
settings.ChangePasswordBrowserProxyImpl.getInstance()
.initializeChangePasswordHandler();
}
if (settings.AndroidAppsBrowserProxyImpl) {
this.addWebUIListener(
'android-apps-info-update', this.androidAppsInfoUpdate_.bind(this));
......
......@@ -55,7 +55,6 @@
#include "components/favicon_base/favicon_url_parser.h"
#include "components/password_manager/core/common/password_manager_features.h"
#include "components/pref_registry/pref_registry_syncable.h"
#include "components/safe_browsing/buildflags.h"
#include "components/unified_consent/feature.h"
#include "content/public/browser/url_data_source.h"
#include "content/public/browser/web_contents.h"
......@@ -148,11 +147,6 @@
#include "chrome/browser/ui/webui/settings/printing_handler.h"
#endif
#if BUILDFLAG(FULL_SAFE_BROWSING)
#include "chrome/browser/safe_browsing/chrome_password_protection_service.h"
#include "chrome/browser/ui/webui/settings/change_password_handler.h"
#endif
namespace settings {
// static
void SettingsUI::RegisterProfilePrefs(
......@@ -251,20 +245,6 @@ SettingsUI::SettingsUI(content::WebUI* web_ui)
std::make_unique<IncompatibleApplicationsHandler>());
#endif // OS_WIN && BUILDFLAG(GOOGLE_CHROME_BRANDING)
bool password_protection_available = false;
#if BUILDFLAG(FULL_SAFE_BROWSING)
safe_browsing::ChromePasswordProtectionService* password_protection =
safe_browsing::ChromePasswordProtectionService::
GetPasswordProtectionService(profile);
password_protection_available = !!password_protection;
if (password_protection) {
AddSettingsPageUIHandler(
std::make_unique<ChangePasswordHandler>(profile, password_protection));
}
#endif
html_source->AddBoolean("passwordProtectionAvailable",
password_protection_available);
#if !defined(OS_CHROMEOS)
html_source->AddBoolean(
"diceEnabled",
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment