Chromium Code Reviews| Index: ppapi/shared_impl/ppb_audio_config_shared.cc |
| =================================================================== |
| --- ppapi/shared_impl/ppb_audio_config_shared.cc (revision 221928) |
| +++ ppapi/shared_impl/ppb_audio_config_shared.cc (working copy) |
| @@ -81,6 +81,17 @@ |
| // care when modifying these values as they impact a large number of users. |
| // TODO(dalecurtis): Land jitter test and add documentation for updating this. |
| + // Should track the value reported by XP and ALSA backends. |
| + const uint32_t kHighLatencySampleFrameCount = 2048; |
| + |
| +#if defined(OS_CHROMEOS) && defined(ARCH_CPU_ARM_FAMILY) |
|
DaleCurtis
2013/09/12 18:12:53
might need #include build/build_config.h
ilja
2013/09/13 20:22:44
Done.
|
| + // TODO(ihf): Remove this once ARM Chromebooks support low latency audio. For |
| + // now we classify them as high latency. See crbug.com/289770. |
| + return CalculateMultipleOfSampleFrameCount( |
| + sample_frame_count, |
| + kHighLatencySampleFrameCount); |
| +#endif // defined(OS_CHROMEOS) && defined(ARCH_CPU_ARM_FAMILY) |
| + |
| // If client is using same sample rate as audio hardware, then recommend a |
| // multiple of the audio hardware's sample frame count. |
| if (hardware_sample_rate == sample_rate) { |
| @@ -88,9 +99,6 @@ |
| hardware_sample_frame_count, sample_frame_count); |
| } |
| - // Should track the value reported by XP and ALSA backends. |
| - const uint32_t kHighLatencySampleFrameCount = 2048; |
| - |
| // If the hardware requires a high latency buffer or we're at a low sample |
| // rate w/ a buffer that's larger than 10ms, choose the nearest multiple of |
| // the high latency sample frame count. An example of too low and too large |