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

Issue 2764133002: Revert of Add framerate to VideoSinkWants and ability to signal on overuse (Closed)

Created:
3 years, 9 months ago by skvlad
Modified:
3 years, 9 months ago
CC:
webrtc-reviews_webrtc.org, video-team_agora.io, yujie_mao (webrtc), zhengzhonghou_agora.io, stefan-webrtc, tterriberry_mozilla.com, mflodman
Target Ref:
refs/heads/master
Project:
webrtc
Visibility:
Public.

Description

Revert of Add framerate to VideoSinkWants and ability to signal on overuse (patchset #14 id:250001 of https://codereview.webrtc.org/2716643002/ ) Reason for revert: Breaks perf tests: https://build.chromium.org/p/client.webrtc.perf/builders/Win7/builds/1679 https://build.chromium.org/p/client.webrtc.perf/builders/Android32%20Tests%20%28L%20Nexus5%29/builds/2325 Original issue's description: > Add framerate to VideoSinkWants and ability to signal on overuse > > In ViEEncoder, try to reduce framerate instead of resolution if the > current degradation preference is maintain-resolution rather than > balanced. > > BUG=webrtc:4172 > > Review-Url: https://codereview.webrtc.org/2716643002 > Cr-Commit-Position: refs/heads/master@{#17327} > Committed: https://chromium.googlesource.com/external/webrtc/+/72acf2526177bb4dbb5103cd6e165eb4361a5ae6 TBR=nisse@webrtc.org,magjed@webrtc.org,kthelgason@webrtc.org,ilnik@webrtc.org,stefan@webrtc.org,sprang@webrtc.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=webrtc:4172 Review-Url: https://codereview.webrtc.org/2764133002 Cr-Commit-Position: refs/heads/master@{#17331} Committed: https://chromium.googlesource.com/external/webrtc/+/8b45b11144c968b4173215c76f78c710c9a2ed0b

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+472 lines, -1402 lines) Patch
M webrtc/call/bitrate_estimator_tests.cc View 1 chunk +1 line, -1 line 0 comments Download
M webrtc/call/call_perf_tests.cc View 1 chunk +3 lines, -2 lines 0 comments Download
M webrtc/media/base/adaptedvideotracksource.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M webrtc/media/base/fakevideocapturer.h View 2 chunks +4 lines, -8 lines 0 comments Download
M webrtc/media/base/fakevideorenderer.h View 1 chunk +0 lines, -1 line 0 comments Download
M webrtc/media/base/videoadapter.h View 3 chunks +7 lines, -14 lines 0 comments Download
M webrtc/media/base/videoadapter.cc View 4 chunks +11 lines, -26 lines 0 comments Download
M webrtc/media/base/videoadapter_unittest.cc View 29 chunks +50 lines, -117 lines 0 comments Download
M webrtc/media/base/videobroadcaster.cc View 2 chunks +6 lines, -8 lines 0 comments Download
M webrtc/media/base/videobroadcaster_unittest.cc View 2 chunks +6 lines, -29 lines 0 comments Download
M webrtc/media/base/videocapturer.cc View 1 chunk +2 lines, -3 lines 0 comments Download
M webrtc/media/base/videocapturer_unittest.cc View 3 chunks +5 lines, -4 lines 0 comments Download
M webrtc/media/base/videosourceinterface.h View 1 chunk +1 line, -3 lines 0 comments Download
M webrtc/media/engine/fakewebrtccall.h View 2 chunks +0 lines, -2 lines 0 comments Download
M webrtc/media/engine/fakewebrtccall.cc View 3 chunks +4 lines, -22 lines 0 comments Download
M webrtc/media/engine/webrtcvideoengine2.h View 1 chunk +0 lines, -3 lines 0 comments Download
M webrtc/media/engine/webrtcvideoengine2.cc View 3 chunks +23 lines, -24 lines 0 comments Download
M webrtc/media/engine/webrtcvideoengine2_unittest.cc View 8 chunks +24 lines, -54 lines 0 comments Download
M webrtc/test/BUILD.gn View 2 chunks +0 lines, -2 lines 0 comments Download
M webrtc/test/call_test.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M webrtc/test/frame_generator_capturer.h View 2 chunks +1 line, -3 lines 0 comments Download
M webrtc/test/frame_generator_capturer.cc View 6 chunks +49 lines, -86 lines 0 comments Download
M webrtc/test/vcm_capturer.h View 1 chunk +0 lines, -2 lines 0 comments Download
M webrtc/test/vcm_capturer.cc View 6 chunks +12 lines, -16 lines 0 comments Download
M webrtc/test/video_capturer.h View 1 chunk +5 lines, -24 lines 0 comments Download
D webrtc/test/video_capturer.cc View 1 chunk +0 lines, -58 lines 0 comments Download
M webrtc/video/end_to_end_tests.cc View 4 chunks +4 lines, -6 lines 0 comments Download
M webrtc/video/overuse_frame_detector.h View 2 chunks +0 lines, -4 lines 0 comments Download
M webrtc/video/overuse_frame_detector.cc View 7 chunks +4 lines, -93 lines 0 comments Download
M webrtc/video/send_statistics_proxy.h View 1 chunk +1 line, -1 line 0 comments Download
M webrtc/video/video_quality_test.h View 1 chunk +1 line, -1 line 0 comments Download
M webrtc/video/video_send_stream_tests.cc View 3 chunks +3 lines, -3 lines 0 comments Download
M webrtc/video/vie_encoder.h View 5 chunks +3 lines, -16 lines 0 comments Download
M webrtc/video/vie_encoder.cc View 12 chunks +72 lines, -257 lines 0 comments Download
M webrtc/video/vie_encoder_unittest.cc View 29 chunks +164 lines, -494 lines 0 comments Download
M webrtc/video_send_stream.h View 1 chunk +2 lines, -11 lines 0 comments Download

Messages

Total messages: 7 (3 generated)
skvlad
Created Revert of Add framerate to VideoSinkWants and ability to signal on overuse
3 years, 9 months ago (2017-03-21 20:25:46 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/2764133002/1
3 years, 9 months ago (2017-03-21 20:25:56 UTC) #3
commit-bot: I haz the power
Committed patchset #1 (id:1) as https://chromium.googlesource.com/external/webrtc/+/8b45b11144c968b4173215c76f78c710c9a2ed0b
3 years, 9 months ago (2017-03-21 20:26:11 UTC) #6
sprang_webrtc
3 years, 9 months ago (2017-03-27 07:45:30 UTC) #7
Message was sent while issue was closed.
A revert of this CL (patchset #1 id:1) has been created in
https://codereview.webrtc.org/2781433002/ by sprang@webrtc.org.

The reason for reverting is: Found issue with test case, will add fix to reland
cl..

Powered by Google App Engine
This is Rietveld 408576698