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

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

Issue 2864493003: Deprecate CredentialRequestOptions.unmediated in favor mediation enum (Closed)
Patch Set: Rebase Created 3 years, 7 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/credential_manager_logger.cc
diff --git a/components/password_manager/core/browser/credential_manager_logger.cc b/components/password_manager/core/browser/credential_manager_logger.cc
index f5173c1868ac341eb453e20430e270c91eca7225..2b50b724560aaef55bfa1ebfcf7b5bb37554175e 100644
--- a/components/password_manager/core/browser/credential_manager_logger.cc
+++ b/components/password_manager/core/browser/credential_manager_logger.cc
@@ -21,11 +21,22 @@ CredentialManagerLogger::~CredentialManagerLogger() = default;
void CredentialManagerLogger::LogRequestCredential(
const GURL& url,
- bool zero_click_only,
+ CredentialMediationRequirement mediation,
const std::vector<GURL>& federations) {
std::string s("CM API get credentials: origin=" +
SavePasswordProgressLogger::ScrubURL(url));
- s += ", zero_click_only=" + base::IntToString(zero_click_only);
+ s += ", mediation=";
+ switch (mediation) {
+ case CredentialMediationRequirement::kSilent:
+ s += "silent";
+ break;
+ case CredentialMediationRequirement::kOptional:
+ s += "optional";
+ break;
+ case CredentialMediationRequirement::kRequired:
+ s += "required";
+ break;
+ }
s += ", federations=";
for (const GURL& federation_provider : federations)
s += SavePasswordProgressLogger::ScrubURL(federation_provider) + ", ";

Powered by Google App Engine
This is Rietveld 408576698