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

Unified Diff: media/blink/webaudiosourceprovider_impl.cc

Issue 2038053002: Change audio render thread checking to use new AudioRendererSink::BelongsToRendererThread() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Using the new BelongsTo...() function for thread checking. Created 4 years, 6 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: media/blink/webaudiosourceprovider_impl.cc
diff --git a/media/blink/webaudiosourceprovider_impl.cc b/media/blink/webaudiosourceprovider_impl.cc
index 6d998e5355a4f6521ae3de6f9862d34f3356fc6a..7690f4c430bbc2162e78a3f0427002997af841fc 100644
--- a/media/blink/webaudiosourceprovider_impl.cc
+++ b/media/blink/webaudiosourceprovider_impl.cc
@@ -163,6 +163,19 @@ void WebAudioSourceProviderImpl::provideInput(
bus_wrapper_->Scale(volume_);
}
+void WebAudioSourceProviderImpl::Initialize(const AudioParameters& params,
+ RenderCallback* renderer) {
+ base::AutoLock auto_lock(sink_lock_);
+ DCHECK_EQ(state_, kStopped);
+
+ tee_filter_->Initialize(renderer, params.channels(), params.sample_rate());
+
+ sink_->Initialize(params, tee_filter_.get());
+
+ if (!set_format_cb_.is_null())
+ base::ResetAndReturn(&set_format_cb_).Run();
+}
+
void WebAudioSourceProviderImpl::Start() {
base::AutoLock auto_lock(sink_lock_);
DCHECK(tee_filter_);
@@ -208,6 +221,11 @@ media::OutputDeviceInfo WebAudioSourceProviderImpl::GetOutputDeviceInfo() {
return sink_->GetOutputDeviceInfo();
}
+bool WebAudioSourceProviderImpl::BelongsToRenderingThread() {
+ NOTIMPLEMENTED();
+ return false;
+}
+
void WebAudioSourceProviderImpl::SwitchOutputDevice(
const std::string& device_id,
const url::Origin& security_origin,
@@ -219,19 +237,6 @@ void WebAudioSourceProviderImpl::SwitchOutputDevice(
sink_->SwitchOutputDevice(device_id, security_origin, callback);
}
-void WebAudioSourceProviderImpl::Initialize(const AudioParameters& params,
- RenderCallback* renderer) {
- base::AutoLock auto_lock(sink_lock_);
- DCHECK_EQ(state_, kStopped);
-
- tee_filter_->Initialize(renderer, params.channels(), params.sample_rate());
-
- sink_->Initialize(params, tee_filter_.get());
-
- if (!set_format_cb_.is_null())
- base::ResetAndReturn(&set_format_cb_).Run();
-}
-
void WebAudioSourceProviderImpl::SetCopyAudioCallback(
const CopyAudioCB& callback) {
DCHECK(!callback.is_null());
« media/base/audio_renderer_mixer_input.cc ('K') | « media/blink/webaudiosourceprovider_impl.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698