Expose flash playback status and volume control to content renderer

content/renderer/pepper/ppb_audio_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/pepper/ppb_audio_impl.h"
 
 #include "base/logging.h"
+#include "content/renderer/pepper/pepper_audio_controller.h"
 #include "content/renderer/pepper/pepper_platform_audio_output.h"
 #include "content/renderer/pepper/pepper_plugin_instance_impl.h"
 #include "content/renderer/pepper/plugin_instance_throttler_impl.h"
 #include "content/renderer/render_frame_impl.h"
-#include "content/renderer/render_view_impl.h"
 #include "media/audio/audio_output_controller.h"
 #include "ppapi/c/pp_completion_callback.h"
 #include "ppapi/c/ppb_audio.h"
 #include "ppapi/c/ppb_audio_config.h"
 #include "ppapi/shared_impl/resource_tracker.h"
 #include "ppapi/thunk/enter.h"
 #include "ppapi/thunk/ppb_audio_config_api.h"
 #include "ppapi/thunk/thunk.h"
 
 using ppapi::PpapiGlobals;
@@ skipping 17 matching lines @@
     plugin_instance->throttler()->AddObserver(this);
   }
 }
 
 PPB_Audio_Impl::~PPB_Audio_Impl() {
   PepperPluginInstanceImpl* instance = static_cast<PepperPluginInstanceImpl*>(
       PepperPluginInstance::Get(pp_instance()));
   if (instance && instance->throttler()) {
     instance->throttler()->RemoveObserver(this);
   }
+  if (instance)
+    instance->audio_controller().RemoveInstance(this);

[mlamouri (slow - plz ping), 2016/06/23 13:04:13]

nit: you could do:
```
if (instance) {
  if (instantce->throttler())
    instance->throttler()->RemoveObserver(this);
  instance->audio_controller().RemoveInstance(this);
}
```

[Zhiqiang Zhang (Slow), 2016/06/23 14:50:29]

Done.

 
   // Calling ShutDown() makes sure StreamCreated cannot be called anymore and
   // releases the audio data associated with the pointer. Note however, that
   // until ShutDown returns, StreamCreated may still be called. This will be
   // OK since we'll just immediately clean up the data it stored later in this
   // destructor.
   if (audio_) {
     audio_->ShutDown();
     audio_ = NULL;
   }
@@ skipping 16 matching lines @@
   // If plugin is in power saver mode, defer audio IPC communication.
   PepperPluginInstanceImpl* instance = static_cast<PepperPluginInstanceImpl*>(
       PepperPluginInstance::Get(pp_instance()));
   if (instance && instance->throttler() &&
       instance->throttler()->power_saver_enabled()) {
     instance->throttler()->NotifyAudioThrottled();
     playback_throttled_ = true;
     return PP_TRUE;
   }
 
+  if (instance)
+    instance->audio_controller().AddInstance(this);
+
   SetStartPlaybackState();
   return PP_FromBool(audio_->StartPlayback());
 }
 
 PP_Bool PPB_Audio_Impl::StopPlayback() {
   if (!audio_)
     return PP_FALSE;
 
   if (playback_throttled_) {
     // If a start playback request is still deferred, we must fulfill it first
     // to shut down the audio thread correctly.
     StartDeferredPlayback();
   }
 
+  PepperPluginInstanceImpl* instance = static_cast<PepperPluginInstanceImpl*>(
+      PepperPluginInstance::Get(pp_instance()));
+  if (instance)
+    instance->audio_controller().RemoveInstance(this);
+
   if (!playing())
     return PP_TRUE;
   if (!audio_->StopPlayback())
     return PP_FALSE;
   SetStopPlaybackState();
 
   return PP_TRUE;
 }
 
 int32_t PPB_Audio_Impl::Open(PP_Resource config,
@@ skipping 56 matching lines @@
       !instance->throttler()->power_saver_enabled()) {
     // If we have become unthrottled, and we have a pending playback, start it.
     StartDeferredPlayback();
   }
 }
 
 void PPB_Audio_Impl::StartDeferredPlayback() {
   DCHECK(playback_throttled_);
   playback_throttled_ = false;
 
+  PepperPluginInstanceImpl* instance = static_cast<PepperPluginInstanceImpl*>(
+      PepperPluginInstance::Get(pp_instance()));
+  if (instance)
+    instance->audio_controller().RemoveInstance(this);

[mlamouri (slow - plz ping), 2016/06/23 13:04:13]

Shouldn't that be AddInstance?

[Zhiqiang Zhang (Slow), 2016/06/23 14:50:29]

Thanks! Done.

+
   SetStartPlaybackState();
   audio_->StartPlayback();
 }
 
+void PPB_Audio_Impl::SetVolume(double volume) {
+  if (audio_)
+    audio_->SetVolume(volume);
+}
 }  // namespace content