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

Issue 2395863002: Roll src/third_party/libvpx/source/libvpx/ 50b9c467d..897870497 (8 commits). (Closed)

Created:
4 years, 2 months ago by Johann
Modified:
4 years, 2 months ago
Reviewers:
jzern
CC:
chromium-reviews, wwcv, fgalligan1, Tom Finegan
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Roll src/third_party/libvpx/source/libvpx/ 50b9c467d..897870497 (8 commits). https://chromium.googlesource.com/webm/libvpx.git/+log/50b9c467da77..897870497024 $ git log 50b9c467d..897870497 --date=short --no-merges --format='%ad %ae %s' 2016-09-21 sarahparker Remove rate deviation metric from vp8 2016-10-03 johannkoenig Connect partial IDCT tests 2016-09-30 angiebird Move highbd txfm input range check from 2d iht transform to 1d idct/iadst 2016-10-03 kaustubh.raste Fix vpx_plane_add_noise_msa functionality bit-mismatch 2016-10-03 marpan Update to vpx_temporal_svc_encoder command line. 2016-04-13 gezalore Fix warning when building with GCC 5. 2016-10-01 jzern invalid_file_test: quiet unused const warning 2016-09-30 jzern cosmetics,*_neon.c: rm redundant return from void fns BUG=651625 Committed: https://crrev.com/7dd5d014381033352b45e84b158b317010520251 Cr-Commit-Position: refs/heads/master@{#423420}

Patch Set 1 #

Patch Set 2 : remove marpan@'s latest #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -6 lines) Patch
M DEPS View 1 1 chunk +1 line, -1 line 0 comments Download
M third_party/libvpx/README.chromium View 1 1 chunk +2 lines, -2 lines 0 comments Download
M third_party/libvpx/source/config/vpx_version.h View 1 1 chunk +3 lines, -3 lines 0 comments Download

Messages

Total messages: 21 (13 generated)
Johann
4 years, 2 months ago (2016-10-05 20:04:24 UTC) #4
jzern
lgtm thanks, I just about to do that
4 years, 2 months ago (2016-10-05 21:24:45 UTC) #5
jzern
On 2016/10/05 21:24:45, jzern wrote: > lgtm > > thanks, I just about to do ...
4 years, 2 months ago (2016-10-05 23:05:59 UTC) #8
jzern
still lgtm
4 years, 2 months ago (2016-10-05 23:14:02 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2395863002/20001
4 years, 2 months ago (2016-10-06 03:21:10 UTC) #16
Johann
On 2016/10/05 23:14:02, jzern wrote: > still lgtm subsequent git cl uploads do not look ...
4 years, 2 months ago (2016-10-06 03:27:21 UTC) #17
commit-bot: I haz the power
Committed patchset #2 (id:20001)
4 years, 2 months ago (2016-10-06 03:27:41 UTC) #19
commit-bot: I haz the power
4 years, 2 months ago (2016-10-06 03:30:01 UTC) #21
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/7dd5d014381033352b45e84b158b317010520251
Cr-Commit-Position: refs/heads/master@{#423420}

Powered by Google App Engine
This is Rietveld 408576698