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

Unified Diff: media/blink/webaudiosourceprovider_impl.h

Issue 1599533003: MediaCaptureFromElement: add support for audio captureStream(). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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.h
diff --git a/media/blink/webaudiosourceprovider_impl.h b/media/blink/webaudiosourceprovider_impl.h
index 6bf3b0aa9e379b9811e06598a53c97186fb74b47..ba2f7a85da5f140431ef6c7156413f9f7274c544 100644
--- a/media/blink/webaudiosourceprovider_impl.h
+++ b/media/blink/webaudiosourceprovider_impl.h
@@ -69,13 +69,17 @@ class MEDIA_BLINK_EXPORT WebAudioSourceProviderImpl
const OutputDeviceStatusCB& callback) override;
// These methods allow a client to get a copy of the rendered audio.
- void SetCopyAudioCallback(const CopyAudioCB& callback);
- void ClearCopyAudioCallback();
+ virtual void SetCopyAudioCallback(const CopyAudioCB& callback);
miu 2016/05/13 23:40:35 Why are these methods being made virtual?
mcasas 2016/05/14 02:23:47 For MockWebAudioSourceProvider in html_audio_elem
+ virtual void ClearCopyAudioCallback();
+ base::WeakPtr<WebAudioSourceProviderImpl> AsWeakPtr() {
+ return weak_factory_.GetWeakPtr();
+ }
- private:
+ protected:
friend class WebAudioSourceProviderImplTest;
~WebAudioSourceProviderImpl() override;
+ private:
// Calls setFormat() on |client_| from the Blink renderer thread.
void OnSetFormat();

Powered by Google App Engine
This is Rietveld 408576698