History log of /external/skia/src/codec/SkRawCodec.cpp
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
20de35a0fd855c60b40a7d5d513bbe8e19fda49b 02-Dec-2016 Leon Scroggins III <scroggo@google.com> DO NOT MERGE Do not create an SkRawCodec with zero dimensions

BUG:33300701

Cherry-pick of Ifb3a824a36998c5e626c4ad58466845f49d18ebf
Conflicts:
src/codec/SkRawCodec.cpp

Original message:
"""
DO NOT MERGE Do not create an SkRawCodec with zero dimensions
Add a test

BUG=skia:3534
BUG=b/33300701

Change-Id: Ifb3a824a36998c5e626c4ad58466845f49d18ebf
Reviewed-on: https://skia-review.googlesource.com/5568
Commit-Queue: Leon Scroggins <scroggo@google.com>
Reviewed-by: Matt Sarett <msarett@google.com>
"""
/external/skia/src/codec/SkRawCodec.cpp
a0e519dd4d837a9f8ebab058de90c57387902738 18-Apr-2016 Jaesung Chung <jaesung@google.com> skia: fix a regression in decoding DNG

Since PIEX can now return an uncompressed RGB thumbnail, check the type
of the image before treating it as a JPEG.

this commit was already reviewed and cherry-picked from skia project,
https://codereview.chromium.org/1883783002/

Bug: 28119810, Bug: 26894354
Change-Id: Ib30d6f73518c77034b23573ac9ac018df2efac66
/external/skia/src/codec/SkRawCodec.cpp
c39d70a44e4412dfc355849d2b6d7998d02cd3ed 10-Mar-2016 scroggo <scroggo@chromium.org> Add a quick check to the TIFF header of DNG image

cherry-pick of d6215cf4a5f416cf0b64a4fbba95c519f03fe467 into chrome/m50
for nyc-dev

BUG=b/27475341
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1784843002
NOTREECHECKS=true
NOTRY=true
NOPRESUBMIT=true

Review URL: https://codereview.chromium.org/1784843002
/external/skia/src/codec/SkRawCodec.cpp
8983f89e3fd8b8556d9a76ebebeec569d68fd5aa 10-Mar-2016 scroggo <scroggo@chromium.org> Limit the maximum buffer size of SkRawBufferedStream

cherry-pick of c04df21bb624e299440ce8b56495ec109d4e824e into chrome/m50
for nyc-dev.

BUG=b/27475341
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1783683003
NOTREECHECKS=true
NOTRY=true
NOPRESUBMIT=true

Review URL: https://codereview.chromium.org/1783683003
/external/skia/src/codec/SkRawCodec.cpp
520dadc72f90c52b29ddfc3d39f92176862eb3e0 03-Mar-2016 scroggo <scroggo@google.com> Verify the full size returned by PIEX

cherry-pick of d0e0885d90cecd43b717d89b565d9a713cffd90c in master

Original commit message below:

========================================================================

Verify the full size returned by PIEX

As the full size info is only optional, one needs to verify the values
before using them

BUG=b/27422694
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1757273002

Review URL: https://codereview.chromium.org/1757273002
NOTREECHECKS=true
NOTRY=true
NOPRESUBMIT=true

Review URL: https://codereview.chromium.org/1757353002
/external/skia/src/codec/SkRawCodec.cpp
ef0be3f41b04f5b882180634817bb767afca00cf 03-Mar-2016 scroggo <scroggo@chromium.org> Fix exception handling in SkRawCodec

cherry-pick of f236ee4e44662e519db4b4997eee5d2bc8543f9c in master

BUG=b/27425935

Original commit message below:

========================================================================

Enable RAW codec for Windows

* Fix the exception catching
* Set preprocessor differently for MSVC

BUG=skia:4889(b/26958348)
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1738913002

Committed: https://skia.googlesource.com/skia/+/474e4c3dd28b67f590851321f15d9983ef7fd031

Review URL: https://codereview.chromium.org/1738913002

========================================================================
NOTREECHECKS=true
NOTRY=true
NOPRESUBMIT=true

Review URL: https://codereview.chromium.org/1757493002
/external/skia/src/codec/SkRawCodec.cpp
9ca27f39ddf7dafdc8ce97dba0fc9c4d1f07b7ed 23-Feb-2016 msarett <msarett@google.com> Remove unnecessary GOOGLE3 ifdefs from SkRawCodec

These should be fine now that SkJpegCodec is building in GOOGLE3.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1718343002

Review URL: https://codereview.chromium.org/1718343002
/external/skia/src/codec/SkRawCodec.cpp
2d172ebb29d7afe47aa179211c9f92187f28ec52 23-Feb-2016 yujieqin <yujieqin@google.com> Use new API instead of the deprecated ones.

This change will allow us to remove the old API in follow up clean up.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1728553002

Review URL: https://codereview.chromium.org/1728553002
/external/skia/src/codec/SkRawCodec.cpp
46d2aa824c0a0ee8218d90e821786bd51a63be1e 17-Feb-2016 ebrauer <ebrauer@google.com> Updates Piex and uses it to obtain the DNG dimensions.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1659873002

Review URL: https://codereview.chromium.org/1659873002
/external/skia/src/codec/SkRawCodec.cpp
9c7a8a464894436fc71a15b5419e818905226cdf 05-Feb-2016 yujieqin <yujieqin@google.com> Optimize the SkRawStream when the input is an asset stream

BUG=b/26841494
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1645963002

Review URL: https://codereview.chromium.org/1645963002
/external/skia/src/codec/SkRawCodec.cpp
22000d1f8e2c554aec2f50a5a4cad0a5d04ca41c 02-Feb-2016 yujieqin <yujieqin@google.com> Avoid too small reads to bufferMoreData()

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1644893003

Committed: https://skia.googlesource.com/skia/+/fd918e8c739065fa467cf6614181d3a2c5dcadcb

Review URL: https://codereview.chromium.org/1644893003
/external/skia/src/codec/SkRawCodec.cpp
2ac722f1f8b675eebc89ca1c2d7d44cfc887b58a 02-Feb-2016 jcgregorio <jcgregorio@google.com> Revert of Avoid too small reads to bufferMoreData() (patchset #4 id:60001 of https://codereview.chromium.org/1644893003/ )

Reason for revert:
Crashed both Nexus 5s:

https://build.chromium.org/p/client.skia.android/builders/Perf-Android-GCC-Nexus5-CPU-NEON-Arm7-Release/builds/3355
https://build.chromium.org/p/client.skia.android/builders/Perf-Android-GCC-Nexus5-CPU-NEON-Arm7-Release/builds/3354

Original issue's description:
> Avoid too small reads to bufferMoreData()
>
> BUG=skia:
> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1644893003
>
> Committed: https://skia.googlesource.com/skia/+/fd918e8c739065fa467cf6614181d3a2c5dcadcb

TBR=msarett@google.com,scroggo@google.com,yujieqin@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/1656303002
/external/skia/src/codec/SkRawCodec.cpp
fd918e8c739065fa467cf6614181d3a2c5dcadcb 02-Feb-2016 yujieqin <yujieqin@google.com> Avoid too small reads to bufferMoreData()

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1644893003

Review URL: https://codereview.chromium.org/1644893003
/external/skia/src/codec/SkRawCodec.cpp
e645965a7c8bc1f439fa4d7655e8c9fda0b969ca 30-Jan-2016 scroggo <scroggo@google.com> Use proper row step in SkRawCodec
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1643373002

Review URL: https://codereview.chromium.org/1643373002
/external/skia/src/codec/SkRawCodec.cpp
fda27a977d4dde37792f675686e8ff977b41958c 27-Jan-2016 yujieqin <yujieqin@google.com> Fix Windows build

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1633413003

Review URL: https://codereview.chromium.org/1633413003
/external/skia/src/codec/SkRawCodec.cpp
076d83d09a5717913cfecabac0440b6c854ca86d 27-Jan-2016 yujieqin <yujieqin@google.com> Enable RAW codec for Windows

* Use new DNG SDK version from ASOP, which fixed some build issues.
* Fix SkRawCodec.
* Fix gyp files.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1641533004

Review URL: https://codereview.chromium.org/1641533004
/external/skia/src/codec/SkRawCodec.cpp
b84b5b42c1d848aa2b87216b9bda2b8c9e5781c1 27-Jan-2016 ebrauer <ebrauer@google.com> It derives the dng_host and overrides PerformAreaTask() to split the task into several sub-tasks which get added to SkTaskGroup.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1634763002

Review URL: https://codereview.chromium.org/1634763002
/external/skia/src/codec/SkRawCodec.cpp
24716be1d1e6d7fba6ee8604823ce326fd69be1e 27-Jan-2016 yujieqin <yujieqin@google.com> Fix complier issue [-Werror, -Wvla]

BUG=skia:4861
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1638323002

Review URL: https://codereview.chromium.org/1638323002
/external/skia/src/codec/SkRawCodec.cpp
916de9ff18cf3caa29c0821b55244060b6f84f9d 25-Jan-2016 yujieqin <yujieqin@google.com> Add RAW decoding into Skia.

TBR=reed@google.com

BUG=skia:

(Based on the work from ebrauer in https://codereview.chromium.org/1459473007)
(Based on the work from adaubert in https://codereview.chromium.org/1494003003)
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1520403003

Committed: https://skia.googlesource.com/skia/+/6bd8639f8c142eedf543f4e5f3b02d2bf11df308

Review URL: https://codereview.chromium.org/1520403003
/external/skia/src/codec/SkRawCodec.cpp
c85a9fde77f91979afea0efa38f63d8efceb4138 25-Jan-2016 msarett <msarett@google.com> Revert of Prototype of RAW decoding in Skia. (patchset #32 id:610001 of https://codereview.chromium.org/1520403003/ )

Reason for revert:
A few build failures on Chrome OS/Android.

Original issue's description:
> Add RAW decoding into Skia.
>
> TBR=reed@google.com
>
> BUG=skia:
>
> (Based on the work from ebrauer in https://codereview.chromium.org/1459473007)
> (Based on the work from adaubert in https://codereview.chromium.org/1494003003)
> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1520403003
>
> Committed: https://skia.googlesource.com/skia/+/6bd8639f8c142eedf543f4e5f3b02d2bf11df308

TBR=scroggo@google.com,adaubert@google.com,yujieqin@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/1635443002
/external/skia/src/codec/SkRawCodec.cpp
6bd8639f8c142eedf543f4e5f3b02d2bf11df308 25-Jan-2016 yujieqin <yujieqin@google.com> Add RAW decoding into Skia.

TBR=reed@google.com

BUG=skia:

(Based on the work from ebrauer in https://codereview.chromium.org/1459473007)
(Based on the work from adaubert in https://codereview.chromium.org/1494003003)
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1520403003

Review URL: https://codereview.chromium.org/1520403003
/external/skia/src/codec/SkRawCodec.cpp