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

Unified Diff: content/renderer/media/webrtc/processed_local_audio_source.cc

Issue 2626533002: Revert of Fix getUserMedia so that failure is reported for invalid audio sources. (Closed)
Patch Set: Created 3 years, 11 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
Index: content/renderer/media/webrtc/processed_local_audio_source.cc
diff --git a/content/renderer/media/webrtc/processed_local_audio_source.cc b/content/renderer/media/webrtc/processed_local_audio_source.cc
index bac130044f25a375c5431adc860ef085ff17c5da..fa89a2137cf5c8dd2edb57c71449dacf75003fc2 100644
--- a/content/renderer/media/webrtc/processed_local_audio_source.cc
+++ b/content/renderer/media/webrtc/processed_local_audio_source.cc
@@ -29,14 +29,10 @@
ProcessedLocalAudioSource::ProcessedLocalAudioSource(
int consumer_render_frame_id,
const StreamDeviceInfo& device_info,
- const blink::WebMediaConstraints& constraints,
- const ConstraintsCallback& started_callback,
PeerConnectionDependencyFactory* factory)
: MediaStreamAudioSource(true /* is_local_source */),
consumer_render_frame_id_(consumer_render_frame_id),
pc_factory_(factory),
- constraints_(constraints),
- started_callback_(started_callback),
volume_(0),
allow_invalid_render_frame_id_for_testing_(false) {
DCHECK(pc_factory_);
@@ -55,6 +51,14 @@
if (source && source->GetClassIdentifier() == kClassIdentifier)
return static_cast<ProcessedLocalAudioSource*>(source);
return nullptr;
+}
+
+void ProcessedLocalAudioSource::SetSourceConstraints(
+ const blink::WebMediaConstraints& constraints) {
+ DCHECK(thread_checker_.CalledOnValidThread());
+ DCHECK(!constraints.isNull());
+ DCHECK(!source_);
+ constraints_ = constraints;
}
void* ProcessedLocalAudioSource::GetClassIdentifier() const {
@@ -259,10 +263,6 @@
return WebRtcAudioDeviceImpl::kMaxVolumeLevel;
}
-void ProcessedLocalAudioSource::OnCaptureStarted() {
- started_callback_.Run(this, MEDIA_DEVICE_OK, "");
-}
-
void ProcessedLocalAudioSource::Capture(const media::AudioBus* audio_bus,
int audio_delay_milliseconds,
double volume,

Powered by Google App Engine
This is Rietveld 408576698