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

Side by Side Diff: webrtc/media/engine/webrtcvoiceengine.cc

Issue 1952123003: Surface the IntelligibilityEnhancer on MediaConstraints (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 7 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 /* 1 /*
2 * Copyright (c) 2004 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2004 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 825 matching lines...) Expand 10 before | Expand all | Expand 10 after
836 836
837 if (options.experimental_ns) { 837 if (options.experimental_ns) {
838 experimental_ns_ = options.experimental_ns; 838 experimental_ns_ = options.experimental_ns;
839 } 839 }
840 if (experimental_ns_) { 840 if (experimental_ns_) {
841 LOG(LS_INFO) << "Experimental ns is enabled? " << *experimental_ns_; 841 LOG(LS_INFO) << "Experimental ns is enabled? " << *experimental_ns_;
842 config.Set<webrtc::ExperimentalNs>( 842 config.Set<webrtc::ExperimentalNs>(
843 new webrtc::ExperimentalNs(*experimental_ns_)); 843 new webrtc::ExperimentalNs(*experimental_ns_));
844 } 844 }
845 845
846 const bool intelligibility_enhancer = adm()->IntelligibilityIsEnabled();
the sun 2016/05/06 09:00:15 Is the idea to pass the option to enable IE via th
aluebs-webrtc 2016/05/06 16:27:10 Because I was pointed to the WebRtcAudioUtils API
847 LOG(LS_INFO) << "Intelligibility Enhancer is enabled? "
848 << intelligibility_enhancer;
849 config.Set<webrtc::Intelligibility>(
850 new webrtc::Intelligibility(intelligibility_enhancer));
851
846 // We check audioproc for the benefit of tests, since FakeWebRtcVoiceEngine 852 // We check audioproc for the benefit of tests, since FakeWebRtcVoiceEngine
847 // returns NULL on audio_processing(). 853 // returns NULL on audio_processing().
848 webrtc::AudioProcessing* audioproc = voe_wrapper_->base()->audio_processing(); 854 webrtc::AudioProcessing* audioproc = voe_wrapper_->base()->audio_processing();
849 if (audioproc) { 855 if (audioproc) {
850 audioproc->SetExtraOptions(config); 856 audioproc->SetExtraOptions(config);
851 } 857 }
852 858
853 if (options.recording_sample_rate) { 859 if (options.recording_sample_rate) {
854 LOG(LS_INFO) << "Recording sample rate is " 860 LOG(LS_INFO) << "Recording sample rate is "
855 << *options.recording_sample_rate; 861 << *options.recording_sample_rate;
(...skipping 1700 matching lines...) Expand 10 before | Expand all | Expand 10 after
2556 } 2562 }
2557 } else { 2563 } else {
2558 LOG(LS_INFO) << "Stopping playout for channel #" << channel; 2564 LOG(LS_INFO) << "Stopping playout for channel #" << channel;
2559 engine()->voe()->base()->StopPlayout(channel); 2565 engine()->voe()->base()->StopPlayout(channel);
2560 } 2566 }
2561 return true; 2567 return true;
2562 } 2568 }
2563 } // namespace cricket 2569 } // namespace cricket
2564 2570
2565 #endif // HAVE_WEBRTC_VOICE 2571 #endif // HAVE_WEBRTC_VOICE
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698