Fix bug in DTMF generation where events with level > 36 would be ignored.

webrtc/modules/audio_coding/neteq/dtmf_buffer.cc

Index: webrtc/modules/audio_coding/neteq/dtmf_buffer.cc
diff --git a/webrtc/modules/audio_coding/neteq/dtmf_buffer.cc b/webrtc/modules/audio_coding/neteq/dtmf_buffer.cc
index 779d1d340b46559169f4ab2140e6710ae36cddf2..3685fc1ef4a8b64d334384e8b7f633fa334f5a07 100644
--- a/webrtc/modules/audio_coding/neteq/dtmf_buffer.cc
+++ b/webrtc/modules/audio_coding/neteq/dtmf_buffer.cc
@@ -99,7 +99,7 @@ int DtmfBuffer::ParseEvent(uint32_t rtp_timestamp,
 // existing one.
 int DtmfBuffer::InsertEvent(const DtmfEvent& event) {
   if (event.event_no < 0 || event.event_no > 15 ||
-      event.volume < 0 || event.volume > 36 ||
+      event.volume < 0 || event.volume > 63 ||

[hlundin-webrtc, 2016/10/11 09:08:36]

This is a bug that was introduced by yours truly in the NetEq3->NetEq4 rewrite.
The old implementation simply discarded low-amplitude events, without throwing
an error. You could argue that it is bad to do that too.

[the sun, 2016/10/11 09:44:53]

Acknowledged.

       event.duration <= 0 || event.duration > 65535) {
     LOG(LS_WARNING) << "InsertEvent invalid parameters";
     return kInvalidEventParameters;