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

Side by Side Diff: talk/media/webrtc/webrtcvideoengine2.cc

Issue 1457653003: Remove SetVideoLogging/SetAudioLogging from ChannelManager and down the stack. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: oops Created 5 years, 1 month 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
« no previous file with comments | « talk/media/webrtc/webrtcvideoengine2.h ('k') | talk/media/webrtc/webrtcvoe.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * libjingle 2 * libjingle
3 * Copyright 2014 Google Inc. 3 * Copyright 2014 Google Inc.
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions are met: 6 * modification, are permitted provided that the following conditions are met:
7 * 7 *
8 * 1. Redistributions of source code must retain the above copyright notice, 8 * 1. Redistributions of source code must retain the above copyright notice,
9 * this list of conditions and the following disclaimer. 9 * this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright notice, 10 * 2. Redistributions in binary form must reproduce the above copyright notice,
(...skipping 602 matching lines...) Expand 10 before | Expand all | Expand 10 after
613 613
614 const std::vector<VideoCodec>& WebRtcVideoEngine2::codecs() const { 614 const std::vector<VideoCodec>& WebRtcVideoEngine2::codecs() const {
615 return video_codecs_; 615 return video_codecs_;
616 } 616 }
617 617
618 const std::vector<RtpHeaderExtension>& 618 const std::vector<RtpHeaderExtension>&
619 WebRtcVideoEngine2::rtp_header_extensions() const { 619 WebRtcVideoEngine2::rtp_header_extensions() const {
620 return rtp_header_extensions_; 620 return rtp_header_extensions_;
621 } 621 }
622 622
623 void WebRtcVideoEngine2::SetLogging(int min_sev, const char* filter) {
624 // TODO(pbos): Set up logging.
625 LOG(LS_VERBOSE) << "SetLogging: " << min_sev << '"' << filter << '"';
626 // if min_sev == -1, we keep the current log level.
627 if (min_sev < 0) {
628 RTC_DCHECK(min_sev == -1);
629 return;
630 }
631 }
632
633 void WebRtcVideoEngine2::SetExternalDecoderFactory( 623 void WebRtcVideoEngine2::SetExternalDecoderFactory(
634 WebRtcVideoDecoderFactory* decoder_factory) { 624 WebRtcVideoDecoderFactory* decoder_factory) {
635 RTC_DCHECK(!initialized_); 625 RTC_DCHECK(!initialized_);
636 external_decoder_factory_ = decoder_factory; 626 external_decoder_factory_ = decoder_factory;
637 } 627 }
638 628
639 void WebRtcVideoEngine2::SetExternalEncoderFactory( 629 void WebRtcVideoEngine2::SetExternalEncoderFactory(
640 WebRtcVideoEncoderFactory* encoder_factory) { 630 WebRtcVideoEncoderFactory* encoder_factory) {
641 RTC_DCHECK(!initialized_); 631 RTC_DCHECK(!initialized_);
642 if (external_encoder_factory_ == encoder_factory) 632 if (external_encoder_factory_ == encoder_factory)
(...skipping 2101 matching lines...) Expand 10 before | Expand all | Expand 10 after
2744 video_codecs[i].rtx_payload_type = rtx_mapping[video_codecs[i].codec.id]; 2734 video_codecs[i].rtx_payload_type = rtx_mapping[video_codecs[i].codec.id];
2745 } 2735 }
2746 } 2736 }
2747 2737
2748 return video_codecs; 2738 return video_codecs;
2749 } 2739 }
2750 2740
2751 } // namespace cricket 2741 } // namespace cricket
2752 2742
2753 #endif // HAVE_WEBRTC_VIDEO 2743 #endif // HAVE_WEBRTC_VIDEO
OLDNEW
« no previous file with comments | « talk/media/webrtc/webrtcvideoengine2.h ('k') | talk/media/webrtc/webrtcvoe.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698