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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2533733002: Roll chromium_revision 5e821a778b..5c22c2afac (432715:434448) (Closed)

Created:
4 years ago by kjellander_webrtc
Modified:
4 years ago
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
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

Roll chromium_revision 5e821a778b..5c22c2afac (432715:434448) Manual changes needed to use our own test runner for Android tests. VideoProcessorIntegrationTest.ProcessNoLossChangeFrameRateFrameDropVP9 is failing for TSan and UBSan configs, so disable the test for them here. Change log: https://chromium.googlesource.com/chromium/src/+log/5e821a778b..5c22c2afac Full diff: https://chromium.googlesource.com/chromium/src/+/5e821a778b..5c22c2afac Changed dependencies: * src/buildtools: https://chromium.googlesource.com/chromium/buildtools.git/+log/1f985091a5..991f459071 * src/third_party/android_tools: https://chromium.googlesource.com/android_tools.git/+log/c02a002b48..811a2c3f91 * src/third_party/catapult: https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/249cfbcd88..671a654736 * src/third_party/ffmpeg: https://chromium.googlesource.com/chromium/third_party/ffmpeg.git/+log/3c7a098821..d16162e3f4 * src/third_party/icu: https://chromium.googlesource.com/chromium/deps/icu.git/+log/7ddf5e9ba1..dda089a98a * src/third_party/libvpx/source/libvpx: https://chromium.googlesource.com/webm/libvpx.git/+log/5c64c01c7c..d7f1d60c51 * src/third_party/openmax_dl: https://chromium.googlesource.com/external/webrtc/deps/third_party/openmax.git/+log/57d33bee78..7acede9c03 DEPS diff: https://chromium.googlesource.com/chromium/src/+/5e821a778b..5c22c2afac/DEPS Clang version changed 284979:287780 Details: https://chromium.googlesource.com/chromium/src/+/5e821a778b..5c22c2afac/tools/clang/scripts/update.py TBR=marpan@webrtc.org, ehmaldonado@webrtc.org BUG=webrtc:6775, webrtc:6739, webrtc:6781 NOTRY=True Committed: https://crrev.com/847f6897f2b79c76227e8924b3e1fe4791ba60b0 Cr-Commit-Position: refs/heads/master@{#15256}

Patch Set 1 #

Patch Set 2 : Disabled 3 more tests #

Patch Set 3 : Also disable ProcessNoLossDenoiserOnVP9 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+46 lines, -6 lines) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download
M build_overrides/build.gni View 1 chunk +2 lines, -0 lines 0 comments Download
M webrtc/base/BUILD.gn View 1 chunk +4 lines, -0 lines 0 comments Download
M webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.cc View 1 2 5 chunks +36 lines, -5 lines 0 comments Download
M webrtc/test/BUILD.gn View 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 14 (9 generated)
kjellander_webrtc
Only running trybots failing in https://codereview.webrtc.org/2533543002/
4 years ago (2016-11-28 09:02:47 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/2533733002/40001
4 years ago (2016-11-28 09:56:56 UTC) #8
commit-bot: I haz the power
Committed patchset #3 (id:40001)
4 years ago (2016-11-28 10:04:42 UTC) #11
commit-bot: I haz the power
Patchset 3 (id:??) landed as https://crrev.com/847f6897f2b79c76227e8924b3e1fe4791ba60b0 Cr-Commit-Position: refs/heads/master@{#15256}
4 years ago (2016-11-28 10:04:50 UTC) #13
ehmaldonado_webrtc
4 years ago (2016-11-28 10:34:14 UTC) #14
Message was sent while issue was closed. 
On 2016/11/28 10:04:50, commit-bot: I haz the power wrote:
> Patchset 3 (id:??) landed as
> https://crrev.com/847f6897f2b79c76227e8924b3e1fe4791ba60b0
> Cr-Commit-Position: refs/heads/master@{#15256}

lgtm

Powered by Google App Engine
This is Rietveld 408576698