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

Unified Diff: content/renderer/media_capture_from_element/html_audio_element_capturer_source_unittest.cc

Issue 2815303006: Convert MediaLog from being ref counted to owned by WebMediaPlayer. (Closed)
Patch Set: Rebase. Created 3 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 | « content/renderer/media/webmediaplayer_ms_unittest.cc ('k') | content/renderer/pepper/video_decoder_shim.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/media_capture_from_element/html_audio_element_capturer_source_unittest.cc
diff --git a/content/renderer/media_capture_from_element/html_audio_element_capturer_source_unittest.cc b/content/renderer/media_capture_from_element/html_audio_element_capturer_source_unittest.cc
index 5b4f350d030dcc581abce4ae7a07b23a5be2cb86..444fc3094c787057c70334408b6f679ee8f95528 100644
--- a/content/renderer/media_capture_from_element/html_audio_element_capturer_source_unittest.cc
+++ b/content/renderer/media_capture_from_element/html_audio_element_capturer_source_unittest.cc
@@ -69,7 +69,7 @@ class HTMLAudioElementCapturerSourceTest : public testing::Test {
: fake_callback_(0.1, kAudioTrackSampleRate),
audio_source_(new media::WebAudioSourceProviderImpl(
new media::NullAudioSink(base::ThreadTaskRunnerHandle::Get()),
- new media::MediaLog())) {}
+ &media_log_)) {}
void SetUp() final {
const media::AudioParameters params(
@@ -117,6 +117,7 @@ class HTMLAudioElementCapturerSourceTest : public testing::Test {
blink::WebMediaStreamSource blink_audio_source_;
blink::WebMediaStreamTrack blink_audio_track_;
+ media::MediaLog media_log_;
media::FakeAudioRenderCallback fake_callback_;
scoped_refptr<media::WebAudioSourceProviderImpl> audio_source_;
};
« no previous file with comments | « content/renderer/media/webmediaplayer_ms_unittest.cc ('k') | content/renderer/pepper/video_decoder_shim.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698