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

Side by Side Diff: content/renderer/media/media_stream_audio_track.cc

Issue 1721273002: MediaStream audio object graph untangling and clean-ups. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: REBASE Created 4 years, 9 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "content/renderer/media/media_stream_audio_track.h" 5 #include "content/renderer/media/media_stream_audio_track.h"
6 6
7 #include "base/callback_helpers.h"
7 #include "base/logging.h" 8 #include "base/logging.h"
8 #include "third_party/WebKit/public/platform/WebMediaStreamSource.h" 9 #include "third_party/WebKit/public/platform/WebMediaStreamSource.h"
9 #include "third_party/webrtc/api/mediastreaminterface.h" 10 #include "third_party/webrtc/api/mediastreaminterface.h"
10 11
11 namespace content { 12 namespace content {
12 13
13 MediaStreamAudioTrack::MediaStreamAudioTrack(bool is_local_track) 14 MediaStreamAudioTrack::MediaStreamAudioTrack(bool is_local_track)
14 : MediaStreamTrack(is_local_track) { 15 : MediaStreamTrack(is_local_track) {
16 DVLOG(1) << "MediaStreamAudioTrack::MediaStreamAudioTrack(is a "
17 << (is_local_track ? "local" : "remote") << " track)";
15 } 18 }
16 19
17 MediaStreamAudioTrack::~MediaStreamAudioTrack() { 20 MediaStreamAudioTrack::~MediaStreamAudioTrack() {
21 DCHECK(main_render_thread_checker_.CalledOnValidThread());
22 DVLOG(1) << "MediaStreamAudioTrack::~MediaStreamAudioTrack()";
23 DCHECK(stop_callback_.is_null())
24 << "BUG: Subclass must ensure Stop() is called.";
18 } 25 }
19 26
20 // static 27 // static
21 MediaStreamAudioTrack* MediaStreamAudioTrack::GetTrack( 28 MediaStreamAudioTrack* MediaStreamAudioTrack::From(
22 const blink::WebMediaStreamTrack& track) { 29 const blink::WebMediaStreamTrack& track) {
23 if (track.isNull() || 30 if (track.isNull() ||
24 track.source().getType() != blink::WebMediaStreamSource::TypeAudio) { 31 track.source().getType() != blink::WebMediaStreamSource::TypeAudio) {
25 return nullptr; 32 return nullptr;
26 } 33 }
27 return static_cast<MediaStreamAudioTrack*>(track.extraData()); 34 return static_cast<MediaStreamAudioTrack*>(track.extraData());
28 } 35 }
29 36
37 void MediaStreamAudioTrack::Start(const base::Closure& stop_callback) {
38 DCHECK(main_render_thread_checker_.CalledOnValidThread());
39 DCHECK(!stop_callback.is_null());
40 DCHECK(stop_callback_.is_null());
41 DVLOG(1) << "MediaStreamAudioTrack::Start()";
42 stop_callback_ = stop_callback;
43 }
44
45 void MediaStreamAudioTrack::Stop() {
46 DCHECK(main_render_thread_checker_.CalledOnValidThread());
47 DVLOG(1) << "MediaStreamAudioTrack::Stop()";
48 if (!stop_callback_.is_null())
49 base::ResetAndReturn(&stop_callback_).Run();
50 OnStop();
51 }
52
53 void MediaStreamAudioTrack::OnStop() {}
54
30 webrtc::AudioTrackInterface* MediaStreamAudioTrack::GetAudioAdapter() { 55 webrtc::AudioTrackInterface* MediaStreamAudioTrack::GetAudioAdapter() {
31 NOTREACHED(); 56 NOTREACHED();
32 return nullptr; 57 return nullptr;
33 } 58 }
34 59
35 } // namespace content 60 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/media/media_stream_audio_track.h ('k') | content/renderer/media/media_stream_center.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698