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

Unified Diff: webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.h

Issue 2383103002: RTPReceiverAudio: Removed frequency from CNGPayloadType and cleaned up (Closed)
Patch Set: Rebase Created 4 years, 2 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 | « no previous file | webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.h
diff --git a/webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.h b/webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.h
index d5d89bae2d4c2f233953cef1033324662bf7a45e..29a4c7c3912d614397b43a4c53f83bcd05fe777e 100644
--- a/webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.h
+++ b/webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.h
@@ -36,16 +36,13 @@ class RTPReceiverAudio : public RTPReceiverStrategy,
// Is forwarding of outband telephone events turned on/off?
bool TelephoneEventForwardToDecoder() const override;
- // Is TelephoneEvent configured with payload type payload_type
+ // Is TelephoneEvent configured with |payload_type|.
bool TelephoneEventPayloadType(const int8_t payload_type) const override;
TelephoneEventHandler* GetTelephoneEventHandler() override { return this; }
- // Returns true if CNG is configured with payload type payload_type. If so,
- // the frequency and cng_payload_type_has_changed are filled in.
- bool CNGPayloadType(const int8_t payload_type,
- uint32_t* frequency,
- bool* cng_payload_type_has_changed);
+ // Returns true if CNG is configured with |payload_type|.
+ bool CNGPayloadType(const int8_t payload_type);
int32_t ParseRtpPacket(WebRtcRTPHeader* rtp_header,
const PayloadUnion& specific_payload,
@@ -55,8 +52,6 @@ class RTPReceiverAudio : public RTPReceiverStrategy,
int64_t timestamp_ms,
bool is_first_packet) override;
- int GetPayloadTypeFrequency() const override;
-
RTPAliveType ProcessDeadOrAlive(uint16_t last_payload_length) const override;
bool ShouldReportCsrcChanges(uint8_t payload_type) const override;
@@ -107,12 +102,6 @@ class RTPReceiverAudio : public RTPReceiverStrategy,
int8_t cng_wb_payload_type_;
int8_t cng_swb_payload_type_;
int8_t cng_fb_payload_type_;
- int8_t cng_payload_type_;
-
- // G722 is special since it use the wrong number of RTP samples in timestamp
- // VS. number of samples in the frame
- int8_t g722_payload_type_;
- bool last_received_g722_;
uint8_t num_energy_;
uint8_t current_remote_energy_[kRtpCsrcSize];
« no previous file with comments | « no previous file | webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698