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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 1624513003: Add RAW decoding support to Skia (Closed)

Created:
4 years, 11 months ago by msarett
Modified:
4 years, 11 months ago
Reviewers:
scroggo, reed1
CC:
reviews_skia.org, yujieqin, adaubert
Base URL:
https://skia.googlesource.com/skia.git@master
Target Ref:
refs/heads/master
Project:
skia
Visibility:
Public.

Description

Patch Set 1 : Disable RAW tests for SkImageDecoder #

Patch Set 2 : Fix sample_dng in resources, Treat less warnings as errors #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1097 lines, -2 lines) Patch
M DEPS View 1 chunk +3 lines, -0 lines 0 comments Download
M dm/DM.cpp View 4 chunks +23 lines, -2 lines 0 comments Download
M gyp/android_deps.gyp View 1 chunk +24 lines, -0 lines 0 comments Download
M gyp/codec.gyp View 1 chunk +51 lines, -0 lines 0 comments Download
M gyp/common_variables.gypi View 1 chunk +5 lines, -0 lines 0 comments Download
A gyp/dng_sdk.gyp View 1 1 chunk +284 lines, -0 lines 0 comments Download
A gyp/piex.gyp View 1 1 chunk +104 lines, -0 lines 0 comments Download
M include/codec/SkEncodedFormat.h View 1 chunk +1 line, -0 lines 0 comments Download
A resources/sample_1mp.dng View 1 Binary file 0 comments Download
M src/codec/SkAndroidCodec.cpp View 2 chunks +7 lines, -0 lines 0 comments Download
M src/codec/SkCodec.cpp View 2 chunks +8 lines, -0 lines 0 comments Download
A src/codec/SkRawAdapterCodec.h View 1 chunk +42 lines, -0 lines 0 comments Download
A src/codec/SkRawAdapterCodec.cpp View 1 chunk +30 lines, -0 lines 0 comments Download
A src/codec/SkRawCodec.h View 1 chunk +60 lines, -0 lines 0 comments Download
A src/codec/SkRawCodec.cpp View 1 chunk +450 lines, -0 lines 0 comments Download
M tests/CodexTest.cpp View 2 chunks +5 lines, -0 lines 0 comments Download

Depends on Patchset:

Issue 1188223002 Patch 80001

Messages

Total messages: 14 (8 generated)
msarett
4 years, 11 months ago (2016-01-22 18:45:21 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1624513003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1624513003/1
4 years, 11 months ago (2016-01-22 18:45:30 UTC) #4
commit-bot: I haz the power
Try jobs failed on following builders: Build-Mac10.9-Clang-Arm7-Debug-iOS-Trybot on client.skia.compile (JOB_FAILED, http://build.chromium.org/p/client.skia.compile/builders/Build-Mac10.9-Clang-Arm7-Debug-iOS-Trybot/builds/930)
4 years, 11 months ago (2016-01-22 18:46:44 UTC) #6
reed1
api lgtm perhaps we should start to add some dox for each encoding format (e.g. ...
4 years, 11 months ago (2016-01-22 19:06:19 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1624513003/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1624513003/20001
4 years, 11 months ago (2016-01-22 19:23:08 UTC) #12
commit-bot: I haz the power
4 years, 11 months ago (2016-01-22 19:24:10 UTC) #14 
Try jobs failed on following builders:
  Build-Ubuntu-GCC-Arm7-Debug-Android-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-GCC-Arm...)

Powered by Google App Engine
This is Rietveld 408576698