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

Issue 2153503002: Revert of roll libyuv from r1579 to r1607 (Closed)

Created:
4 years, 5 months ago by wjmaclean
Modified:
4 years, 5 months ago
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of roll libyuv from r1579 to r1607 (patchset #19 id:350001 of https://codereview.chromium.org/2126333002/ ) Reason for revert: Causing compile failures on: https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20MSAN Original issue's description: > roll libyuv from r1579 to r1607 > > roll libyuv for GN fix to chromium internal settings, enabling more warnings > and consistant with gyp build options. > > Add ARGBExtractAlpha function. > Fix YUV bilinear filter improved filter accuracy; uses 128-f instead of 127-f. > unittest fixes for Arm scaler - use J420 for J422 pointing to same row, > improving performance and accuracy. > Fix ios jpeg build error. > Fix BlendPlaneRow_SSSE3 impossible constraints on android full debug > Fix unittests to avoid private row.h header. > Fix vector_size(32) build error on yuv conversion for intel on gcc 4.4. > > BUG=libyuv:572, chromium:592745, libyuv:575, libyuv:579, libyuv:573, libyuv:583, chromium:601140, libyuv:582, libyuv:581, chromium:601140, libyuv:578, libyuv:581, libyuv:580, libyuv:586, libyuv:590, webrtc:5006, libyuv:593, libyuv:592, libyuv:594, libyuv:595, libyuv:597, libyuv:599, libyuv:598, libyuv:600, libyuv:602, libyuv:605, libyuv:610, libyuv:612, libyuv:618, libyuv:620, webrtc:6091, webrtc:6094, libyuv:616, libyuv:623 > TEST=try bots > R=rsesek@chromium.org > > Committed: https://crrev.com/053185feb2645e1628e6b25fbf1e21f309c26776 > Cr-Commit-Position: refs/heads/master@{#405512} TBR=rsesek@chromium.org,fbarchard@google.com # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=libyuv:572, chromium:592745, libyuv:575, libyuv:579, libyuv:573, libyuv:583, chromium:601140, libyuv:582, libyuv:581, chromium:601140, libyuv:578, libyuv:581, libyuv:580, libyuv:586, libyuv:590, webrtc:5006, libyuv:593, libyuv:592, libyuv:594, libyuv:595, libyuv:597, libyuv:599, libyuv:598, libyuv:600, libyuv:602, libyuv:605, libyuv:610, libyuv:612, libyuv:618, libyuv:620, webrtc:6091, webrtc:6094, libyuv:616, libyuv:623 Committed: https://crrev.com/d35937737cf9706dd535b72b7a13f94993d32187 Cr-Commit-Position: refs/heads/master@{#405524}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 8 (3 generated)
wjmaclean
Created Revert of roll libyuv from r1579 to r1607
4 years, 5 months ago (2016-07-14 18:42:52 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.chromium.org/2153503002/1
4 years, 5 months ago (2016-07-14 18:43:36 UTC) #3
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 5 months ago (2016-07-14 18:44:29 UTC) #5
commit-bot: I haz the power
CQ bit was unchecked.
4 years, 5 months ago (2016-07-14 18:44:33 UTC) #6
commit-bot: I haz the power
4 years, 5 months ago (2016-07-14 18:46:44 UTC) #8
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/d35937737cf9706dd535b72b7a13f94993d32187
Cr-Commit-Position: refs/heads/master@{#405524}

Powered by Google App Engine
This is Rietveld 408576698