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

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

Created:
5 years, 5 months ago by msarett
Modified:
5 years, 5 months ago
Reviewers:
scroggo, emmaleer, djsollen
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 #28 id:710001 of https://codereview.chromium.org/1180983002/) Reason for revert: Broke iOS build. 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 TBR=scroggo@google.com,djsollen@google.com,emmaleer@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Committed: https://skia.googlesource.com/skia/+/aa2a7de0ffaf8f864a7afc68bbd34ccb25876246

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+189 lines, -29361 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, -340 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)
msarett
Created Revert of Switch SkJpegCode to libjpeg-turbo
5 years, 5 months ago (2015-07-01 20:10:43 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1213093003/1
5 years, 5 months ago (2015-07-01 20:10:57 UTC) #2
commit-bot: I haz the power
5 years, 5 months ago (2015-07-01 20:11:12 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/aa2a7de0ffaf8f864a7afc68bbd34ccb25876246

Powered by Google App Engine
This is Rietveld 408576698