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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 1540243002: Lint fix for webrtc/modules/video_coding PART 3! (Closed)

Created:
5 years ago by philipel
Modified:
5 years ago
Reviewers:
mflodman
CC:
webrtc-reviews_webrtc.org, video-team_agora.io, yujie_mao (webrtc), zhengzhonghou_agora.io, stefan-webrtc, tterriberry_mozilla.com, qiang.lu, niklas.enbom, peah-webrtc, mflodman
Base URL:
https://chromium.googlesource.com/external/webrtc.git@master
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

Lint fix for webrtc/modules/video_coding PART 3! Trying to submit all changes at once proved impossible since there were too many changes in too many files. The changes to PRESUBMIT.py will be uploaded in the last CL. (original CL: https://codereview.webrtc.org/1528503003/) BUG=webrtc:5309 TBR=mflodman@webrtc.org Committed: https://crrev.com/5908c71128aea207e42f86468aedb0a6fce3cccb Cr-Commit-Position: refs/heads/master@{#11105}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+644 lines, -749 lines) Patch
M PRESUBMIT.py View 1 chunk +1 line, -0 lines 0 comments Download
M webrtc/modules/video_coding/test/receiver_tests.h View 3 chunks +5 lines, -5 lines 0 comments Download
M webrtc/modules/video_coding/test/rtp_player.h View 2 chunks +10 lines, -7 lines 0 comments Download
M webrtc/modules/video_coding/test/rtp_player.cc View 12 chunks +24 lines, -25 lines 0 comments Download
M webrtc/modules/video_coding/test/stream_generator.cc View 4 chunks +9 lines, -6 lines 0 comments Download
M webrtc/modules/video_coding/test/test_util.h View 3 chunks +6 lines, -6 lines 0 comments Download
M webrtc/modules/video_coding/test/test_util.cc View 7 chunks +18 lines, -15 lines 0 comments Download
M webrtc/modules/video_coding/test/tester_main.cc View 3 chunks +26 lines, -23 lines 0 comments Download
M webrtc/modules/video_coding/test/vcm_payload_sink_factory.h View 1 chunk +4 lines, -2 lines 0 comments Download
M webrtc/modules/video_coding/test/vcm_payload_sink_factory.cc View 3 chunks +4 lines, -8 lines 0 comments Download
M webrtc/modules/video_coding/test/video_rtp_play.cc View 2 chunks +4 lines, -4 lines 0 comments Download
M webrtc/modules/video_coding/test/video_source.h View 1 chunk +60 lines, -57 lines 0 comments Download
M webrtc/modules/video_coding/timestamp_map.h View 1 chunk +1 line, -1 line 0 comments Download
M webrtc/modules/video_coding/timestamp_map.cc View 2 chunks +3 lines, -5 lines 0 comments Download
M webrtc/modules/video_coding/timing.h View 1 chunk +1 line, -2 lines 0 comments Download
M webrtc/modules/video_coding/timing.cc View 10 chunks +25 lines, -22 lines 0 comments Download
M webrtc/modules/video_coding/timing_unittest.cc View 5 chunks +11 lines, -11 lines 0 comments Download
M webrtc/modules/video_coding/utility/frame_dropper.h View 1 chunk +57 lines, -57 lines 0 comments Download
M webrtc/modules/video_coding/utility/frame_dropper.cc View 3 chunks +226 lines, -301 lines 0 comments Download
M webrtc/modules/video_coding/utility/mock/mock_frame_dropper.h View 1 chunk +7 lines, -14 lines 0 comments Download
M webrtc/modules/video_coding/utility/moving_average.h View 3 chunks +8 lines, -8 lines 0 comments Download
M webrtc/modules/video_coding/utility/quality_scaler.cc View 4 chunks +5 lines, -11 lines 0 comments Download
M webrtc/modules/video_coding/utility/quality_scaler_unittest.cc View 8 chunks +19 lines, -13 lines 0 comments Download
M webrtc/modules/video_coding/utility/vp8_header_parser.h View 1 chunk +21 lines, -33 lines 0 comments Download
M webrtc/modules/video_coding/utility/vp8_header_parser.cc View 3 chunks +11 lines, -11 lines 0 comments Download
M webrtc/modules/video_coding/video_coding_impl.cc View 4 chunks +17 lines, -20 lines 0 comments Download
M webrtc/modules/video_coding/video_coding_robustness_unittest.cc View 5 chunks +41 lines, -51 lines 0 comments Download
M webrtc/modules/video_coding/video_sender.cc View 4 chunks +9 lines, -13 lines 0 comments Download
M webrtc/modules/video_coding/video_sender_unittest.cc View 4 chunks +11 lines, -18 lines 0 comments Download

Messages

Total messages: 9 (4 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1540243002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1540243002/1
5 years ago (2015-12-21 12:18:14 UTC) #2
commit-bot: I haz the power
Try jobs failed on following builders: win_baremetal on tryserver.webrtc (JOB_TIMED_OUT, no build URL)
5 years ago (2015-12-21 14:18:31 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1540243002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1540243002/1
5 years ago (2015-12-21 16:19:50 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years ago (2015-12-21 16:23:24 UTC) #7
commit-bot: I haz the power
5 years ago (2015-12-21 16:23:34 UTC) #9
Message was sent while issue was closed. 
Patchset 1 (id:??) landed as
https://crrev.com/5908c71128aea207e42f86468aedb0a6fce3cccb
Cr-Commit-Position: refs/heads/master@{#11105}

Powered by Google App Engine
This is Rietveld 408576698