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

Issue 1687273002: Revert of Make cricket::VideoCapturer implement VideoSourceInterface (Closed)

Created:
4 years, 10 months ago by perkj_webrtc
Modified:
4 years, 10 months ago
CC:
webrtc-reviews_webrtc.org
Base URL:
https://chromium.googlesource.com/external/webrtc.git@master
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

Revert of Make cricket::VideoCapturer implement VideoSourceInterface (patchset #12 id:260001 of https://codereview.webrtc.org/1655793003/ ) Reason for revert: Somehow breaks Chromium FYI.... https://build.chromium.org/p/chromium.webrtc.fyi/builders/Win%20Builder/builds/3483/steps/compile/logs/stdio Original issue's description: > This cl introduce a VideoSourceInterface and let cricket::VideoCapturer implement it. > Further more, it adds a VideoBroadcaster than is used for delivering frames to multiple sinks. > > > BUG=webrtc:5426 TBR=pthatcher@google.com,nisse@webrtc.org,pthatcher@webrtc.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=webrtc:5426 Committed: https://crrev.com/4b2a5a8095ed1a96787a609c99bcc423f6989812 Cr-Commit-Position: refs/heads/master@{#11565}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+370 lines, -361 lines) Patch
M talk/session/media/channelmanager.h View 1 chunk +1 line, -0 lines 0 comments Download
M talk/session/media/channelmanager.cc View 1 chunk +1 line, -0 lines 0 comments Download
M webrtc/api/androidvideocapturer.cc View 2 chunks +3 lines, -2 lines 0 comments Download
M webrtc/api/remotevideocapturer_unittest.cc View 3 chunks +18 lines, -0 lines 0 comments Download
M webrtc/media/base/capturemanager.h View 2 chunks +4 lines, -3 lines 0 comments Download
M webrtc/media/base/capturemanager.cc View 8 chunks +46 lines, -12 lines 0 comments Download
A webrtc/media/base/capturerenderadapter.h View 1 chunk +58 lines, -0 lines 0 comments Download
A webrtc/media/base/capturerenderadapter.cc View 1 chunk +83 lines, -0 lines 0 comments Download
M webrtc/media/base/fakevideorenderer.h View 3 chunks +0 lines, -6 lines 0 comments Download
D webrtc/media/base/videobroadcaster.h View 1 chunk +0 lines, -60 lines 0 comments Download
D webrtc/media/base/videobroadcaster.cc View 1 chunk +0 lines, -82 lines 0 comments Download
M webrtc/media/base/videocapturer.h View 8 chunks +13 lines, -16 lines 0 comments Download
M webrtc/media/base/videocapturer.cc View 4 chunks +13 lines, -21 lines 0 comments Download
M webrtc/media/base/videocapturer_unittest.cc View 13 chunks +53 lines, -66 lines 0 comments Download
M webrtc/media/base/videoengine_unittest.h View 1 chunk +15 lines, -0 lines 0 comments Download
M webrtc/media/base/videosinkinterface.h View 1 chunk +2 lines, -1 line 0 comments Download
D webrtc/media/base/videosourceinterface.h View 1 chunk +0 lines, -46 lines 0 comments Download
M webrtc/media/media.gyp View 2 chunks +2 lines, -2 lines 0 comments Download
M webrtc/media/webrtc/webrtcvideocapturer.h View 3 chunks +10 lines, -7 lines 0 comments Download
M webrtc/media/webrtc/webrtcvideocapturer.cc View 5 chunks +14 lines, -9 lines 0 comments Download
M webrtc/media/webrtc/webrtcvideoengine2.h View 5 chunks +3 lines, -7 lines 0 comments Download
M webrtc/media/webrtc/webrtcvideoengine2.cc View 11 chunks +31 lines, -21 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
perkj_webrtc
Created Revert of Make cricket::VideoCapturer implement VideoSourceInterface
4 years, 10 months ago (2016-02-11 09:19:57 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1687273002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1687273002/1
4 years, 10 months ago (2016-02-11 09:20:05 UTC) #2
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 10 months ago (2016-02-11 09:20:21 UTC) #4
commit-bot: I haz the power
4 years, 10 months ago (2016-02-11 10:40:53 UTC) #6
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/4b2a5a8095ed1a96787a609c99bcc423f6989812
Cr-Commit-Position: refs/heads/master@{#11565}

Powered by Google App Engine
This is Rietveld 408576698