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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: ppapi/proxy/ppb_audio_proxy.cc

Issue 240523002: Add test to make sure if PPB_Audio_Shared::StartThread() works. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 8 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
« no previous file with comments | « ppapi/native_client/src/untrusted/irt_stub/thread_creator.h ('k') | ppapi/shared_impl/ppb_audio_shared.h » ('j') | ppapi/tests/test_audio.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/proxy/ppb_audio_proxy.cc
diff --git a/ppapi/proxy/ppb_audio_proxy.cc b/ppapi/proxy/ppb_audio_proxy.cc
index 58f5abc0d4141f747e3a2845dd1682ccfc4388ff..3eb8dc46059868af44d11223f556de9da2a7e862 100644
--- a/ppapi/proxy/ppb_audio_proxy.cc
+++ b/ppapi/proxy/ppb_audio_proxy.cc
@@ -95,7 +95,8 @@ PP_Resource Audio::GetCurrentConfig() {
PP_Bool Audio::StartPlayback() {
if (playing())
return PP_TRUE;
- SetStartPlaybackState();
+ if (!SetStartPlaybackState())
+ return PP_FALSE;
PluginDispatcher::GetForResource(this)->Send(
new PpapiHostMsg_PPBAudio_StartOrStop(
API_ID_PPB_AUDIO, host_resource(), true));
« no previous file with comments | « ppapi/native_client/src/untrusted/irt_stub/thread_creator.h ('k') | ppapi/shared_impl/ppb_audio_shared.h » ('j') | ppapi/tests/test_audio.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698