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

Issue 1226543003: Revert of Switch SkJpegCode to libjpeg-turbo (Closed)

Created:
5 years, 5 months ago by jvanverth1
Modified:
5 years, 5 months ago
CC:
reviews_skia.org
Base URL:
https://skia.googlesource.com/skia.git@master
Target Ref:
refs/heads/master
Project:
skia
Visibility:
Public.

Description

Revert of Switch SkJpegCode to libjpeg-turbo (patchset #29 id:750001 of https://codereview.chromium.org/1180983002/) Reason for revert: DEPS roll failing Original issue's description: > Add libjpeg-turbo library (depends on yasm) > Mangle external function names to avoid conflict with libjpeg > Take advantage of direct color conversion (RGBA, BGRA, 565) > Prepare to use jpeg_skip_scanlines (when it is upstreamed) > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/b60c3f8291529303299262dba19b1a896060bd2d > > Committed: https://skia.googlesource.com/skia/+/f8bf9181d7b0463c8e371755cfbb9ece90b34fc5 > > Committed: https://skia.googlesource.com/skia/+/e9e3ee33f30c14c31afd5fc3fe4dda7f15783c75 > > Committed: https://skia.googlesource.com/skia/+/40141b57f061fbfcc2fa38da942d9efe25aca4d0 TBR=scroggo@google.com,djsollen@google.com,emmaleer@google.com,msarett@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Committed: https://skia.googlesource.com/skia/+/94a2fbc6cf7cc14d7e6149eb22455db6aca06b8f

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+189 lines, -29362 lines) Patch
M DEPS View 1 chunk +0 lines, -5 lines 0 comments Download
M dm/DM.cpp View 1 chunk +1 line, -1 line 0 comments Download
M gyp/codec.gyp View 2 chunks +1 line, -17 lines 0 comments Download
D gyp/libjpeg-turbo.gyp View 1 chunk +0 lines, -341 lines 0 comments Download
D gyp/yasm.gyp View 1 chunk +0 lines, -644 lines 0 comments Download
M src/codec/SkCodec.cpp View 2 chunks +0 lines, -4 lines 0 comments Download
M src/codec/SkJpegCodec.h View 2 chunks +2 lines, -7 lines 0 comments Download
M src/codec/SkJpegCodec.cpp View 12 chunks +170 lines, -151 lines 0 comments Download
M src/codec/SkJpegDecoderMgr.h View 1 chunk +0 lines, -1 line 0 comments Download
M src/codec/SkJpegDecoderMgr.cpp View 2 chunks +14 lines, -1 line 0 comments Download
M src/codec/SkJpegUtility_codec.h View 1 chunk +0 lines, -1 line 0 comments Download
M src/codec/SkJpegUtility_codec.cpp View 1 chunk +1 line, -1 line 0 comments Download
D third_party/yasm/README.skia View 1 chunk +0 lines, -138 lines 0 comments Download
D third_party/yasm/config/android/Makefile View 1 chunk +0 lines, -3822 lines 0 comments Download
D third_party/yasm/config/android/config.h View 1 chunk +0 lines, -173 lines 0 comments Download
D third_party/yasm/config/android/libyasm-stdint.h View 1 chunk +0 lines, -9 lines 0 comments Download
D third_party/yasm/config/chromeos/Makefile View 1 chunk +0 lines, -3822 lines 0 comments Download
D third_party/yasm/config/chromeos/config.h View 1 chunk +0 lines, -173 lines 0 comments Download
D third_party/yasm/config/chromeos/libyasm-stdint.h View 1 chunk +0 lines, -9 lines 0 comments Download
D third_party/yasm/config/ios/Makefile View 1 chunk +0 lines, -3822 lines 0 comments Download
D third_party/yasm/config/ios/config.h View 1 chunk +0 lines, -173 lines 0 comments Download
D third_party/yasm/config/ios/libyasm-stdint.h View 1 chunk +0 lines, -9 lines 0 comments Download
D third_party/yasm/config/linux/Makefile View 1 chunk +0 lines, -3822 lines 0 comments Download
D third_party/yasm/config/linux/config.h View 1 chunk +0 lines, -173 lines 0 comments Download
D third_party/yasm/config/linux/libyasm-stdint.h View 1 chunk +0 lines, -9 lines 0 comments Download
D third_party/yasm/config/mac/Makefile View 1 chunk +0 lines, -3822 lines 0 comments Download
D third_party/yasm/config/mac/config.h View 1 chunk +0 lines, -173 lines 0 comments Download
D third_party/yasm/config/mac/libyasm-stdint.h View 1 chunk +0 lines, -9 lines 0 comments Download
D third_party/yasm/config/openbsd/Makefile View 1 chunk +0 lines, -3822 lines 0 comments Download
D third_party/yasm/config/openbsd/config.h View 1 chunk +0 lines, -165 lines 0 comments Download
D third_party/yasm/config/openbsd/libyasm-stdint.h View 1 chunk +0 lines, -9 lines 0 comments Download
D third_party/yasm/config/win/Makefile View 1 chunk +0 lines, -3822 lines 0 comments Download
D third_party/yasm/config/win/config.h View 1 chunk +0 lines, -173 lines 0 comments Download
D third_party/yasm/config/win/libyasm-stdint.h View 1 chunk +0 lines, -9 lines 0 comments Download
D third_party/yasm/config/win/stdint.h View 1 chunk +0 lines, -30 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
jvanverth1
Created Revert of Switch SkJpegCode to libjpeg-turbo
5 years, 5 months ago (2015-07-02 17:36:00 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1226543003/1
5 years, 5 months ago (2015-07-02 17:36:14 UTC) #2
commit-bot: I haz the power
5 years, 5 months ago (2015-07-02 17:40:30 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/94a2fbc6cf7cc14d7e6149eb22455db6aca06b8f

Powered by Google App Engine
This is Rietveld 408576698