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

Unified Diff: chrome/browser/resources/options/password_manager.js

Issue 1615653005: [Password manager] Human readable origins for Android credentials on chrome://settings/passwords (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Transfer multi-request logic from PasswordStore to AffiliatedMatchHelper Created 4 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/resources/options/password_manager.js
diff --git a/chrome/browser/resources/options/password_manager.js b/chrome/browser/resources/options/password_manager.js
index 3e501727ff7b904a63df1d7a3843d10c35f21985..6a792345ba22cf2fd80622150cb72637a6196dd2 100644
--- a/chrome/browser/resources/options/password_manager.js
+++ b/chrome/browser/resources/options/password_manager.js
@@ -176,9 +176,9 @@ cr.define('options', function() {
var query = this.lastQuery_;
var filter = function(entry, index, list) {
// Search both shown URL and username.
- var shownUrl = entry[options.passwordManager.SHOWN_URL_FIELD];
+ var shownOrigin = entry[options.passwordManager.SHOWN_ORIGIN_FIELD];
var username = entry[options.passwordManager.USERNAME_FIELD];
- if (shownUrl.toLowerCase().indexOf(query.toLowerCase()) >= 0 ||
+ if (shownOrigin.toLowerCase().indexOf(query.toLowerCase()) >= 0 ||
username.toLowerCase().indexOf(query.toLowerCase()) >= 0) {
// Keep the original index so we can delete correctly. See also
// deleteItemAtIndex() in password_manager_list.js that uses this.

Powered by Google App Engine
This is Rietveld 408576698