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

Unified Diff: ppapi/native_client/src/shared/ppapi_proxy/plugin_ppb_audio_config.cc

Issue 9129007: Work on improving PpbAudioConfig:RecommendSampleFrameCount (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 8 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ppapi/native_client/src/shared/ppapi_proxy/plugin_ppb_audio_config.cc
===================================================================
--- ppapi/native_client/src/shared/ppapi_proxy/plugin_ppb_audio_config.cc (revision 118765)
+++ ppapi/native_client/src/shared/ppapi_proxy/plugin_ppb_audio_config.cc (working copy)
@@ -51,13 +51,15 @@
return 0;
}
-uint32_t RecommendSampleFrameCount(PP_AudioSampleRate sample_rate,
+uint32_t RecommendSampleFrameCount(PP_Instance instance,
+ PP_AudioSampleRate sample_rate,
uint32_t request_sample_frame_count) {
DebugPrintf("PPB_AudioConfig::RecommendSampleFrameCount");
int32_t out_sample_frame_count;
NaClSrpcError srpc_result =
PpbAudioConfigRpcClient::PPB_AudioConfig_RecommendSampleFrameCount(
GetMainSrpcChannel(),
+ instance,
static_cast<int32_t>(sample_rate),
static_cast<int32_t>(request_sample_frame_count),
&out_sample_frame_count);
@@ -106,6 +108,23 @@
}
return kInvalidResourceId;
}
+
+PP_AudioSampleRate RecommendSampleRate(PP_Instance instance) {
+ DebugPrintf("PPB_AudioConfig::RecommendSampleRate");
+ int32_t out_sample_rate;
+ NaClSrpcError srpc_result =
+ PpbAudioConfigRpcClient::PPB_AudioConfig_RecommendSampleRate(
+ GetMainSrpcChannel(),
+ instance,
+ &out_sample_rate);
+ DebugPrintf("PPB_AudioConfig::RecommendSampleRate: %s\n",
+ NaClSrpcErrorString(srpc_result));
+ if (NACL_SRPC_RESULT_OK == srpc_result) {
+ return static_cast<PP_AudioSampleRate>(out_sample_rate);
+ }
+ return PP_AUDIOSAMPLERATE_NONE;
+}
+
} // namespace
const PPB_AudioConfig* PluginAudioConfig::GetInterface() {
@@ -115,6 +134,7 @@
IsAudioConfig,
GetSampleRate,
GetSampleFrameCount,
+ RecommendSampleRate
};
return &audio_config_interface;
}

Powered by Google App Engine
This is Rietveld 408576698