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

Clean up InterfacePtrInfo for WilcoDtcSupportdServiceFactory

This CL cleans up InterfacePtrInfo for WilcoDtcSupportdServiceFactory
from WilcoDtcSupportdBridge and uses PendingRemote instead of it.

Bug: 955171
Change-Id: I806411e876144c68594cab6c93f3bb45a74996a3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1935635
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Reviewed-by: default avatarPolina Bondarenko <pbond@chromium.org>
Reviewed-by: default avatarDave Tapuska <dtapuska@chromium.org>
Cr-Commit-Position: refs/heads/master@{#719309}
parent f16a1886
......@@ -20,7 +20,6 @@
#include "chrome/browser/chromeos/wilco_dtc_supportd/wilco_dtc_supportd_notification_controller.h"
#include "chrome/browser/profiles/profile_manager.h"
#include "chromeos/dbus/dbus_thread_manager.h"
#include "mojo/public/cpp/bindings/interface_ptr_info.h"
#include "mojo/public/cpp/bindings/pending_remote.h"
#include "mojo/public/cpp/platform/platform_channel.h"
#include "mojo/public/cpp/platform/platform_channel_endpoint.h"
......@@ -85,7 +84,7 @@ void WilcoDtcSupportdBridgeDelegateImpl::
base::kNullProcessHandle,
channel.TakeLocalEndpoint());
wilco_dtc_supportd_service_factory_mojo_remote->Bind(
mojo::InterfacePtrInfo<
mojo::PendingRemote<
wilco_dtc_supportd::mojom::WilcoDtcSupportdServiceFactory>(
std::move(server_pipe), 0 /* version */));
*remote_endpoint_fd =
......
......@@ -40,7 +40,7 @@ class WilcoDtcSupportdBridge final
// Creates a Mojo invitation that requests the remote implementation of the
// WilcoDtcSupportdServiceFactory interface.
// Returns |wilco_dtc_supportd_service_factory_mojo_ptr| - interface pointer
// Returns |wilco_dtc_supportd_service_factory_mojo_remote| - remote
// that points to the remote implementation of the interface,
// |remote_endpoint_fd| - file descriptor of the remote endpoint to be sent.
virtual void CreateWilcoDtcSupportdServiceFactoryMojoInvitation(
......@@ -129,8 +129,7 @@ class WilcoDtcSupportdBridge final
// Current consecutive connection attempt number.
int connection_attempt_ = 0;
// Interface pointers to the Mojo services exposed by the wilco_dtc_supportd
// daemon.
// Remotes to the Mojo services exposed by the wilco_dtc_supportd daemon.
mojo::Remote<wilco_dtc_supportd::mojom::WilcoDtcSupportdServiceFactory>
wilco_dtc_supportd_service_factory_mojo_remote_;
mojo::Remote<wilco_dtc_supportd::mojom::WilcoDtcSupportdService>
......
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