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

Side by Side Diff: content/renderer/media/webrtc_audio_capturer.h

Issue 1304173002: Log error instead of CHECKing input audio sequence. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Code review. Created 5 years, 4 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #ifndef CONTENT_RENDERER_MEDIA_WEBRTC_AUDIO_CAPTURER_H_ 5 #ifndef CONTENT_RENDERER_MEDIA_WEBRTC_AUDIO_CAPTURER_H_
6 #define CONTENT_RENDERER_MEDIA_WEBRTC_AUDIO_CAPTURER_H_ 6 #define CONTENT_RENDERER_MEDIA_WEBRTC_AUDIO_CAPTURER_H_
7 7
8 #include <list> 8 #include <list>
9 #include <string> 9 #include <string>
10 10
(...skipping 111 matching lines...) Expand 10 before | Expand all | Expand 10 after
122 const blink::WebMediaConstraints& constraints, 122 const blink::WebMediaConstraints& constraints,
123 WebRtcAudioDeviceImpl* audio_device, 123 WebRtcAudioDeviceImpl* audio_device,
124 MediaStreamAudioSource* audio_source); 124 MediaStreamAudioSource* audio_source);
125 125
126 // AudioCapturerSource::CaptureCallback implementation. 126 // AudioCapturerSource::CaptureCallback implementation.
127 // Called on the AudioInputDevice audio thread. 127 // Called on the AudioInputDevice audio thread.
128 void Capture(const media::AudioBus* audio_source, 128 void Capture(const media::AudioBus* audio_source,
129 int audio_delay_milliseconds, 129 int audio_delay_milliseconds,
130 double volume, 130 double volume,
131 bool key_pressed) override; 131 bool key_pressed) override;
132 void OnCaptureError() override; 132 void OnCaptureError(const std::string& message) override;
133 133
134 // Initializes the default audio capturing source using the provided render 134 // Initializes the default audio capturing source using the provided render
135 // frame id and device information. Return true if success, otherwise false. 135 // frame id and device information. Return true if success, otherwise false.
136 bool Initialize(); 136 bool Initialize();
137 137
138 // SetCapturerSourceInternal() is called if the client on the source side 138 // SetCapturerSourceInternal() is called if the client on the source side
139 // desires to provide their own captured audio data. Client is responsible 139 // desires to provide their own captured audio data. Client is responsible
140 // for calling Start() on its own source to get the ball rolling. 140 // for calling Start() on its own source to get the ball rolling.
141 // Called on the main render thread. 141 // Called on the main render thread.
142 // buffer_size is optional. Set to 0 to let it be chosen automatically. 142 // buffer_size is optional. Set to 0 to let it be chosen automatically.
(...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
203 // guaranteed to exist as long as a WebRtcLocalAudioTrack is connected to this 203 // guaranteed to exist as long as a WebRtcLocalAudioTrack is connected to this
204 // WebRtcAudioCapturer. 204 // WebRtcAudioCapturer.
205 MediaStreamAudioSource* const audio_source_; 205 MediaStreamAudioSource* const audio_source_;
206 206
207 DISALLOW_COPY_AND_ASSIGN(WebRtcAudioCapturer); 207 DISALLOW_COPY_AND_ASSIGN(WebRtcAudioCapturer);
208 }; 208 };
209 209
210 } // namespace content 210 } // namespace content
211 211
212 #endif // CONTENT_RENDERER_MEDIA_WEBRTC_AUDIO_CAPTURER_H_ 212 #endif // CONTENT_RENDERER_MEDIA_WEBRTC_AUDIO_CAPTURER_H_
OLDNEW
« no previous file with comments | « chrome/renderer/media/cast_receiver_session.cc ('k') | content/renderer/media/webrtc_audio_capturer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698