Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(713)

Unified Diff: chrome/renderer/media/cast_transport_ipc.h

Issue 1878883003: Refactor: simplify interface of SenderRtcpSession and CastTransport. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/media/cast_transport_host_filter.cc ('k') | chrome/renderer/media/cast_transport_ipc.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/renderer/media/cast_transport_ipc.h
diff --git a/chrome/renderer/media/cast_transport_ipc.h b/chrome/renderer/media/cast_transport_ipc.h
index bc9c809af5bc74a34e17aa33cc3b086ce10a8f67..d622d7478f4c6e57561e994f25cdd85c6496d051 100644
--- a/chrome/renderer/media/cast_transport_ipc.h
+++ b/chrome/renderer/media/cast_transport_ipc.h
@@ -34,14 +34,10 @@ class CastTransportIPC : public media::cast::CastTransport {
// media::cast::CastTransport implementation.
void InitializeAudio(
const media::cast::CastTransportRtpConfig& config,
- const media::cast::RtcpCastMessageCallback& cast_message_cb,
- const media::cast::RtcpRttCallback& rtt_cb,
- const media::cast::RtcpPliCallback& pli_cb) override;
+ std::unique_ptr<media::cast::RtcpObserver> rtcp_observer) override;
void InitializeVideo(
const media::cast::CastTransportRtpConfig& config,
- const media::cast::RtcpCastMessageCallback& cast_message_cb,
- const media::cast::RtcpRttCallback& rtt_cb,
- const media::cast::RtcpPliCallback& pli_cb) override;
+ std::unique_ptr<media::cast::RtcpObserver> rtcp_observer) override;
void InsertFrame(uint32_t ssrc,
const media::cast::EncodedFrame& frame) override;
void SendSenderReport(
@@ -77,23 +73,14 @@ class CastTransportIPC : public media::cast::CastTransport {
void OnReceivedPacket(const media::cast::Packet& packet);
private:
- struct ClientCallbacks {
- ClientCallbacks();
- ClientCallbacks(const ClientCallbacks& other);
- ~ClientCallbacks();
-
- media::cast::RtcpCastMessageCallback cast_message_cb;
- media::cast::RtcpRttCallback rtt_cb;
- media::cast::RtcpPliCallback pli_cb;
- };
-
void Send(IPC::Message* message);
int32_t channel_id_;
media::cast::PacketReceiverCallback packet_callback_;
media::cast::CastTransportStatusCallback status_callback_;
media::cast::BulkRawEventsCallback raw_events_callback_;
- typedef std::map<uint32_t, ClientCallbacks> ClientMap;
+ using ClientMap =
+ std::map<uint32_t, std::unique_ptr<media::cast::RtcpObserver>>;
ClientMap clients_;
DISALLOW_COPY_AND_ASSIGN(CastTransportIPC);
« no previous file with comments | « chrome/browser/media/cast_transport_host_filter.cc ('k') | chrome/renderer/media/cast_transport_ipc.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698