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

Side by Side Diff: media/blink/webaudiosourceprovider_impl.cc

Issue 2004963002: Rename delay_milliseconds to frames_delayed. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase, simplify 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "media/blink/webaudiosourceprovider_impl.h" 5 #include "media/blink/webaudiosourceprovider_impl.h"
6 6
7 #include <vector> 7 #include <vector>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/callback_helpers.h" 10 #include "base/callback_helpers.h"
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
59 int sample_rate) 59 int sample_rate)
60 : renderer_(renderer), channels_(channels), sample_rate_(sample_rate) { 60 : renderer_(renderer), channels_(channels), sample_rate_(sample_rate) {
61 DCHECK(renderer_); 61 DCHECK(renderer_);
62 } 62 }
63 ~TeeFilter() override {} 63 ~TeeFilter() override {}
64 64
65 // AudioRendererSink::RenderCallback implementation. 65 // AudioRendererSink::RenderCallback implementation.
66 // These are forwarders to |renderer_| and are here to allow for a client to 66 // These are forwarders to |renderer_| and are here to allow for a client to
67 // get a copy of the rendered audio by SetCopyAudioCallback(). 67 // get a copy of the rendered audio by SetCopyAudioCallback().
68 int Render(AudioBus* audio_bus, 68 int Render(AudioBus* audio_bus,
69 uint32_t delay_milliseconds, 69 uint32_t frames_delayed,
70 uint32_t frames_skipped) override; 70 uint32_t frames_skipped) override;
71 void OnRenderError() override; 71 void OnRenderError() override;
72 72
73 int channels() const { return channels_; } 73 int channels() const { return channels_; }
74 int sample_rate() const { return sample_rate_; } 74 int sample_rate() const { return sample_rate_; }
75 void set_copy_audio_bus_callback( 75 void set_copy_audio_bus_callback(
76 const WebAudioSourceProviderImpl::CopyAudioCB& callback) { 76 const WebAudioSourceProviderImpl::CopyAudioCB& callback) {
77 copy_audio_bus_callback_ = callback; 77 copy_audio_bus_callback_ = callback;
78 } 78 }
79 79
(...skipping 169 matching lines...) Expand 10 before | Expand all | Expand 10 after
249 249
250 // Inform Blink about the audio stream format. 250 // Inform Blink about the audio stream format.
251 client_->setFormat(tee_filter_->channels(), tee_filter_->sample_rate()); 251 client_->setFormat(tee_filter_->channels(), tee_filter_->sample_rate());
252 } 252 }
253 253
254 int WebAudioSourceProviderImpl::RenderForTesting(AudioBus* audio_bus) { 254 int WebAudioSourceProviderImpl::RenderForTesting(AudioBus* audio_bus) {
255 return tee_filter_->Render(audio_bus, 0, 0); 255 return tee_filter_->Render(audio_bus, 0, 0);
256 } 256 }
257 257
258 int WebAudioSourceProviderImpl::TeeFilter::Render(AudioBus* audio_bus, 258 int WebAudioSourceProviderImpl::TeeFilter::Render(AudioBus* audio_bus,
259 uint32_t delay_milliseconds, 259 uint32_t frames_delayed,
260 uint32_t frames_skipped) { 260 uint32_t frames_skipped) {
261 const int num_rendered_frames = 261 const int num_rendered_frames =
262 renderer_->Render(audio_bus, delay_milliseconds, frames_skipped); 262 renderer_->Render(audio_bus, frames_delayed, frames_skipped);
263 263
264 if (!copy_audio_bus_callback_.is_null()) { 264 if (!copy_audio_bus_callback_.is_null()) {
265 std::unique_ptr<AudioBus> bus_copy = 265 std::unique_ptr<AudioBus> bus_copy =
266 AudioBus::Create(audio_bus->channels(), audio_bus->frames()); 266 AudioBus::Create(audio_bus->channels(), audio_bus->frames());
267 audio_bus->CopyTo(bus_copy.get()); 267 audio_bus->CopyTo(bus_copy.get());
268 copy_audio_bus_callback_.Run(std::move(bus_copy), delay_milliseconds, 268 copy_audio_bus_callback_.Run(std::move(bus_copy), frames_delayed,
o1ka 2016/05/30 08:07:41 HtmlAudioElementCapturerSource::OnAudioBus() (http
269 sample_rate_); 269 sample_rate_);
270 } 270 }
271 271
272 return num_rendered_frames; 272 return num_rendered_frames;
273 } 273 }
274 274
275 void WebAudioSourceProviderImpl::TeeFilter::OnRenderError() { 275 void WebAudioSourceProviderImpl::TeeFilter::OnRenderError() {
276 renderer_->OnRenderError(); 276 renderer_->OnRenderError();
277 } 277 }
278 278
279 } // namespace media 279 } // namespace media
OLDNEW
« no previous file with comments | « media/blink/webaudiosourceprovider_impl.h ('k') | media/blink/webaudiosourceprovider_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698