changed MSAP to work with 96000KHZ

changed MSAP to work with 96000KHZ.
The problem why it did not work is that the FIFO used in MSAP has a size smaller than 10ms buffer before, so it crashed before webrtc::AudioProcessing consumes it.

FYI, previously the APM in Chrome was only enabled for finch experiment on Canary, so the problem does not occur in stable and beta. And I recently enabled it as default.


BUG=378895
TEST= use 96000KHz as sample rate, go to apprtc.appspot.com/?&debug=loopback to make a call.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=274216

[no longer working on chromium, 2014-05-30 15:07:30]

Henrik, could you please review this small fix?

Thanks
SX

[no longer working on chromium, 2014-05-30 15:52:37]

Added unittest to verify MSAP works with all supported sample rates. lg now?

[henrika (OOO until Aug 14), 2014-06-02 08:23:40]

LGTM w/ nit.

https://codereview.chromium.org/308643004/diff/40001/content/renderer/media/m...
File content/renderer/media/media_stream_audio_processor.cc (right):

https://codereview.chromium.org/308643004/diff/40001/content/renderer/media/m...
content/renderer/media/media_stream_audio_processor.cc:74: // Also, FIFO needs
to have enough space to store pre-processed data
Is it possible to set the buffer size once only and not twice?

[no longer working on chromium, 2014-06-02 09:15:07]

Thanks Henrik. Landing soon.

https://codereview.chromium.org/308643004/diff/40001/content/renderer/media/m...
File content/renderer/media/media_stream_audio_processor.cc (right):

https://codereview.chromium.org/308643004/diff/40001/content/renderer/media/m...
content/renderer/media/media_stream_audio_processor.cc:74: // Also, FIFO needs
to have enough space to store pre-processed data
On 2014/06/02 08:23:41, henrika wrote:
> Is it possible to set the buffer size once only and not twice?

Yes, I added a max_frame_size as a temp variable to make it possible.

[no longer working on chromium, 2014-06-02 09:15:11]

The CQ bit was checked by xians@chromium.org

[commit-bot: I haz the power, 2014-06-02 09:15:46]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/xians@chromium.org/308643004/60001

[commit-bot: I haz the power, 2014-06-02 12:40:14]

Change committed as 274216