Deprecate CredentialRequestOptions.unmediated in favor mediation enum

components/password_manager/core/browser/password_manager_metrics_util.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_PASSWORD_MANAGER_CORE_BROWSER_PASSWORD_MANAGER_METRICS_UTIL_H_
 #define COMPONENTS_PASSWORD_MANAGER_CORE_BROWSER_PASSWORD_MANAGER_METRICS_UTIL_H_
 
 #include <stddef.h>
 
 #include <string>
 
 #include "components/autofill/core/common/password_form.h"
+#include "components/password_manager/core/common/credential_manager_types.h"
 
 namespace password_manager {
 
 namespace metrics_util {
 
 // Metrics: "PasswordManager.InfoBarResponse"
 enum ResponseType {
   NO_RESPONSE = 0,
   REMEMBER_PASSWORD,
   NEVER_REMEMBER_PASSWORD,
@@ skipping 140 matching lines @@
   CREDENTIAL_MANAGER_GET_NONE_FIRST_RUN,
   // Return empty credential for whatever reason.
   CREDENTIAL_MANAGER_GET_NONE,
   // Return a credential from the account chooser.
   CREDENTIAL_MANAGER_GET_ACCOUNT_CHOOSER,
   // User is auto signed in.
   CREDENTIAL_MANAGER_GET_AUTOSIGNIN,
   CREDENTIAL_MANAGER_GET_COUNT
 };
 
-enum CredentialManagerGetMediation {
-  CREDENTIAL_MANAGER_GET_MEDIATED,
-  CREDENTIAL_MANAGER_GET_UNMEDIATED
-};
-
 // Metrics: "PasswordManager.HttpPasswordMigrationMode"
 enum HttpPasswordMigrationMode {
   HTTP_PASSWORD_MIGRATION_MODE_MOVE,
   HTTP_PASSWORD_MIGRATION_MODE_COPY,
   HTTP_PASSWORD_MIGRATION_MODE_COUNT
 };
 
 enum PasswordReusePasswordFieldDetected {
   NO_PASSWORD_FIELD,
   HAS_PASSWORD_FIELD,
@@ skipping 52 matching lines @@
 
 // Logs mode of HTTP password migration.
 void LogHttpPasswordMigrationMode(HttpPasswordMigrationMode mode);
 
 // Log if the account chooser has empty username or duplicate usernames. In
 // addition record number of the placeholder avatars and total number of rows.
 void LogAccountChooserUsability(AccountChooserUsabilityMetric usability,
                                 int count_empty_icons,
                                 int count_accounts);
 
 // Log the result of navigator.credentials.get. |status| specifies the

[vasilii, 2017/05/18 12:34:03]

Not |status| anymore.

[jdoerrie, 2017/05/18 14:57:25]

Acknowledged, I'll drop it. '|mediation| specifies the "mediation" parameter of
the API method.' would be pretty redundant.

-// "unmediated" parameter of the API method.
+// "mediation" parameter of the API method.
 void LogCredentialManagerGetResult(CredentialManagerGetResult result,
-                                   CredentialManagerGetMediation status);
+                                   CredentialMediationRequirement mediation);
 
 // Log the password reuse.
 void LogPasswordReuse(int password_length,
                       int saved_passwords,
                       int number_matches,
                       bool password_field_detected);
 
 // Log when the user selects the "Login not secure" warning in the password
 // autofill dropdown to show more information about the warning.
 void LogShowedHttpNotSecureExplanation();
 
 // Log that the Form-Not-Secure warning was shown. Should be called at most once
 // per main-frame navigation.
 void LogShowedFormNotSecureWarningOnCurrentNavigation();
 
 // Log a password successful submission event.
 void LogPasswordSuccessfulSubmissionIndicatorEvent(
     autofill::PasswordForm::SubmissionIndicatorEvent event);
 
 }  // namespace metrics_util
 
 }  // namespace password_manager
 
 #endif  // COMPONENTS_PASSWORD_MANAGER_CORE_BROWSER_PASSWORD_MANAGER_METRICS_UTIL_H_