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

Side by Side Diff: media/cast/audio_sender/audio_sender.cc

Issue 163553006: Cast: Refactoring Cast API's (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Refactoring and responding to review Created 6 years, 10 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 | Annotate | Revision Log
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/cast/audio_sender/audio_sender.h" 5 #include "media/cast/audio_sender/audio_sender.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/message_loop/message_loop.h" 9 #include "base/message_loop/message_loop.h"
10 #include "media/cast/audio_sender/audio_encoder.h" 10 #include "media/cast/audio_sender/audio_encoder.h"
(...skipping 91 matching lines...) Expand 10 before | Expand all | Expand 10 after
102 transport_sender_, 102 transport_sender_,
103 NULL, // paced sender. 103 NULL, // paced sender.
104 rtp_audio_sender_statistics_.get(), 104 rtp_audio_sender_statistics_.get(),
105 NULL, 105 NULL,
106 audio_config.rtcp_mode, 106 audio_config.rtcp_mode,
107 base::TimeDelta::FromMilliseconds(audio_config.rtcp_interval), 107 base::TimeDelta::FromMilliseconds(audio_config.rtcp_interval),
108 audio_config.sender_ssrc, 108 audio_config.sender_ssrc,
109 audio_config.incoming_feedback_ssrc, 109 audio_config.incoming_feedback_ssrc,
110 audio_config.rtcp_c_name), 110 audio_config.rtcp_c_name),
111 timers_initialized_(false), 111 timers_initialized_(false),
112 initialization_status_(STATUS_INITIALIZED), 112 cast_initialization_cb_(STATUS_AUDIO_UNINITIALIZED),
113 weak_factory_(this) { 113 weak_factory_(this) {
114 if (!audio_config.use_external_encoder) { 114 if (!audio_config.use_external_encoder) {
115 audio_encoder_ = 115 audio_encoder_ =
116 new AudioEncoder(cast_environment, 116 new AudioEncoder(cast_environment,
117 audio_config, 117 audio_config,
118 base::Bind(&AudioSender::SendEncodedAudioFrame, 118 base::Bind(&AudioSender::SendEncodedAudioFrame,
119 weak_factory_.GetWeakPtr())); 119 weak_factory_.GetWeakPtr()));
120 initialization_status_ = audio_encoder_->InitializationResult(); 120 cast_initialization_cb_ = audio_encoder_->InitializationResult();
121 } 121 }
122 } 122 }
123 123
124 AudioSender::~AudioSender() {} 124 AudioSender::~AudioSender() {}
125 125
126 void AudioSender::InitializeTimers() { 126 void AudioSender::InitializeTimers() {
127 DCHECK(cast_environment_->CurrentlyOn(CastEnvironment::MAIN)); 127 DCHECK(cast_environment_->CurrentlyOn(CastEnvironment::MAIN));
128 if (!timers_initialized_) { 128 if (!timers_initialized_) {
129 timers_initialized_ = true; 129 timers_initialized_ = true;
130 ScheduleNextRtcpReport(); 130 ScheduleNextRtcpReport();
131 } 131 }
132 } 132 }
133 133
134 void AudioSender::InsertAudio(const AudioBus* audio_bus, 134 void AudioSender::InsertAudio(const AudioBus* audio_bus,
135 const base::TimeTicks& recorded_time, 135 const base::TimeTicks& recorded_time,
136 const base::Closure& done_callback) { 136 const base::Closure& done_callback) {
137 DCHECK(cast_environment_->CurrentlyOn(CastEnvironment::MAIN)); 137 DCHECK(cast_environment_->CurrentlyOn(CastEnvironment::MAIN));
138 DCHECK(audio_encoder_.get()) << "Invalid internal state"; 138 DCHECK(audio_encoder_.get()) << "Invalid internal state";
139
140 audio_encoder_->InsertAudio(audio_bus, recorded_time, done_callback); 139 audio_encoder_->InsertAudio(audio_bus, recorded_time, done_callback);
141 } 140 }
142 141
143 void AudioSender::SendEncodedAudioFrame( 142 void AudioSender::SendEncodedAudioFrame(
144 scoped_ptr<transport::EncodedAudioFrame> audio_frame, 143 scoped_ptr<transport::EncodedAudioFrame> audio_frame,
145 const base::TimeTicks& recorded_time) { 144 const base::TimeTicks& recorded_time) {
146 DCHECK(cast_environment_->CurrentlyOn(CastEnvironment::MAIN)); 145 DCHECK(cast_environment_->CurrentlyOn(CastEnvironment::MAIN));
147 InitializeTimers(); 146 InitializeTimers();
148 cast_environment_->PostTask( 147 cast_environment_->PostTask(
149 CastEnvironment::TRANSPORT, 148 CastEnvironment::TRANSPORT,
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
202 } 201 }
203 202
204 void AudioSender::ResendPacketsOnTransportThread( 203 void AudioSender::ResendPacketsOnTransportThread(
205 const transport::MissingFramesAndPacketsMap& missing_packets) { 204 const transport::MissingFramesAndPacketsMap& missing_packets) {
206 DCHECK(cast_environment_->CurrentlyOn(CastEnvironment::TRANSPORT)); 205 DCHECK(cast_environment_->CurrentlyOn(CastEnvironment::TRANSPORT));
207 transport_sender_->ResendPackets(true, missing_packets); 206 transport_sender_->ResendPackets(true, missing_packets);
208 } 207 }
209 208
210 } // namespace cast 209 } // namespace cast
211 } // namespace media 210 } // namespace media
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698