robertphillips
188d44c549
Move SkPixelXorXfermode.h to include/client/android
...
This intended to make clients aware that the SkPixelXorXfermode is Android-specific
This needs to land after the Chromium CL https://codereview.chromium.org/1647953006/ (Add include/client/android to skia .gn & .gypi files)
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1645093002
Committed: https://skia.googlesource.com/skia/+/aa9cb62901075e9d2858353cb041e5e6d4719614
Review URL: https://codereview.chromium.org/1645093002
2016-02-01 04:54:15 -08:00
fmalita
d2a4271c75
Fix S32_D32_constX_shaderproc assert
...
After https://crrev.com/1642273002 , fFilterOneY includes a rounding bias
which is not accounted for in the debug-mode S32_D32_constX_shaderproc
check.
Update the S32_D32_constX_shaderproc assert to use the (biased)
SkBitmapProcStateAutoMapper utility.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1655823002
R=reed@google.com
BUG=skia:4874
Review URL: https://codereview.chromium.org/1655823002
2016-02-01 04:48:39 -08:00
caryclark
2baa84b7ba
batch merge new edges
...
Find where newly introduced edges go in the edge list once,
then stitch all of them into the edge list.
R=reed@google.com
BUG=573166
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1647223002
Committed: https://skia.googlesource.com/skia/+/ae658e15477df86d1a864feb48d0274af2784f40
Review URL: https://codereview.chromium.org/1647223002
2016-02-01 04:34:58 -08:00
fmalita
bef1adfb56
Suppress SkBitmapProcState assert temporarily
...
TBR=jcgregorio@google.com ,reed@google.com
BUG=skia:4874
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1658573002
Review URL: https://codereview.chromium.org/1658573002
2016-02-01 03:20:27 -08:00
yujieqin
86e6b55b7c
Only define NO_POSIX_MEMALIGN for arm
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1647893003
Review URL: https://codereview.chromium.org/1647893003
2016-02-01 02:56:40 -08:00
fmalita
1b46a57365
Remove SkDEPRECATED_InstallDiscardablePixelRef from SkPictureShader
...
Use an SkImage generator/shader instead.
R=reed@google.com
Committed: https://skia.googlesource.com/skia/+/fa694ad067976a5b713eec0b6f92ba5efeb0aa9d
Review URL: https://codereview.chromium.org/1405103003
2016-02-01 02:34:03 -08:00
mtklein
507ef6d681
SkNx Load/store: take any pointer.
...
This means we can remove a lot of explicit casts in code that uses SkNx.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1650653002
CQ_EXTRA_TRYBOTS=client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot
Review URL: https://codereview.chromium.org/1650653002
2016-01-31 08:02:47 -08:00
update-skps
b1c32b7bde
Update SKP version
...
Automatic commit by the RecreateSKPs bot.
TBR=
NO_MERGE_BUILDS
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1651803002
Review URL: https://codereview.chromium.org/1651803002
2016-01-31 00:30:55 -08:00
fmalita
8a8eb5242b
Remove SkBitmapProcStateAutoMapper's overflow check
...
(follow-up to https://codereview.chromium.org/1642273002 )
Add an optional SkPoint outparam, and relocate the overflow check to
the only client which needs it.
R=reed@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1650403002
Review URL: https://codereview.chromium.org/1650403002
2016-01-30 18:56:34 -08:00
reed
395eabeb0e
float components in xfermodes
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1623483002
TBR=mtklein
Review URL: https://codereview.chromium.org/1634273002
2016-01-30 18:52:31 -08:00
caryclark
afd25f703d
combine vertical overlapping edges
...
Paths outside clips, and sometimes paths inside clips, devolve
to multiple adjacent or overlapping vertical edges. Combine
these edges when possible to reduce the overall edge count.
R=reed@google.com
BUG=573166
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1654433002
Review URL: https://codereview.chromium.org/1654433002
2016-01-30 14:07:20 -08:00
msarett
02a6bfaaaa
Revert of batch merge new edges (patchset #1 id:1 of https://codereview.chromium.org/1647223002/ )
...
Reason for revert:
Breaks the roll.
Original issue's description:
> batch merge new edges
>
> Find where newly introduced edges go in the edge list once,
> then stitch all of them into the edge list.
>
> R=reed@google.com
> BUG=573166
> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1647223002
>
> Committed: https://skia.googlesource.com/skia/+/ae658e15477df86d1a864feb48d0274af2784f40
TBR=reed@google.com ,caryclark@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=573166
Review URL: https://codereview.chromium.org/1654593002
2016-01-30 10:16:31 -08:00
caryclark
70e6d6074a
add new tests
...
These tests are for upcoming changes to optimize
the path edge list.
TBR=reed@google.com
BUG=573166
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1651573002
Review URL: https://codereview.chromium.org/1651573002
2016-01-30 10:11:21 -08:00
fmalita
653c12d773
Add sampler bias for the nofilter/translate specializations
...
Convert SkBitmapProcState::setupForTranslate() to use
SkBitmapProcStateAutoMapper. This adds bias for the translate procs:
Clamp_S32_D32_nofilter_trans_shaderproc,
Repeat_S32_D32_nofilter_trans_shaderproc
Since the original impl checks for int overflow, extend
SkBitmapProcStateAutoMapper to detect this condition.
BUG=chromium:581870
R=reed@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1642273002
Review URL: https://codereview.chromium.org/1642273002
2016-01-30 10:06:46 -08:00
scroggo
e645965a7c
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
2016-01-30 10:06:11 -08:00
bsalomon
8e74f80ca1
Add SkImage factory method that forces image to be resolved to a texture.
...
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1631053003
Review URL: https://codereview.chromium.org/1631053003
2016-01-30 10:01:40 -08:00
reed
9cd016e9b6
allow the caller to specified raster-surface rowbytes.
...
along the way, simplify how we copy the surface's bitmap
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1643873002
Review URL: https://codereview.chromium.org/1643873002
2016-01-30 10:01:06 -08:00
caryclark
ae658e1547
batch merge new edges
...
Find where newly introduced edges go in the edge list once,
then stitch all of them into the edge list.
R=reed@google.com
BUG=573166
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1647223002
Review URL: https://codereview.chromium.org/1647223002
2016-01-30 09:59:18 -08:00
ethannicholas
2279325d53
added support for PLS path rendering
...
BUG=skia:3555
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1541903002
Committed: https://skia.googlesource.com/skia/+/7df3f5e127f8016d17b637cc48a6a4718f1a6822
Review URL: https://codereview.chromium.org/1541903002
2016-01-30 09:59:10 -08:00
scroggo
ad38ed6003
Revert of Treat bad values passed to --images as a fatal error (patchset #17 id:320001 of https://codereview.chromium.org/1611323004/ )
...
Reason for revert:
Speculative to fix windows bots
Original issue's description:
> Treat bad values passed to --images as a fatal error
>
> If an option is passed to --images that is either a non-existent path or
> a folder with no images matching the supported types, assume this is
> an error and exit, so they can supply a valid path instead.
>
> Share code between DM and nanobench in SkCommonFlags.
>
> nanobench now behaves more like DM - it will check a directory for
> images that match the supported extensions.
>
> Only consider image paths ending in RAW suffixes as images if
> SK_CODE_DECODES_RAW is defined. This prevents us from seeing failure
> to decode errors on platforms that cannot decode it.
>
> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1611323004
>
> Committed: https://skia.googlesource.com/skia/+/7579786f3bd5a8fda84a1abc45b16213c3371f93
TBR=mtklein@google.com ,borenet@google.com,msarett@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
# Not skipping CQ checks because original CL landed more than 1 days ago.
Review URL: https://codereview.chromium.org/1653543002
2016-01-29 14:41:55 -08:00
scroggo
e82a6c7e71
Revert of Update SK_IMAGE_VERSION to test RAW (patchset #1 id:1 of https://codereview.chromium.org/1612113002/ )
...
Reason for revert:
https://skia.googlesource.com/skia/+/7579786f3bd5a8fda84a1abc45b16213c3371f93 seems to have caused problems, but it allows us to test RAW. In order to reland this, we'll need to reland that one, or have a simpler fix to disable RAW on platforms where it's not supported (e.g. ChromeOS)
Original issue's description:
> Update SK_IMAGE_VERSION to test RAW
>
> Once https://codereview.chromium.org/1520403003/ lands, we can start
> testing RAW images. I have already added RAW images to a folder named
> v2/; this updates the version so the script (once written) will check
> the new version.
>
> Depends on skbug.com/4558
>
> BUG=skia:4829
> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1612113002
>
> Committed: https://skia.googlesource.com/skia/+/317a45e879889f9ed876549a10bbfd51aab41b45
TBR=borenet@google.com ,msarett@google.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:4829
Review URL: https://codereview.chromium.org/1651603004
2016-01-29 14:32:32 -08:00
caryclark
703348f5fa
fix teeny dashed path bug
...
If the path dashed is sufficiently small, there may be no
segments generated to dash. Check for an empty segment list.
R=reed@google.com
BUG=skia:4871
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1645613006
Review URL: https://codereview.chromium.org/1645613006
2016-01-29 09:54:20 -08:00
djsollen
f2b340fc88
Consolidate SK_CRASH and sk_throw into SK_ABORT
...
TBR=reed@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1648343003
Review URL: https://codereview.chromium.org/1648343003
2016-01-29 08:51:04 -08:00
robertphillips
2a1d401bf2
Move SkAvoidXfermode over from Android
...
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1649503002
Review URL: https://codereview.chromium.org/1649503002
2016-01-29 08:44:17 -08:00
ethannicholas
c05f7c3d57
fix for skiaserve build error
...
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1651443002
TBR=jcgregorio@google.com
Review URL: https://codereview.chromium.org/1651443002
2016-01-29 08:02:34 -08:00
scroggo
4d9eaea174
Print a message when DM finishes
...
When running from the command line, it is obvious when DM finishes,
since it brings the prompt back up. But in XCode, it just sits
there, so my glance doesn't let me know that it's finished. Make
it more obvious that DM is done.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1650113002
Review URL: https://codereview.chromium.org/1650113002
2016-01-29 07:48:33 -08:00
ethannicholas
78fc22af41
added support for more features in JSON (blurs, dashing, different path fill types, etc.)
...
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1644903003
Review URL: https://codereview.chromium.org/1644903003
2016-01-29 07:15:08 -08:00
robertphillips
6b2c3461b0
Revert of Move SkPixelXorXfermode.h to include/client/android (patchset #5 id:80001 of https://codereview.chromium.org/1645093002/ )
...
Reason for revert:
GN woes
Original issue's description:
> Move SkPixelXorXfermode.h to include/client/android
>
> This intended to make clients aware that the SkPixelXorXfermode is Android-specific
>
> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1645093002
>
> Committed: https://skia.googlesource.com/skia/+/aa9cb62901075e9d2858353cb041e5e6d4719614
TBR=scroggo@google.com ,reed@google.com,djsollen@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1644903004
2016-01-29 05:34:26 -08:00
reed
fbc1e296b2
starter procs for blending with pm4f
...
curr/maxrss loops min median mean max stddev samples config bench
8/8 MB 4 87.1µs 91µs 89.8µs 92µs 2% ▇▇▇▇█▇▅▁▁▁ nonrendering xfer4f_srcover_N_opaque_linear
9/9 MB 2 196µs 196µs 215µs 383µs 27% ▁▁▁▁█▁▁▁▁▁ nonrendering xfer4f_srcover_N_opaque_srgb
9/9 MB 1 313µs 313µs 313µs 313µs 0% ▁▄▅▅▅▂████ nonrendering xfer4f_srcover_N_alpha_linear
9/9 MB 1 580µs 580µs 582µs 602µs 1% ▁▁▁▁▁▁▂▁▁█ nonrendering xfer4f_srcover_N_alpha_srgb
9/9 MB 23 13.1µs 13.1µs 13.1µs 13.1µs 0% ▆▄▄█▂▂▂▁▂▁ nonrendering xfer4f_srcover_1_opaque_linear
9/9 MB 23 13.2µs 13.2µs 13.2µs 13.2µs 0% █▄▂▁▃▁▂▂▂▂ nonrendering xfer4f_srcover_1_opaque_srgb
9/9 MB 2 178µs 183µs 183µs 185µs 1% ▇▇▇█▇▇▇▇▇▁ nonrendering xfer4f_srcover_1_alpha_linear
9/9 MB 1 517µs 517µs 517µs 517µs 0% ▇█▄▃▄▁▂▁▂▄ nonrendering xfer4f_srcover_1_alpha_srgb
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1642703003
TBR=
landing now so these incremental types/functions can be used to collaborate with herb's work. nothing is active at this point
Review URL: https://codereview.chromium.org/1642703003
2016-01-29 05:22:59 -08:00
robertphillips
aa9cb62901
Move SkPixelXorXfermode.h to include/client/android
...
This intended to make clients aware that the SkPixelXorXfermode is Android-specific
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1645093002
Review URL: https://codereview.chromium.org/1645093002
2016-01-29 05:11:14 -08:00
djsollen
817c72ad40
Fix broken Android framework builds
...
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1641173003
Review URL: https://codereview.chromium.org/1641173003
2016-01-28 13:57:02 -08:00
fmalita
54d60293ce
SaveFlags begone!
...
This is now dead code (post https://codereview.chromium.org/1636103002 ).
R=reed@google.com ,robertphillips@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1648773002
Review URL: https://codereview.chromium.org/1648773002
2016-01-28 13:14:53 -08:00
benjaminwagner
5962957d95
Fix typo on trooper page.
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1642183002
NOTRY=true
DOCS_PREVIEW= https://skia.org/?cl=1642183002
Review URL: https://codereview.chromium.org/1642183002
2016-01-28 12:58:02 -08:00
benjaminwagner
900901fb1b
Work around exasperating Google3 Android build error.
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1649493005
Review URL: https://codereview.chromium.org/1649493005
2016-01-28 12:55:31 -08:00
scroggo
317a45e879
Update SK_IMAGE_VERSION to test RAW
...
Once https://codereview.chromium.org/1520403003/ lands, we can start
testing RAW images. I have already added RAW images to a folder named
v2/; this updates the version so the script (once written) will check
the new version.
Depends on skbug.com/4558
BUG=skia:4829
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1612113002
Review URL: https://codereview.chromium.org/1612113002
2016-01-28 12:30:07 -08:00
halcanary
682ee016a3
SkPDF: roll back 53b1c09
and 05b48e2
...
See:
https://crrev.com/1552263003
https://crrev.com/1438503002
BUG=skia:257
BUG=skia:734
BUG=577794
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1645803003
Review URL: https://codereview.chromium.org/1645803003
2016-01-28 10:59:34 -08:00
halcanary
83024f99d6
tools/git-sync-deps: fix error in git syntax for changing origin
...
Error introduced in https://crrev.com/1475513004
NOTRY=true
Review URL: https://codereview.chromium.org/1646863002
2016-01-28 09:12:52 -08:00
robertphillips
5381b81af2
Fix SkPixelXorXfermode serialization bug
...
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1647963002
Review URL: https://codereview.chromium.org/1647963002
2016-01-28 08:50:26 -08:00
scroggo
7579786f3b
Treat bad values passed to --images as a fatal error
...
If an option is passed to --images that is either a non-existent path or
a folder with no images matching the supported types, assume this is
an error and exit, so they can supply a valid path instead.
Share code between DM and nanobench in SkCommonFlags.
nanobench now behaves more like DM - it will check a directory for
images that match the supported extensions.
Only consider image paths ending in RAW suffixes as images if
SK_CODE_DECODES_RAW is defined. This prevents us from seeing failure
to decode errors on platforms that cannot decode it.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1611323004
Review URL: https://codereview.chromium.org/1611323004
2016-01-28 08:41:10 -08:00
senorblanco
d8ff5b336e
Image filters: Make a recursive, forward-mapping bounds
...
traversal which respects the CropRect. This is useful when
you want the device-space bounds of a primitive after
filtering. (This may also eventually subsume
computeFastBounds()).
This CL generalizes filterBounds() and onFilterBounds() to
take a mapping direction. It also makes filterBounds()
responsible for calling onFilterNodeBounds() and applying
the crop rect, simplifying onFilterBounds().
BUG=skia:4627
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1612953004
Review URL: https://codereview.chromium.org/1612953004
2016-01-28 08:23:02 -08:00
adaubert
382f8bdbf0
Disable verbose mode for DNG SDK on mac/ios.
...
The issue was, that the qDNGReportErrors was not disabled in xcode_settings.
With that also removed the complexety of the dng_sdk.gyp and unified the flags handling.
BUG=26821211
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1641553004
Review URL: https://codereview.chromium.org/1641553004
2016-01-28 07:56:15 -08:00
bsalomon
b8909d3df7
Revert "Enable GL_CHROMIUM_bind_uniform_location, command buffer is fixed"
...
This reverts commit 1dc85cda5e
.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1645013002
TBR=
Review URL: https://codereview.chromium.org/1645013002
2016-01-28 07:09:52 -08:00
robertphillips
98124ef2f9
Make SkPixelXorXfermode's opColor be SkPMColor
...
Xoring an SkColor with 2 SkPMColors creates rendering inconsistencies on Macs and some Android devices in 8888.
AFAICT Android doesn't compensate for this so we may be changing SkPixelXorXfermode's behavior on Android.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1646453003
Review URL: https://codereview.chromium.org/1646453003
2016-01-28 06:41:11 -08:00
joshualitt
b35c82dc94
Add a background timing thread to kilobench
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1612513002
Review URL: https://codereview.chromium.org/1612513002
2016-01-28 06:26:35 -08:00
jcgregorio
21ab1209f5
skiaserve: Clean up flags.
...
Remove unused flags.
Add --port and --source.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1643963002
Review URL: https://codereview.chromium.org/1643963002
2016-01-28 06:24:19 -08:00
svaisanen
c47635e0ed
Adding support for parsing extended gpu config parameters in nanobench
...
BUG=skia:2992
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1603063005
Review URL: https://codereview.chromium.org/1603063005
2016-01-28 06:05:43 -08:00
jcgregorio
6a2046ea34
skiaserve: Add CORS header, and debugger code is served over HTTPS.
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1646683002
Review URL: https://codereview.chromium.org/1646683002
2016-01-28 05:31:31 -08:00
fmalita
4c5f1f6114
Remove SK_SUPPORT_LEGACY_SAVEFLAGS define on Android
...
Android is the only platform still using SK_SUPPORT_LEGACY_SAVEFLAGS
(indirectly, due to SK_SUPPORT_LEGACY_CLIPTOLAYERFLAG).
At this point it should be safe to remove the define.
R=djsollen@google.com ,scroggo@google.com,reed@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1636103002
Review URL: https://codereview.chromium.org/1636103002
2016-01-27 18:46:30 -08:00
mtklein
62358e77c2
SkPngFilters: guard with libpng >= 1.5.7
...
The read_filter array was added in libpng-1.5.7.
No point to these filter methods if we can't plug them into libpng.
TBR=msarett@google.com
NOTRY=true
NOPRESUBMIT=true
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1640933004
Review URL: https://codereview.chromium.org/1640933004
2016-01-27 18:26:31 -08:00
mtklein
1c971faab7
SkPngFilters: use normal pointer types.
...
TBR=msarett@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1641793004
Review URL: https://codereview.chromium.org/1641793004
2016-01-27 16:10:53 -08:00