Commit 3ee9d815 authored by Derek Cheng's avatar Derek Cheng Committed by Commit Bot

[Media Router] Remove SendMessage from PresentationServiceDelegate

It is no longer used as a result of sending presentation connection
messages via the PresentationConnection interface.

Bug: 687011
Change-Id: Ic537619a08865d33a79f3dffa29d82473d1b4e5e
Reviewed-on: https://chromium-review.googlesource.com/580049Reviewed-by: default avatarAvi Drissman <avi@chromium.org>
Commit-Queue: Derek Cheng <imcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488510}
parent b9f97f80
......@@ -597,32 +597,6 @@ void PresentationServiceDelegateImpl::Terminate(
RemovePresentation(rfh_id, presentation_id);
}
void PresentationServiceDelegateImpl::SendMessage(
int render_process_id,
int render_frame_id,
const content::PresentationInfo& presentation_info,
content::PresentationConnectionMessage message,
SendMessageCallback send_message_cb) {
auto route_id =
GetRouteId(RenderFrameHostId(render_process_id, render_frame_id),
presentation_info.presentation_id);
if (route_id.empty()) {
DVLOG(1) << "No active route for " << presentation_info.presentation_id;
std::move(send_message_cb).Run(false);
return;
}
if (message.is_binary()) {
router_->SendRouteBinaryMessage(
route_id,
base::MakeUnique<std::vector<uint8_t>>(std::move(message.data.value())),
std::move(send_message_cb));
} else {
router_->SendRouteMessage(route_id, message.message.value(),
std::move(send_message_cb));
}
}
void PresentationServiceDelegateImpl::ListenForConnectionStateChange(
int render_process_id,
int render_frame_id,
......
......@@ -27,7 +27,6 @@
namespace content {
class PresentationScreenAvailabilityListener;
class WebContents;
struct PresentationConnectionMessage;
struct PresentationInfo;
} // namespace content
......@@ -115,11 +114,6 @@ class PresentationServiceDelegateImpl
void Terminate(int render_process_id,
int render_frame_id,
const std::string& presentation_id) override;
void SendMessage(int render_process_id,
int render_frame_id,
const content::PresentationInfo& presentation_info,
content::PresentationConnectionMessage message,
SendMessageCallback send_message_cb) override;
void ListenForConnectionStateChange(
int render_process_id,
int render_frame_id,
......
......@@ -178,18 +178,6 @@ class CONTENT_EXPORT ControllerPresentationServiceDelegate
int render_frame_id,
const std::string& presentation_id) = 0;
// Sends a message (string or binary data) to a presentation.
// |render_process_id|, |render_frame_id|: ID of originating frame.
// |presentation_info|: The presentation to send the message to.
// |message|: The message to send. The embedder takes ownership of |message|.
// Must not be null.
// |send_message_cb|: Invoked after handling the send message request.
virtual void SendMessage(int render_process_id,
int render_frame_id,
const content::PresentationInfo& presentation_info,
PresentationConnectionMessage message,
SendMessageCallback send_message_cb) = 0;
// Continuously listen for state changes for a PresentationConnection in a
// frame.
// |render_process_id|, |render_frame_id|: ID of frame.
......
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