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

Unified Diff: chrome/browser/resources/settings/passwords_and_forms_page/passwords_section.html

Issue 2237533002: Add checkbox to allow auto signing in with a saved password. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix closure Created 4 years, 4 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/resources/settings/passwords_and_forms_page/passwords_section.html
diff --git a/chrome/browser/resources/settings/passwords_and_forms_page/passwords_section.html b/chrome/browser/resources/settings/passwords_and_forms_page/passwords_section.html
index 59abdc9b0149559419fa2fe7cdf8165aaafc3697..3fc33356d28177ad9cea1e5a6b8926f34a07e1ea 100644
--- a/chrome/browser/resources/settings/passwords_and_forms_page/passwords_section.html
+++ b/chrome/browser/resources/settings/passwords_and_forms_page/passwords_section.html
@@ -6,6 +6,8 @@
<link rel="import" href="chrome://resources/cr_elements/cr_shared_menu/cr_shared_menu.html">
<link rel="import" href="/passwords_and_forms_page/password_edit_dialog.html">
<link rel="import" href="/passwords_and_forms_page/passwords_shared_css.html">
+<link rel="import" href="/prefs/prefs.html">
+<link rel="import" href="/prefs/prefs_behavior.html">
<link rel="import" href="/settings_shared_css.html">
<dom-module id="passwords-section">
@@ -25,6 +27,10 @@
width: 0;
}
+ #autosigninCheckbox {
+ margin-bottom: 24px;
+ }
+
.website-column {
flex: 3;
}
@@ -49,6 +55,11 @@
@apply(--settings-actionable);
}
</style>
+ <settings-checkbox id="autosigninCheckbox"
michaelpg 2016/08/12 21:51:28 autosignin or autoSignIn or autoSignin, don't care
hcarmona 2016/08/16 22:33:39 All changed to autosignin to match setting.
michaelpg 2016/08/16 23:21:26 Thanks. It's a pet peeve of mine -- I had to wage
+ pref="{{prefs.credentials_enable_autosignin}}"
+ label="$i18n{passwordsAutoSigninLabel}"
+ sub-label="$i18n{passwordsAutoSigninDescription}">
+ </settings-checkbox>
<div id="manageLink">$i18nRaw{managePasswordsLabel}</div>
<div class="heading">$i18n{savedPasswordsHeading}</div>
<div class="item-list">

Powered by Google App Engine
This is Rietveld 408576698