| Index: components/password_manager/content/browser/credential_manager_impl.h
|
| diff --git a/components/password_manager/content/browser/credential_manager_dispatcher.h b/components/password_manager/content/browser/credential_manager_impl.h
|
| similarity index 68%
|
| rename from components/password_manager/content/browser/credential_manager_dispatcher.h
|
| rename to components/password_manager/content/browser/credential_manager_impl.h
|
| index 1471a7b767e7d7a2cbff2f28ca9912433a30244a..e6429eb0fa0c874f70c084176bfd0c4e1997aa5b 100644
|
| --- a/components/password_manager/content/browser/credential_manager_dispatcher.h
|
| +++ b/components/password_manager/content/browser/credential_manager_impl.h
|
| @@ -2,20 +2,22 @@
|
| // Use of this source code is governed by a BSD-style license that can be
|
| // found in the LICENSE file.
|
|
|
| -#ifndef COMPONENTS_PASSWORD_MANAGER_CONTENT_BROWSER_CONTENT_CREDENTIAL_MANAGER_DISPATCHER_H_
|
| -#define COMPONENTS_PASSWORD_MANAGER_CONTENT_BROWSER_CONTENT_CREDENTIAL_MANAGER_DISPATCHER_H_
|
| +#ifndef COMPONENTS_PASSWORD_MANAGER_CONTENT_BROWSER_CONTENT_CREDENTIAL_MANAGER_IMPL_H_
|
| +#define COMPONENTS_PASSWORD_MANAGER_CONTENT_BROWSER_CONTENT_CREDENTIAL_MANAGER_IMPL_H_
|
|
|
| #include <memory>
|
|
|
| #include "base/callback.h"
|
| #include "base/macros.h"
|
| #include "base/memory/weak_ptr.h"
|
| +#include "components/password_manager/content/public/interfaces/credential_manager.mojom.h"
|
| #include "components/password_manager/core/browser/credential_manager_password_form_manager.h"
|
| #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/prefs/pref_member.h"
|
| #include "content/public/browser/web_contents_observer.h"
|
| +#include "mojo/public/cpp/bindings/binding_set.h"
|
|
|
| class GURL;
|
|
|
| @@ -34,42 +36,35 @@ class PasswordManagerDriver;
|
| class PasswordStore;
|
| struct CredentialInfo;
|
|
|
| -class CredentialManagerDispatcher
|
| - : public content::WebContentsObserver,
|
| +class CredentialManagerImpl
|
| + : public mojom::CredentialManager,
|
| + public content::WebContentsObserver,
|
| public CredentialManagerPasswordFormManagerDelegate,
|
| public CredentialManagerPendingRequestTaskDelegate,
|
| public CredentialManagerPendingRequireUserMediationTaskDelegate {
|
| public:
|
| - CredentialManagerDispatcher(content::WebContents* web_contents,
|
| - PasswordManagerClient* client);
|
| - ~CredentialManagerDispatcher() override;
|
| -
|
| - // Called in response to an IPC from the renderer, triggered by a page's call
|
| - // to 'navigator.credentials.store'.
|
| - virtual void OnStore(int request_id, const password_manager::CredentialInfo&);
|
| -
|
| - // Called in response to an IPC from the renderer, triggered by a page's call
|
| - // to 'navigator.credentials.requireUserMediation'.
|
| - virtual void OnRequireUserMediation(int request_id);
|
| -
|
| - // Called in response to an IPC from the renderer, triggered by a page's call
|
| - // to 'navigator.credentials.request'.
|
| - //
|
| - // TODO(vabr): Determine if we can drop the `const` here to save some copies
|
| - // while processing the request.
|
| - virtual void OnRequestCredential(int request_id,
|
| - bool zero_click_only,
|
| - bool include_passwords,
|
| - const std::vector<GURL>& federations);
|
| -
|
| - // content::WebContentsObserver implementation.
|
| - bool OnMessageReceived(const IPC::Message& message) override;
|
| + CredentialManagerImpl(content::WebContents* web_contents,
|
| + PasswordManagerClient* client);
|
| + ~CredentialManagerImpl() override;
|
| +
|
| + void BindRequest(mojom::CredentialManagerRequest request);
|
| +
|
| + // mojom::CredentialManager methods:
|
| + void Store(mojom::CredentialInfoPtr credential,
|
| + const StoreCallback& callback) override;
|
| + void RequireUserMediation(
|
| + const RequireUserMediationCallback& callback) override;
|
| + void Get(bool zero_click_only,
|
| + bool include_passwords,
|
| + mojo::Array<mojo::String> federations,
|
| + const GetCallback& callback) override;
|
|
|
| // CredentialManagerPendingRequestTaskDelegate:
|
| bool IsZeroClickAllowed() const override;
|
| GURL GetOrigin() const override;
|
| - void SendCredential(int request_id, const CredentialInfo& info) override;
|
| - void SendPasswordForm(int request_id,
|
| + void SendCredential(const SendCredentialCallback& send_callback,
|
| + const CredentialInfo& info) override;
|
| + void SendPasswordForm(const SendCredentialCallback& send_callback,
|
| const autofill::PasswordForm* form) override;
|
| PasswordManagerClient* client() const override;
|
| autofill::PasswordForm GetSynthesizedFormForOrigin() const override;
|
| @@ -89,7 +84,7 @@ class CredentialManagerDispatcher
|
| // Schedules a CredentiaManagerPendingRequestTask (during
|
| // |OnRequestCredential()|) after the PasswordStore's AffiliationMatchHelper
|
| // grabs a list of realms related to the current web origin.
|
| - void ScheduleRequestTask(int request_id,
|
| + void ScheduleRequestTask(const GetCallback& callback,
|
| bool zero_click_only,
|
| bool include_passwords,
|
| const std::vector<GURL>& federations,
|
| @@ -99,7 +94,7 @@ class CredentialManagerDispatcher
|
| // AffiliationMatchHelper grabs a list of realms related to the current
|
| // web origin.
|
| void ScheduleRequireMediationTask(
|
| - int request_id,
|
| + const RequireUserMediationCallback& callback,
|
| const std::vector<std::string>& android_realms);
|
|
|
| // Returns true iff it's OK to update credentials in the password store.
|
| @@ -119,11 +114,13 @@ class CredentialManagerDispatcher
|
| std::unique_ptr<CredentialManagerPendingRequireUserMediationTask>
|
| pending_require_user_mediation_;
|
|
|
| - base::WeakPtrFactory<CredentialManagerDispatcher> weak_factory_;
|
| + mojo::BindingSet<mojom::CredentialManager> bindings_;
|
|
|
| - DISALLOW_COPY_AND_ASSIGN(CredentialManagerDispatcher);
|
| + base::WeakPtrFactory<CredentialManagerImpl> weak_factory_;
|
| +
|
| + DISALLOW_COPY_AND_ASSIGN(CredentialManagerImpl);
|
| };
|
|
|
| } // namespace password_manager
|
|
|
| -#endif // COMPONENTS_PASSWORD_MANAGER_CONTENT_BROWSER_CONTENT_CREDENTIAL_MANAGER_DISPATCHER_H_
|
| +#endif // COMPONENTS_PASSWORD_MANAGER_CONTENT_BROWSER_CONTENT_CREDENTIAL_MANAGER_IMPL_H_
|
|
|