 Chromium Code Reviews
 Chromium Code Reviews Issue 2826263004:
  Move responsibility for RTP header extensions on video receive.  (Closed)
    
  
    Issue 2826263004:
  Move responsibility for RTP header extensions on video receive.  (Closed) 
  | Index: webrtc/pc/mediacontroller.h | 
| diff --git a/webrtc/pc/mediacontroller.h b/webrtc/pc/mediacontroller.h | 
| index f5a9034918b1df58ab55115a11d91d10168da5b1..f279192862f9475c235021201ef2cd507f1b266b 100644 | 
| --- a/webrtc/pc/mediacontroller.h | 
| +++ b/webrtc/pc/mediacontroller.h | 
| @@ -12,6 +12,7 @@ | 
| #define WEBRTC_PC_MEDIACONTROLLER_H_ | 
| #include "webrtc/base/thread.h" | 
| +#include "webrtc/config.h" | 
| namespace cricket { | 
| class ChannelManager; | 
| @@ -40,6 +41,10 @@ class MediaControllerInterface { | 
| virtual webrtc::Call* call_w() = 0; | 
| virtual cricket::ChannelManager* channel_manager() const = 0; | 
| virtual const cricket::MediaConfig& config() const = 0; | 
| + // Proxy for Call (extensions may change mid-call). | 
| + // TODO(nisse): Should belong to RtpTransportController. | 
| + virtual void SetVideoReceiveRtpHeaderExtensions( | 
| + const std::vector<RtpExtension>& extensions) = 0; | 
| 
pthatcher1
2017/04/29 00:25:18
This shouldn't be at the per-MediaController level
 | 
| }; | 
| } // namespace webrtc |