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

Unified Diff: components/password_manager/core/browser/login_database.cc

Issue 2716583003: Rename Origin.unique() to opaque().
Patch Set: Mac fixes Created 3 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: components/password_manager/core/browser/login_database.cc
diff --git a/components/password_manager/core/browser/login_database.cc b/components/password_manager/core/browser/login_database.cc
index ff7d4da279494411702d1a367111191618c5bbc3..b15f2a1d0b5897523ac2081bd8985e912665d4e5 100644
--- a/components/password_manager/core/browser/login_database.cc
+++ b/components/password_manager/core/browser/login_database.cc
@@ -147,7 +147,7 @@ void BindAddStatement(const PasswordForm& form,
s->BindString(COLUMN_ICON_URL, form.icon_url.spec());
// An empty Origin serializes as "null" which would be strange to store here.
s->BindString(COLUMN_FEDERATION_URL,
- form.federation_origin.unique()
+ form.federation_origin.opaque()
? std::string()
: form.federation_origin.Serialize());
s->BindInt(COLUMN_SKIP_ZERO_CLICK, form.skip_zero_click);
@@ -863,7 +863,7 @@ PasswordStoreChangeList LoginDatabase::UpdateLogin(const PasswordForm& form) {
s.BindString16(12, form.display_name);
s.BindString(13, form.icon_url.spec());
// An empty Origin serializes as "null" which would be strange to store here.
- s.BindString(14, form.federation_origin.unique()
+ s.BindString(14, form.federation_origin.opaque()
? std::string()
: form.federation_origin.Serialize());
s.BindInt(15, form.skip_zero_click);

Powered by Google App Engine
This is Rietveld 408576698