Commit fb4a941a authored by Julie Jeongeun Kim's avatar Julie Jeongeun Kim Committed by Commit Bot

Clean up old Mojo type for mojom::PasswordAutofillAgent

This CL cleans up old mojo types for mojom::PasswordAutofillAgent
replacing AssociatedBinding with AssociatedReceiver.

Bug: 955171
Change-Id: I521cdbbc9a23203f0a33ffd2f0e265bd9dc7b1dc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1903113Reviewed-by: default avatarVasilii Sukhanov <vasilii@chromium.org>
Reviewed-by: default avatarKen Rockot <rockot@google.com>
Commit-Queue: Julie Kim <jkim@igalia.com>
Cr-Commit-Position: refs/heads/master@{#714048}
parent a4e1df98
...@@ -57,7 +57,8 @@ ...@@ -57,7 +57,8 @@
#include "content/public/common/url_constants.h" #include "content/public/common/url_constants.h"
#include "content/public/test/navigation_simulator.h" #include "content/public/test/navigation_simulator.h"
#include "content/public/test/web_contents_tester.h" #include "content/public/test/web_contents_tester.h"
#include "mojo/public/cpp/bindings/associated_binding.h" #include "mojo/public/cpp/bindings/associated_receiver.h"
#include "mojo/public/cpp/bindings/pending_associated_receiver.h"
#include "mojo/public/cpp/bindings/pending_receiver.h" #include "mojo/public/cpp/bindings/pending_receiver.h"
#include "services/service_manager/public/cpp/interface_provider.h" #include "services/service_manager/public/cpp/interface_provider.h"
#include "testing/gmock/include/gmock/gmock.h" #include "testing/gmock/include/gmock/gmock.h"
...@@ -144,15 +145,14 @@ class FakePasswordAutofillAgent ...@@ -144,15 +145,14 @@ class FakePasswordAutofillAgent
: public autofill::mojom::PasswordAutofillAgent { : public autofill::mojom::PasswordAutofillAgent {
public: public:
FakePasswordAutofillAgent() FakePasswordAutofillAgent()
: called_set_logging_state_(false), : called_set_logging_state_(false), logging_state_active_(false) {}
logging_state_active_(false),
binding_(this) {}
~FakePasswordAutofillAgent() override = default; ~FakePasswordAutofillAgent() override = default;
void BindRequest(mojo::ScopedInterfaceEndpointHandle handle) { void BindReceiver(mojo::ScopedInterfaceEndpointHandle handle) {
binding_.Bind(autofill::mojom::PasswordAutofillAgentAssociatedRequest( receiver_.Bind(
std::move(handle))); mojo::PendingAssociatedReceiver<autofill::mojom::PasswordAutofillAgent>(
std::move(handle)));
} }
bool called_set_logging_state() { return called_set_logging_state_; } bool called_set_logging_state() { return called_set_logging_state_; }
...@@ -187,7 +187,8 @@ class FakePasswordAutofillAgent ...@@ -187,7 +187,8 @@ class FakePasswordAutofillAgent
// Records data received via SetLoggingState() call. // Records data received via SetLoggingState() call.
bool logging_state_active_; bool logging_state_active_;
mojo::AssociatedBinding<autofill::mojom::PasswordAutofillAgent> binding_; mojo::AssociatedReceiver<autofill::mojom::PasswordAutofillAgent> receiver_{
this};
}; };
std::unique_ptr<KeyedService> CreateTestSyncService( std::unique_ptr<KeyedService> CreateTestSyncService(
...@@ -247,7 +248,7 @@ void ChromePasswordManagerClientTest::SetUp() { ...@@ -247,7 +248,7 @@ void ChromePasswordManagerClientTest::SetUp() {
web_contents()->GetMainFrame()->GetRemoteAssociatedInterfaces(); web_contents()->GetMainFrame()->GetRemoteAssociatedInterfaces();
remote_interfaces->OverrideBinderForTesting( remote_interfaces->OverrideBinderForTesting(
autofill::mojom::PasswordAutofillAgent::Name_, autofill::mojom::PasswordAutofillAgent::Name_,
base::BindRepeating(&FakePasswordAutofillAgent::BindRequest, base::BindRepeating(&FakePasswordAutofillAgent::BindReceiver,
base::Unretained(&fake_agent_))); base::Unretained(&fake_agent_)));
prefs_.registry()->RegisterBooleanPref( prefs_.registry()->RegisterBooleanPref(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment