Multi frequency DTMF support - sender side

webrtc/modules/rtp_rtcp/source/dtmf_queue.h

Index: webrtc/modules/rtp_rtcp/source/dtmf_queue.h
diff --git a/webrtc/modules/rtp_rtcp/source/dtmf_queue.h b/webrtc/modules/rtp_rtcp/source/dtmf_queue.h
index fbfc4efcb14fd461b227bb2804afd06ef5b77191..7a1e6e05a108e3df536edf20fbbf365f88ae2859 100644
--- a/webrtc/modules/rtp_rtcp/source/dtmf_queue.h
+++ b/webrtc/modules/rtp_rtcp/source/dtmf_queue.h
@@ -11,26 +11,30 @@
 #ifndef WEBRTC_MODULES_RTP_RTCP_SOURCE_DTMF_QUEUE_H_
 #define WEBRTC_MODULES_RTP_RTCP_SOURCE_DTMF_QUEUE_H_
 
+#include <list>
+
 #include "webrtc/base/criticalsection.h"
-#include "webrtc/modules/rtp_rtcp/source/rtp_rtcp_config.h"
-#include "webrtc/typedefs.h"
 
 namespace webrtc {
-class DTMFqueue {
+class DTMFQueue {

[hlundin-webrtc, 2016/10/28 08:12:37]

I think it should even be DtmfQueue (style guide says "prefer to capitalize
acronyms as single words").

[the sun, 2016/11/07 13:33:31]

Done.

  public:
-  DTMFqueue();
-  ~DTMFqueue();
+  struct Event {
+    uint16_t duration_ms = 0;
+    uint8_t payload_type = 0;
+    uint8_t key = 0;
+    uint8_t level = 0;
+  };
+
+  DTMFQueue();
+  ~DTMFQueue();
 
-  int32_t AddDTMF(uint8_t dtmf_key, uint16_t len, uint8_t level);
-  int8_t NextDTMF(uint8_t* dtmf_key, uint16_t* len, uint8_t* level);
+  int AddDTMF(const Event& event);

[hlundin-webrtc, 2016/10/28 08:12:36]

AddDtmf

[the sun, 2016/11/07 13:33:31]

Done.

+  int NextDTMF(Event* event);

[hlundin-webrtc, 2016/10/28 08:12:37]

NextDtmf

[the sun, 2016/11/07 13:33:31]

Done.

   bool PendingDTMF();

[hlundin-webrtc, 2016/10/28 08:12:36]

const method.

[hlundin-webrtc, 2016/10/28 08:12:37]

PendingDtmf

[the sun, 2016/11/07 13:33:31]

Done.

[the sun, 2016/11/07 13:33:31]

Done.

 
  private:
   rtc::CriticalSection dtmf_critsect_;
-  uint8_t next_empty_index_;
-  uint8_t dtmf_key_[DTMF_OUTBAND_MAX];
-  uint16_t dtmf_length[DTMF_OUTBAND_MAX];
-  uint8_t dtmf_level_[DTMF_OUTBAND_MAX];
+  std::list<Event> queue_;
 };
 }  // namespace webrtc