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

Issue 9479012: Merge 123292 - Increase the buffer size in AudioRendererImpl to fix muted playback rate (Closed)

Created:
8 years, 10 months ago by vrk (LEFT CHROMIUM)
Modified:
8 years, 10 months ago
Reviewers:
Victoria
CC:
chromium-reviews, hclam+watch_chromium.org, ddorwin+watch_chromium.org, fischman+watch_chromium.org, jam, acolwell+watch_chromium.org, annacc+watch_chromium.org, joi+watch-content_chromium.org, darin-cc_chromium.org, vrk (LEFT CHROMIUM), scherkus (not reviewing), ihf+watch_chromium.org
Visibility:
Public.

Description

Merge 123292 - Increase the buffer size in AudioRendererImpl to fix muted playback rate This CL temporarily removes GetBufferSizeForSampleRate() and uses SelectSamplesPerPacket() to determine the size of samples_per_packet in AudioDevice. This restores the buffer size that AudioRendererImpl used prior to r113821. SelectSamplesPerPacket() calculates a much larger samples_per_packet value than GetBufferSizeForSampleRate(), so it fixes the issue described in 108239. This CL also does some mild refactoring to move audio_common.cc into audio_util.cc. BUG=108239 TEST=media_unittests,content_unittests, manual testing of different playback rates Review URL: https://chromiumcodereview.appspot.com/9416085 TBR=vrk@google.com Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=123774

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+38 lines, -77 lines) Patch
D content/browser/renderer_host/media/audio_common.h View 1 chunk +0 lines, -17 lines 0 comments Download
D content/browser/renderer_host/media/audio_common.cc View 1 chunk +0 lines, -33 lines 0 comments Download
M content/browser/renderer_host/media/audio_input_renderer_host.cc View 2 chunks +3 lines, -2 lines 0 comments Download
M content/browser/renderer_host/media/audio_renderer_host.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M content/content_browser.gypi View 1 chunk +0 lines, -2 lines 0 comments Download
M content/renderer/media/audio_renderer_impl.cc View 2 chunks +1 line, -21 lines 0 comments Download
M media/audio/audio_util.h View 2 chunks +5 lines, -0 lines 0 comments Download
M media/audio/audio_util.cc View 2 chunks +27 lines, -0 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
vrk (LEFT CHROMIUM)
8 years, 10 months ago (2012-02-27 19:09:12 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698