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

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

Issue 1780653002: Revert of MediaStream audio object graph untangling and clean-ups. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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"
8 #include "base/logging.h" 7 #include "base/logging.h"
9 #include "third_party/WebKit/public/platform/WebMediaStreamSource.h" 8 #include "third_party/WebKit/public/platform/WebMediaStreamSource.h"
10 #include "third_party/webrtc/api/mediastreaminterface.h" 9 #include "third_party/webrtc/api/mediastreaminterface.h"
11 10
12 namespace content { 11 namespace content {
13 12
14 MediaStreamAudioTrack::MediaStreamAudioTrack(bool is_local_track) 13 MediaStreamAudioTrack::MediaStreamAudioTrack(bool is_local_track)
15 : MediaStreamTrack(is_local_track) { 14 : MediaStreamTrack(is_local_track) {
16 DVLOG(1) << "MediaStreamAudioTrack::MediaStreamAudioTrack(is a "
17 << (is_local_track ? "local" : "remote") << " track)";
18 } 15 }
19 16
20 MediaStreamAudioTrack::~MediaStreamAudioTrack() { 17 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.";
25 } 18 }
26 19
27 // static 20 // static
28 MediaStreamAudioTrack* MediaStreamAudioTrack::From( 21 MediaStreamAudioTrack* MediaStreamAudioTrack::GetTrack(
29 const blink::WebMediaStreamTrack& track) { 22 const blink::WebMediaStreamTrack& track) {
30 if (track.isNull() || 23 if (track.isNull() ||
31 track.source().getType() != blink::WebMediaStreamSource::TypeAudio) { 24 track.source().getType() != blink::WebMediaStreamSource::TypeAudio) {
32 return nullptr; 25 return nullptr;
33 } 26 }
34 return static_cast<MediaStreamAudioTrack*>(track.extraData()); 27 return static_cast<MediaStreamAudioTrack*>(track.extraData());
35 } 28 }
36 29
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
55 webrtc::AudioTrackInterface* MediaStreamAudioTrack::GetAudioAdapter() { 30 webrtc::AudioTrackInterface* MediaStreamAudioTrack::GetAudioAdapter() {
56 NOTREACHED(); 31 NOTREACHED();
57 return nullptr; 32 return nullptr;
58 } 33 }
59 34
60 } // namespace content 35 } // 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