Deprecate CredentialRequestOptions.unmediated in favor mediation enum

components/password_manager/content/browser/credential_manager_impl.h

Index: components/password_manager/content/browser/credential_manager_impl.h
diff --git a/components/password_manager/content/browser/credential_manager_impl.h b/components/password_manager/content/browser/credential_manager_impl.h
index 9694627d07c7b49b2c654e835cb6ac78ccfbc795..e4d49b585a3437cab3458dfa825caaa4276aeff4 100644
--- a/components/password_manager/content/browser/credential_manager_impl.h
+++ b/components/password_manager/content/browser/credential_manager_impl.h
@@ -15,6 +15,7 @@
 #include "components/password_manager/core/browser/credential_manager_pending_request_task.h"
 #include "components/password_manager/core/browser/credential_manager_pending_require_user_mediation_task.h"
 #include "components/password_manager/core/browser/password_store_consumer.h"
+#include "components/password_manager/core/common/credential_manager_types.h"
 #include "components/prefs/pref_member.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "mojo/public/cpp/bindings/binding_set.h"
@@ -52,7 +53,7 @@ class CredentialManagerImpl
   // mojom::CredentialManager methods:
   void Store(const CredentialInfo& credential, StoreCallback callback) override;
   void RequireUserMediation(RequireUserMediationCallback callback) override;
-  void Get(bool zero_click_only,
+  void Get(mojom::CredentialMediationRequirement mediation,

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

Can we use only the enum declared in credential_manager_types?
See components/autofill/content/common/autofill_types.typemap 

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

Done, this required entries in

components/password_manager/content/common/credential_manager.typemap

and
components/password_manager/content/common/credential_manager_struct_traits.{h,cc}

            bool include_passwords,
            const std::vector<GURL>& federations,
            GetCallback callback) override;
@@ -87,6 +88,10 @@ class CredentialManagerImpl
   // Set to false to disable automatic signing in.
   BooleanPrefMember auto_signin_enabled_;
 
+  // Set during |Get|, needed to log the correct result during
+  // |SendPasswordForm|.
+  CredentialMediationRequirement mediation_;
+
   // When 'OnRequestCredential' is called, it in turn calls out to the
   // PasswordStore; we push enough data into Pending*Task objects so that
   // they can properly respond to the request once the PasswordStore gives