User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:18.0) Gecko/20100101 Firefox/18.0 (Beta/Release) Build ID: 20130116073211 Steps to reproduce: The problem appears in changeset 138350:18467a85acf6. I have attached a simple one-line patch that fixes it. In method WebRtcNetEQ_DtmfInsertEvent() in media/webrtc/trunk/webrtc/modules/audio_coding/neteq/dtmf_buffer.c, the loop in line 121 keeps overriding "position" with "i" when "((DTMFdec_inst->EventQueue[i] == value) && (!DTMFdec_inst->EventQueueEnded[i] || endEvent))" is true. Therefore, only the last written value is visible out of the loop and all the other writes and iterations are not necessary. The patch iterates from the end of "i" and breaks the first time when "position" is set.

Please see https://developer.mozilla.org/en-US/docs/Developer_Guide/How_to_Submit_a_Patch > The problem appears in changeset 138350:18467a85acf6. https://hg.mozilla.org/mozilla-central/rev/18467a85acf6 ? > In method WebRtcNetEQ_DtmfInsertEvent() in media/webrtc/trunk/webrtc/modules/audio_coding/neteq/dtmf_buffer.c, the loop in line 121 https://hg.mozilla.org/mozilla-central/file/18467a85acf6/media/webrtc/trunk/webrtc/modules/audio_coding/neteq/dtmf_buffer.c#l120

This is a minor perf issue in the upstream library. As it's not a significant flaw or problem in Mozilla, please file an Issue at http://code.google.com/p/webrtc/issues/list See http://www.webrtc.org/reference/contributing for how to prepare a CL for review and upload it there; we'll pick up the fix when it gets to a stable branch of webrtc.org. Thanks!