egdaniel [Tue, 27 Sep 2016 16:03:05 +0000 (09:03 -0700)]
Add command buffer submits before copy calls in vulkan.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2363773006
Review-Url: https://codereview.chromium.org/
2363773006
Eric Boren [Tue, 27 Sep 2016 15:30:48 +0000 (11:30 -0400)]
Fix dm/nanobench uploads for Gerrit-based trybots
BUG=skia:5719
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2710
Change-Id: I43f03b19a9d7223b67de7dc7c4efea8cb75dba05
Reviewed-on: https://skia-review.googlesource.com/2710
Reviewed-by: Eric Boren <borenet@google.com>
Commit-Queue: Eric Boren <borenet@google.com>
Mike Klein [Tue, 27 Sep 2016 15:21:15 +0000 (11:21 -0400)]
Turn on -Wnewline-eof.
I don't know _why_ Clang would like these .inc files to have a newline at the end of the file, but it seems a harmless way to silence the warning.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2711
Change-Id: I6c530ee5096c48c91ddf322aca916e70a0dd770b
Reviewed-on: https://skia-review.googlesource.com/2711
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
caryclark [Tue, 27 Sep 2016 15:41:33 +0000 (08:41 -0700)]
another attempt to fix nonrectangular
For hairlines extended by the end cap,
set the first point to the first curve
following a move, which has had its
position adjusted.
R=fmalita@chromium.org
BUG=617658
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2371143002
Review-Url: https://codereview.chromium.org/
2371143002
Mike Reed [Tue, 27 Sep 2016 15:07:11 +0000 (11:07 -0400)]
need to apply CTM to clip rect in arithmetic imagefilter
BUG=skia:5800
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2708
NOTRY=True
upload steps are failing, but the tests themselves passed
Change-Id: I0210d0095b4ffdd376f18fad895655d39714bf38
Reviewed-on: https://skia-review.googlesource.com/2708
Commit-Queue: Mike Reed <reed@google.com>
Reviewed-by: Florin Malita <fmalita@chromium.org>
borenet [Tue, 27 Sep 2016 15:22:03 +0000 (08:22 -0700)]
[task scheduler] Add uploads to tasks.json
BUG=skia:5719
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2376713002
Review-Url: https://codereview.chromium.org/
2376713002
borenet [Tue, 27 Sep 2016 15:21:44 +0000 (08:21 -0700)]
Remove gen_bench_expectations_from_codereview
It hasn't been used in a long time.
BUG=skia:2969
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2373803002
Review-Url: https://codereview.chromium.org/
2373803002
Mike Klein [Tue, 27 Sep 2016 14:52:35 +0000 (10:52 -0400)]
remove more stray semicolons from Vulkan code
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2707
Change-Id: Idc00efeda2f9901a6f33c987202be2a781b24269
Reviewed-on: https://skia-review.googlesource.com/2707
Reviewed-by: Mike Klein <mtklein@chromium.org>
egdaniel [Tue, 27 Sep 2016 15:00:53 +0000 (08:00 -0700)]
Add GrVkCopyPipeline to handle vulkan copies as draws
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2274663005
Review-Url: https://codereview.chromium.org/
2274663005
borenet [Tue, 27 Sep 2016 14:51:15 +0000 (07:51 -0700)]
Delete unused upload_*_results.py
BUG=skia:5719
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2371883004
Review-Url: https://codereview.chromium.org/
2371883004
branimir.vasic [Tue, 27 Sep 2016 14:37:20 +0000 (07:37 -0700)]
Add mips dsp arch variants to gyp_to_android.py.
Fixes Android build for variants mips32r2dsp-fp and mips32r2dspr2-fp by
enabling correct Android.mk file generation.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2359793002
Review-Url: https://codereview.chromium.org/
2359793002
Mike Klein [Tue, 27 Sep 2016 13:10:15 +0000 (09:10 -0400)]
We really shouldn't pass SkFontStyle to printf as %d.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2721
Change-Id: I362063bea3909b2581d8e9b7860d4ac321082808
Reviewed-on: https://skia-review.googlesource.com/2721
Commit-Queue: Ben Wagner <bungeman@google.com>
Reviewed-by: Ben Wagner <bungeman@google.com>
Mike Klein [Tue, 27 Sep 2016 13:34:10 +0000 (09:34 -0400)]
Remove stray semicolons.
Turns out function declarations don't end in semicolons...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2720
No public API changes.
TBR=reed@google.com
Change-Id: I72b56d52e1ff7fa6e89c295b0de8c46599791ebb
Reviewed-on: https://skia-review.googlesource.com/2720
Reviewed-by: Mike Klein <mtklein@chromium.org>
Commit-Queue: Mike Klein <mtklein@chromium.org>
borenet [Tue, 27 Sep 2016 13:52:26 +0000 (06:52 -0700)]
Add swarming task for upload_dm_results
BUG=skia:5719
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2360203004
Review-Url: https://codereview.chromium.org/
2360203004
Mike Klein [Tue, 27 Sep 2016 11:41:45 +0000 (07:41 -0400)]
Focus -Wno-over-aligned to just 32-bit x86 Android. (2)
I've even found the code that's making this happen, just don't know why.
I've added a test to assert that it's safe to assume malloc() is 8-byte aligned.
Test should compile this time.
CQ_INCLUDE_TRYBOTS=master.client.skia.android:Test-Android-Clang-NexusPlayer-CPU-Moorefield-x86-Release-GN_Android-Trybot
Change-Id: I48714b99670c20704adf4f7f216da0d60d7d9bcd
Reviewed-on: https://skia-review.googlesource.com/2662
Reviewed-on: https://skia-review.googlesource.com/2703
Reviewed-by: Mike Klein <mtklein@chromium.org>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Mike Reed [Tue, 27 Sep 2016 10:42:47 +0000 (10:42 +0000)]
Revert "Focus -Wno-over-aligned to just 32-bit x86 Android."
This reverts commit If8a2898ab3a77571622eb125c97f676e029b902c.
Reason for revert:
../../../../../work/skia/tests/OverAlignedTest.cpp: In function 'void test_OverAligned(skiatest::Reporter*, sk_gpu_test::GrContextFactory*)':
../../../../../work/skia/tests/OverAlignedTest.cpp:19:33: error: invalid operands of types 'void*' and 'int' to binary 'operator&'
REPORTER_ASSERT(r, SkIsAlign8(p));
^
ninja: build stopped: subcommand failed.
Original issue's description:
> Focus -Wno-over-aligned to just 32-bit x86 Android.
>
> I've even found the code that's making this happen, just don't know why.
> I've added a test to assert that it's safe to assume malloc() is 8-byte aligned.
>
> BUG=skia:
>
> GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2662
>
> CQ_INCLUDE_TRYBOTS=master.client.skia.android:Test-Android-Clang-NexusPlayer-CPU-Moorefield-x86-Release-GN_Android-Trybot
>
TBR=mtklein@chromium.org,bungeman@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Change-Id: Ic9b30ce980d8d5155528a6f2b4e1913e5fa95dc0
Reviewed-on: https://skia-review.googlesource.com/2702
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Mike Reed <reed@google.com>
brianosman [Tue, 27 Sep 2016 10:33:12 +0000 (03:33 -0700)]
Remove outdated comments about pre-sk_sp semantics
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2371603003
Review-Url: https://codereview.chromium.org/
2371603003
jvanverth [Tue, 27 Sep 2016 10:33:05 +0000 (03:33 -0700)]
Clamp Vulkan caps to INT_MAX
AMD sets some Vulkan caps values to 0xffffffff as a uint32_t, which
overflows our int values and causes merry havok.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2371923002
Review-Url: https://codereview.chromium.org/
2371923002
Mike Reed [Mon, 26 Sep 2016 22:51:24 +0000 (18:51 -0400)]
Revert "Revert "replace Arithmetic xfermode with imagefilter""
This reverts commit
10ff5bfa789b6b602464e8511fdf676c6f5b1bd4.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2680
TBR=
Change-Id: Iac2f4d48d227ff426ccf02b7eae280f382ad3580
Reviewed-on: https://skia-review.googlesource.com/2680
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Mike Reed <reed@google.com>
Mike Klein [Mon, 26 Sep 2016 20:51:59 +0000 (16:51 -0400)]
Focus -Wno-over-aligned to just 32-bit x86 Android.
I've even found the code that's making this happen, just don't know why.
I've added a test to assert that it's safe to assume malloc() is 8-byte aligned.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2662
CQ_INCLUDE_TRYBOTS=master.client.skia.android:Test-Android-Clang-NexusPlayer-CPU-Moorefield-x86-Release-GN_Android-Trybot
Change-Id: If8a2898ab3a77571622eb125c97f676e029b902c
Reviewed-on: https://skia-review.googlesource.com/2662
Commit-Queue: Ben Wagner <bungeman@google.com>
Reviewed-by: Ben Wagner <bungeman@google.com>
mtklein [Tue, 27 Sep 2016 01:20:57 +0000 (18:20 -0700)]
Revert of My take on SkAlign changes. (patchset #3 id:40001 of https://codereview.chromium.org/
2368293002/ )
Reason for revert:
Let's see if reverting this helps the roll.
Original issue's description:
> My take on SkAlign changes.
>
> Like the other change, it makes SkAlignN(x) macros work for pointers, and makes the macros themselves just syntax sugar for SkAlign<N>(x). We can still decide if we want to sed away the macros independently.
>
> This just does it in a somewhat less repetitive way, and adds some tests.
>
> BUG=skia:
> GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2368293002
>
> No public API changes.
> TBR=reed@google.com
>
> Committed: https://skia.googlesource.com/skia/+/
e1a5f4e292384046678edc5c1e360b3e13dc118c
TBR=cblume@chromium.org,mtklein@chromium.org
# 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/
2372083002
Mike Reed [Mon, 26 Sep 2016 20:40:57 +0000 (20:40 +0000)]
Revert "replace Arithmetic xfermode with imagefilter"
This reverts commit Ia3f3b721854c516f3b7f8c44f71f40a8a2eeb9b4.
Reason for revert: need to guard the no-gpu codepath
Original issue's description:
> replace Arithmetic xfermode with imagefilter
>
> chrome pre-cl: https://codereview.chromium.org/
2369023002/
>
> BUG=skia:
>
> GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2595
>
> Change-Id: Ia3f3b721854c516f3b7f8c44f71f40a8a2eeb9b4
> Reviewed-on: https://skia-review.googlesource.com/2595
> Commit-Queue: Mike Reed <reed@google.com>
> Reviewed-by: Florin Malita <fmalita@chromium.org>
> Reviewed-by: Robert Phillips <robertphillips@google.com>
>
TBR=robertphillips@google.com,fmalita@chromium.org,fmalita@google.com,reed@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Change-Id: I761799b594a0379c6bf356e6abc73552c3d19480
Reviewed-on: https://skia-review.googlesource.com/2661
Reviewed-by: Florin Malita <fmalita@chromium.org>
Commit-Queue: Florin Malita <fmalita@chromium.org>
ethannicholas [Mon, 26 Sep 2016 20:43:47 +0000 (13:43 -0700)]
Revert of Turned on SkSL->GLSL compiler (patchset #37 id:800001 of https://codereview.chromium.org/
2288033003/ )
Reason for revert:
Gold image breakage
Original issue's description:
> Turned on SkSL->GLSL compiler
> GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2288033003
>
> Committed: https://skia.googlesource.com/skia/+/
9b0fe3d125f237d9884732a48414fa85fc71b4e3
> Committed: https://skia.googlesource.com/skia/+/
b12b3c6908c62c908b3680be01e3b5bfd30de310
TBR=benjaminwagner@google.com,bsalomon@google.com,egdaniel@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/
2372773002
Mike Klein [Mon, 26 Sep 2016 19:49:04 +0000 (15:49 -0400)]
Move undefined-func-template to wont-fix warnings.
The warning looks to helpfully pre-warn about possible link failures, but it's warning exclusively in places where we're doing things right.
The worst that happens ignoring this warning is a missing-symbol linker error.
I've taken the opportunity to batch in a few other de-escalations:
- Wconditional-uninitialized is done better by MSAN
- It'll take some work to dig Wformat-literal out of our shader compiler, but nothing looks unsafe
- Most of Wshift-sign-overflow is 0xff << 24. Don't want to ban that.
- Wdeprecated is mostly warning about features C++11 technically deprecated that might be removed in later releases. Punt!
- Wcovered-switch-default is pretty much the opposite of what we want.
- Wshadow is triggering too often to fix quickly. Probably mostly false positives.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2620
Change-Id: I20a85a77d2e19ed05a536b23037bd988350f821e
Reviewed-on: https://skia-review.googlesource.com/2620
Reviewed-by: Ben Wagner <bungeman@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Mike Reed [Mon, 26 Sep 2016 19:33:00 +0000 (15:33 -0400)]
replace Arithmetic xfermode with imagefilter
chrome pre-cl: https://codereview.chromium.org/
2369023002/
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2595
Change-Id: Ia3f3b721854c516f3b7f8c44f71f40a8a2eeb9b4
Reviewed-on: https://skia-review.googlesource.com/2595
Commit-Queue: Mike Reed <reed@google.com>
Reviewed-by: Florin Malita <fmalita@chromium.org>
Reviewed-by: Robert Phillips <robertphillips@google.com>
liyuqian [Mon, 26 Sep 2016 19:49:06 +0000 (12:49 -0700)]
Fix typo
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2370003002
NOTRY=true
DOCS_PREVIEW= https://skia.org/?cl=
2370003002
Review-Url: https://codereview.chromium.org/
2370003002
egdaniel [Mon, 26 Sep 2016 19:30:46 +0000 (12:30 -0700)]
Revert of Fix bufferIndex check in VulkanWindowContext (patchset #1 id:1 of https://codereview.chromium.org/
2363353003/ )
Reason for revert:
Fix not correct.
Original issue's description:
> Fix bufferIndex check in VulkanWindowContext
>
> BUG=skia:
> GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2363353003
>
> Committed: https://skia.googlesource.com/skia/+/
e6bd20e62b827a1f26fa491fc628f015d208350f
TBR=jvanverth@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/
2370953002
egdaniel [Mon, 26 Sep 2016 19:20:56 +0000 (12:20 -0700)]
Fix bufferIndex check in VulkanWindowContext
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2363353003
Review-Url: https://codereview.chromium.org/
2363353003
ethannicholas [Mon, 26 Sep 2016 18:58:52 +0000 (11:58 -0700)]
Turned on SkSL->GLSL compiler
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2288033003
Committed: https://skia.googlesource.com/skia/+/
9b0fe3d125f237d9884732a48414fa85fc71b4e3
Review-Url: https://codereview.chromium.org/
2288033003
Robert Phillips [Mon, 26 Sep 2016 18:07:48 +0000 (14:07 -0400)]
Minor clean up of GrAAConvexTessellator
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2604
Change-Id: Ie94361256675cded9ca241ddad6443e695a596e9
Reviewed-on: https://skia-review.googlesource.com/2604
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
borenet [Mon, 26 Sep 2016 18:13:11 +0000 (11:13 -0700)]
Roll recipes
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2366323002
Review-Url: https://codereview.chromium.org/
2366323002
caryclark [Mon, 26 Sep 2016 18:03:54 +0000 (11:03 -0700)]
allow conic chop to fail
Fuzzy values may cause the conic chop to fail.
Check to see if the values are all finite, and
require the caller to do the same.
R=reed@google.com
BUG=650178
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2368993002
Review-Url: https://codereview.chromium.org/
2368993002
Mike Klein [Mon, 26 Sep 2016 17:19:12 +0000 (13:19 -0400)]
Turn on -Wundefined-reinterpret-cast.
It was not a fan of this (blatant) aliasing.
I suspect this best_non_simd_srcover_srgb_srgb() function has several
other aliasing issues that use undefined behavior, but this is all it's
complaining about for now.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2606
Change-Id: I25a8800e810bccf5068c8a10e9c8c8f565e57304
Reviewed-on: https://skia-review.googlesource.com/2606
Commit-Queue: Mike Klein <mtklein@chromium.org>
Commit-Queue: Herb Derby <herb@google.com>
Reviewed-by: Herb Derby <herb@google.com>
brianosman [Mon, 26 Sep 2016 17:43:21 +0000 (10:43 -0700)]
Tag checkerboard bitmaps as sRGB
Significantly reduces the diff between legacy and sRGB/F16 on about 25
GMs. This is just the biggest piece of low-hanging fruit. Many GMs create
N32 raster surfaces to procedurally generate source textures, and I'd like
to fix all of them. It's much easier to reason about the GMs (is sRGB
doing the right thing) when everything is tagged like this - the only
expected differences are due to filtering and blending.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2368933003
Review-Url: https://codereview.chromium.org/
2368933003
Mike Klein [Mon, 26 Sep 2016 17:02:35 +0000 (13:02 -0400)]
-Wcomma may be good to go now.
This cleans up 3 remaining sites using , that probably meant ;
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2605
Change-Id: I5e48bcd85d72a205d2b0c860461dab1ec793dd18
Reviewed-on: https://skia-review.googlesource.com/2605
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
mtklein [Mon, 26 Sep 2016 17:31:12 +0000 (10:31 -0700)]
My take on SkAlign changes.
Like the other change, it makes SkAlignN(x) macros work for pointers, and makes the macros themselves just syntax sugar for SkAlign<N>(x). We can still decide if we want to sed away the macros independently.
This just does it in a somewhat less repetitive way, and adds some tests.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2368293002
No public API changes.
TBR=reed@google.com
Review-Url: https://codereview.chromium.org/
2368293002
halcanary [Mon, 26 Sep 2016 17:30:39 +0000 (10:30 -0700)]
bin/coverage: a GN version of the coverage script
intended to replace tools/coverage when GYP is gone.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2343243002
Review-Url: https://codereview.chromium.org/
2343243002
borenet [Mon, 26 Sep 2016 17:25:32 +0000 (10:25 -0700)]
Include timestamp in nanobench JSON file name
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2366623003
TBR=jcgregorio
Review-Url: https://codereview.chromium.org/
2366623003
caryclark [Mon, 26 Sep 2016 17:25:26 +0000 (10:25 -0700)]
add tiger tests
Add the tiger test suite.
In extended mode (-x) this
adds 700K new pathop tests.
Normally, it adds about 3500
tests.
TBR=reed@google.com
BUG=skia:5131
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2373533002
Review-Url: https://codereview.chromium.org/
2373533002
mtklein [Mon, 26 Sep 2016 15:44:47 +0000 (08:44 -0700)]
Typo?
This can't be what you intended...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2368273002
NOTREECHECKS=true
Review-Url: https://codereview.chromium.org/
2368273002
mtklein [Mon, 26 Sep 2016 15:40:12 +0000 (08:40 -0700)]
GN: detect is_clang, use it to switch Clang to warning blacklist.
Same as the last CL, now with -Wno-over-aligned.
CQ_INCLUDE_TRYBOTS=master.client.skia.compile:Build-Ubuntu-Clang-x86-Debug-GN_Android-Trybot
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2369033002
TBR=bungeman@google.com
NOTREECHECKS=true
Review-Url: https://codereview.chromium.org/
2369033002
mtklein [Mon, 26 Sep 2016 15:39:43 +0000 (08:39 -0700)]
Quiet a -Wcomma warning.
Clang's got a warning -Wcomma that warns about possible unintentional uses of
the comma operator, usually where a semicolon looks itended.
I don't think there's a bug here, but a semicolon does work just as well
and reads a bit more conventionally. Though, I'm not actually sure how this
parses, as fp = ...; glslFP = ...; or fp = (..., glsl = ...). The second
would most definitely be a bug.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2365403002
NOTREECHECKS=true
Review-Url: https://codereview.chromium.org/
2365403002
mtklein [Mon, 26 Sep 2016 15:18:43 +0000 (08:18 -0700)]
beziers: I missed an unsequenced moveTo() pair.
TBR=brianosman@google.com
BUG=skia:
NOTREECHECKS=true
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2369933002
Review-Url: https://codereview.chromium.org/
2369933002
Robert Phillips [Mon, 26 Sep 2016 13:00:36 +0000 (09:00 -0400)]
Fix some fuzzer complaints
In one case the fuzzer was switching the picture's op code to an invalid value
In the other two the fuzzer was maxing out the number of points passed to drawPoints and the number of characters passed to drawTextRSXform. In these cases the validation would fail but still return a pointer into the data stream.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2593
Change-Id: Id6d4e6b7bcbae38ace7ad1d92ffcfa5c02f9fb61
Reviewed-on: https://skia-review.googlesource.com/2593
Reviewed-by: Mike Reed <reed@google.com>
Mike Klein [Mon, 26 Sep 2016 14:34:14 +0000 (14:34 +0000)]
Revert "GN: detect is_clang, use it to switch Clang to warning blacklist."
This reverts commit I09c3487adfeb26a6fb07e1939cb927c5d7de3107.
Reason for revert: -Wover-aligned on x86 Android bots
Original issue's description:
> GN: detect is_clang, use it to switch Clang to warning blacklist.
>
> BUG=skia:
>
> GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2599
>
> Change-Id: I09c3487adfeb26a6fb07e1939cb927c5d7de3107
> Reviewed-on: https://skia-review.googlesource.com/2599
> Reviewed-by: Ben Wagner <bungeman@google.com>
> Commit-Queue: Mike Klein <mtklein@chromium.org>
>
TBR=mtklein@chromium.org,bungeman@google.com,reviews@skia.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Change-Id: Ic07bf7896a305be6c377f14a573cb10836f1a6f8
Reviewed-on: https://skia-review.googlesource.com/2601
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>
halcanary [Mon, 26 Sep 2016 14:27:04 +0000 (07:27 -0700)]
Documentation: fix links
TBR=
NOTRY=true
Review-Url: https://codereview.chromium.org/
2366403002
Mike Klein [Mon, 26 Sep 2016 13:14:59 +0000 (09:14 -0400)]
GN: detect is_clang, use it to switch Clang to warning blacklist.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2599
Change-Id: I09c3487adfeb26a6fb07e1939cb927c5d7de3107
Reviewed-on: https://skia-review.googlesource.com/2599
Reviewed-by: Ben Wagner <bungeman@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
bsalomon [Mon, 26 Sep 2016 13:55:02 +0000 (06:55 -0700)]
Stop aggregating texture/buffer access objects in GrFragmentProcessor parents.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2349243002
Review-Url: https://codereview.chromium.org/
2365943003
Brian Osman [Fri, 23 Sep 2016 15:41:14 +0000 (11:41 -0400)]
In makeLinearGamma, return the original CS if it's already linear
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2589
Change-Id: I51d8f8f64bc5f73a0b7cbf5f01f3d23b56c1895e
Reviewed-on: https://skia-review.googlesource.com/2589
Reviewed-by: Mike Reed <reed@google.com>
Reviewed-by: Matt Sarett <msarett@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
mtklein [Mon, 26 Sep 2016 13:29:18 +0000 (06:29 -0700)]
These calls to SkRandom are not sequenced.
This is why we see two versions of this GM: some compilers evaluate the
rand.nextRangeScalar() calls left to right, others right to left.
Remember this bug?
BUG=skia:3241
(https://codereview.chromium.org/
1157943006 fixed hittestpath last year.)
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2368263002
Review-Url: https://codereview.chromium.org/
2368263002
caryclark [Mon, 26 Sep 2016 12:36:58 +0000 (05:36 -0700)]
fix tiger b
The tiger tests have uncovered numerous bugs.
This CL fixes the last of them.
If a pair of curves do not intersect, but
have one or both ends very close to the opposite
curve, consider that an intersection.
TBR=reed@google.com
BUG=skia:5131
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2356363003
Review-Url: https://codereview.chromium.org/
2356363003
cblume [Mon, 26 Sep 2016 11:22:59 +0000 (04:22 -0700)]
Dash to use SkIsAlign2
Dash checks for alignment by actively aligning.
It should really just check for alignment.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2366283002
Review-Url: https://codereview.chromium.org/
2366283002
brianosman [Sun, 25 Sep 2016 20:15:58 +0000 (13:15 -0700)]
Add SkColor4f serialization
Adjusted usage in color shader, and will also be using this
in gradients, soon.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2334123003
Review-Url: https://codereview.chromium.org/
2334123003
update-skps [Sun, 25 Sep 2016 09:15:17 +0000 (02:15 -0700)]
Update SKP version
Automatic commit by the RecreateSKPs bot.
TBR=
NO_MERGE_BUILDS
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2368053002
Review-Url: https://codereview.chromium.org/
2368053002
benjaminwagner [Sat, 24 Sep 2016 00:45:45 +0000 (17:45 -0700)]
Disable msaa on iHD530 bot.
BUG=skia:5792
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2356213005
Review-Url: https://codereview.chromium.org/
2356213005
mtklein [Fri, 23 Sep 2016 20:18:41 +0000 (13:18 -0700)]
GN: ANGLE
Angle's existing GN files only work in Chrome, so I've written a new one.
This won't work on Windows, but our GN build doesn't work on Windows anyway. So this CL is an attempt to get a ahead of that curve on ANGLE. It looks large but fairly straightforward.
Now working on Linux:
$ gn gen angle --args=skia_use_angle=true
$ ninja -C angle
$ angle/dm --config angle-gl --src gm -w dm-out
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2361983002
Review-Url: https://codereview.chromium.org/
2361983002
brianosman [Fri, 23 Sep 2016 20:04:05 +0000 (13:04 -0700)]
Change SkSpecialImage::makeSurface and makeTightSurface to take output
properties (color space), bounds, and (optional) alphaType.
We were being pretty inconsistent before. Raster was honoring all
components of the info. GPU was using the supplied color type, but
propagating the source's color space. All call sites were saying N32.
What we want to do is propagate the original device's color space, and
pick a good format from that. Rather than force all the clients to
jump through hoops constructing an SkImageInfo that meets our criteria,
just have them supply the few bits we care about, and do everything else
internally.
This also lets us always use RGBA on GPU, but N32 on raster.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2349373004
Committed: https://skia.googlesource.com/skia/+/
53c38087949252d27cde668368a3eeb59cc2eb00
Review-Url: https://codereview.chromium.org/
2349373004
Robert Phillips [Fri, 23 Sep 2016 18:23:22 +0000 (14:23 -0400)]
Add validation of RRects to SkValidatingReadBuffer
This comes from the Skia fuzzer where it is inverting the RRect's rect which causes trouble down the line.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2591
Change-Id: I5c34105a47369492d2df99d39a4e29116060ad37
Reviewed-on: https://skia-review.googlesource.com/2591
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
bsalomon [Fri, 23 Sep 2016 19:09:16 +0000 (12:09 -0700)]
Don't compute path keys for volatile paths in GrShape.
Otherwise, we will compute cache keys for internally transformed paths that don't repeat (e.g. clip paths transformed into device space with a changing view matrix).
BUG=chromium:649562
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2369513002
Review-Url: https://codereview.chromium.org/
2369513002
csmartdalton [Fri, 23 Sep 2016 18:36:11 +0000 (11:36 -0700)]
Add Pixel C knobs to skpbench
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2369533002
Review-Url: https://codereview.chromium.org/
2369533002
jvanverth [Fri, 23 Sep 2016 17:30:04 +0000 (10:30 -0700)]
Some Vulkan memory fixes and cleanup
* Switch back to not setting transfer_dst on all buffers
* Add some missing unit tests
* Add tracking of heap usage for debugging purposes
* Fall back to non-device-local memory if device-local allocation fails
BUG=skia:5031
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2356343003
Committed: https://skia.googlesource.com/skia/+/
c5850e9fdb62cc4ae5ed2b6af51aea92cac07455
Review-Url: https://codereview.chromium.org/
2356343003
Mike Reed [Fri, 23 Sep 2016 15:43:54 +0000 (11:43 -0400)]
change SkXfermodeImageFilter to carry no impl information
BUG=skia:
NOTREECHECKS=True
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2584
Change-Id: Ibf2dfa7722348ae71a99f1c895de0d2996350e58
Reviewed-on: https://skia-review.googlesource.com/2584
Commit-Queue: Mike Reed <reed@google.com>
Reviewed-by: Florin Malita <fmalita@chromium.org>
Reviewed-by: Robert Phillips <robertphillips@google.com>
caryclark [Fri, 23 Sep 2016 16:32:26 +0000 (09:32 -0700)]
fix msan bug in pathops
Msan and Valgrind found an uninitialized memory mistake in
pathops. This also fixes similar bugs where not all parts
of the geometry were covered in the loop iteration.
R=borenet@google.com
NOTREECHECKS=true
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2366893003
Review-Url: https://codereview.chromium.org/
2366893003
Mike Klein [Fri, 23 Sep 2016 15:10:08 +0000 (11:10 -0400)]
Include GLTestContext_command_buffer.cpp in G3 build.
This is now required to build our GPU testing tools.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2587
NOTREECHECKS=true
Change-Id: I219665fbe50e9bbea9abb88a6d63c34b76ca92e6
Reviewed-on: https://skia-review.googlesource.com/2587
Commit-Queue: Mike Klein <mtklein@chromium.org>
Reviewed-by: Ben Wagner <benjaminwagner@google.com>
brianosman [Fri, 23 Sep 2016 15:49:50 +0000 (08:49 -0700)]
Revert of Create special surfaces according to original device (not always in N32) (patchset #9 id:160001 of https://codereview.chromium.org/
2349373004/ )
Reason for revert:
DM crash and/or TSAN failure
Original issue's description:
> Change SkSpecialImage::makeSurface and makeTightSurface to take output
> properties (color space), bounds, and (optional) alphaType.
>
> We were being pretty inconsistent before. Raster was honoring all
> components of the info. GPU was using the supplied color type, but
> propagating the source's color space. All call sites were saying N32.
>
> What we want to do is propagate the original device's color space, and
> pick a good format from that. Rather than force all the clients to
> jump through hoops constructing an SkImageInfo that meets our criteria,
> just have them supply the few bits we care about, and do everything else
> internally.
>
> This also lets us always use RGBA on GPU, but N32 on raster.
>
> BUG=skia:
> GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2349373004
>
> Committed: https://skia.googlesource.com/skia/+/
53c38087949252d27cde668368a3eeb59cc2eb00
TBR=robertphillips@google.com,reed@google.com,bsalomon@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/
2366723004
Mike Reed [Fri, 23 Sep 2016 14:05:05 +0000 (10:05 -0400)]
use SkCanvas::ClipOp
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2583
Change-Id: I651e2c278bb4d2df08b6e134c734c5403c7e7ff7
Reviewed-on: https://skia-review.googlesource.com/2583
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Mike Reed <reed@google.com>
bungeman [Fri, 23 Sep 2016 15:16:04 +0000 (08:16 -0700)]
Json ide script for gn to output cmake.
Can be run like
gn gen out/debug --ide=json --json-ide-script=../../gn/gn_to_cmake.py
or
gn gen out/debug --ide=json
python gn/gn_to_cmake.py out/debug/project.json
This first pass creates CMakeLists.txt with just enough information for an
ide. Each target is just a static library with sources, include
directories, and defines.
NOTRY=true
This isn't automatically tested.
Review-Url: https://codereview.chromium.org/
2363873002
brianosman [Fri, 23 Sep 2016 15:11:55 +0000 (08:11 -0700)]
Change SkSpecialImage::makeSurface and makeTightSurface to take output
properties (color space), bounds, and (optional) alphaType.
We were being pretty inconsistent before. Raster was honoring all
components of the info. GPU was using the supplied color type, but
propagating the source's color space. All call sites were saying N32.
What we want to do is propagate the original device's color space, and
pick a good format from that. Rather than force all the clients to
jump through hoops constructing an SkImageInfo that meets our criteria,
just have them supply the few bits we care about, and do everything else
internally.
This also lets us always use RGBA on GPU, but N32 on raster.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2349373004
Review-Url: https://codereview.chromium.org/
2349373004
brianosman [Fri, 23 Sep 2016 14:11:34 +0000 (07:11 -0700)]
Make GrRenderableConfigForColorSpace argument const
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2367813002
Review-Url: https://codereview.chromium.org/
2367813002
borenet [Fri, 23 Sep 2016 13:37:57 +0000 (06:37 -0700)]
Add swarming task for nanobench upload
BUG=skia:5719
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2364693003
Review-Url: https://codereview.chromium.org/
2364693003
caryclark [Fri, 23 Sep 2016 12:47:20 +0000 (05:47 -0700)]
split tight quads and conics
Tight quads and conics may nearly fold over on themselves, confusing
coincidence against other curves. Split them at their max curvature
early on to avoid complicating later logic.
TBR=reed@google.com
BUG=skia:5131
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2357353002
Review-Url: https://codereview.chromium.org/
2357353002
brianosman [Thu, 22 Sep 2016 21:50:58 +0000 (14:50 -0700)]
sRGB cleanup in image filters. Ensure we configure paints correctly.
Net effect of the two calls is (basically) the same, but given that we're
propagating from isGammaCorrect on the DC, I figured it makes sense to use
the same-named API.
More importantly, a couple places had slipped through, so those draws would
ignore sRGB-ness of inputs entirely.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2357413004
Review-Url: https://codereview.chromium.org/
2357413004
brianosman [Thu, 22 Sep 2016 21:27:34 +0000 (14:27 -0700)]
It is possible to try and draw to an unsupported format. Let this go.
Squelches assert firing in gradient code on low-end Android bots.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2365703002
Review-Url: https://codereview.chromium.org/
2365703002
halcanary [Thu, 22 Sep 2016 21:13:16 +0000 (14:13 -0700)]
SkPDF: do not assert when openStream return nullptr
BUG=163892
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2332813002
Review-Url: https://codereview.chromium.org/
2332813002
reed [Thu, 22 Sep 2016 21:12:46 +0000 (14:12 -0700)]
document.close from bool to void
Nothing meaningful is returned, it is inconsistent with the bulk of the rest of skia's api. The C api is waiting on this change as well.
Required chrome CL: https://codereview.chromium.org/
2355343003/
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2354403002
Review-Url: https://codereview.chromium.org/
2354403002
jvanverth [Thu, 22 Sep 2016 20:45:24 +0000 (13:45 -0700)]
Revert of Some Vulkan memory fixes and cleanup (patchset #2 id:20001 of https://codereview.chromium.org/
2356343003/ )
Reason for revert:
fHeapIndex is not used in release, need to hide behind SK_DEBUG. Failing on Perf-Android-Clang-NVIDIA_Shield-GPU-TegraX1-arm64-Release-GN_Android_Vulkan.
Original issue's description:
> Some Vulkan memory fixes and cleanup
>
> * Switch back to not setting transfer_dst on all buffers
> * Add some missing unit tests
> * Add tracking of heap usage for debugging purposes
> * Fall back to non-device-local memory if device-local allocation fails
>
> BUG=skia:5031
> GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2356343003
>
> Committed: https://skia.googlesource.com/skia/+/
c5850e9fdb62cc4ae5ed2b6af51aea92cac07455
TBR=egdaniel@google.com,brianosman@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:5031
Review-Url: https://codereview.chromium.org/
2358123004
msarett [Thu, 22 Sep 2016 20:23:56 +0000 (13:23 -0700)]
G instead of R, thanks Gold
NOTRY=true
TBR=mtklein@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2362863002
Review-Url: https://codereview.chromium.org/
2362863002
caryclark [Thu, 22 Sep 2016 19:52:20 +0000 (12:52 -0700)]
don't close a contour with a line if nothing's there
R=kjlubick@google.com
BUG=skia:5789
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2360083006
Review-Url: https://codereview.chromium.org/
2360083006
jvanverth [Thu, 22 Sep 2016 19:47:35 +0000 (12:47 -0700)]
Some Vulkan memory fixes and cleanup
* Switch back to not setting transfer_dst on all buffers
* Add some missing unit tests
* Add tracking of heap usage for debugging purposes
* Fall back to non-device-local memory if device-local allocation fails
BUG=skia:5031
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2356343003
Review-Url: https://codereview.chromium.org/
2356343003
hcm [Thu, 22 Sep 2016 19:47:14 +0000 (12:47 -0700)]
Update Chrome section to include Skia roll/sheriff/bug info
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2351593003
NOTRY=true
DOCS_PREVIEW= https://skia.org/?cl=
2351593003
Review-Url: https://codereview.chromium.org/
2351593003
robertphillips [Thu, 22 Sep 2016 19:42:30 +0000 (12:42 -0700)]
Make AALinearizingConvexPathRenderer able to handle stroke and fill
This is intended to catch stroke-and-fill convex paths with potentially small stroke widths (e.g., .1).
It does have the disconcerting side effect of changing bevel-joined stroke-and-filled degenerate-single-line-rects into plain rects (w/o triangular end-caps).
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2301353004
Committed: https://skia.googlesource.com/skia/+/
522bcd99fa65a8abd130880f59b500cf367d0845
Review-Url: https://codereview.chromium.org/
2301353004
bsalomon [Thu, 22 Sep 2016 19:42:11 +0000 (12:42 -0700)]
Change implementation of flush-count based GrGpuResource purging
Change default to approx 30seconds (given some API usage assumptions)
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2361093002
Review-Url: https://codereview.chromium.org/
2361093002
csmartdalton [Thu, 22 Sep 2016 19:37:21 +0000 (12:37 -0700)]
skpbench fixes for linux
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2354273004
Review-Url: https://codereview.chromium.org/
2354273004
msarett [Thu, 22 Sep 2016 19:37:04 +0000 (12:37 -0700)]
Add BGRA as input format to SkColorSpaceXform
This is immediately useful for webp and I think it's a fair guess
that BGRA src formats are not uncommon.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2353363008
Review-Url: https://codereview.chromium.org/
2353363008
bungeman [Thu, 22 Sep 2016 19:32:07 +0000 (12:32 -0700)]
Remove assert that current color is never used.
The COLR table format allows for use of the 'current color'.
Currently the code asserts if this is used, but will use a nearby
luminance grey to this color instead if the assert is ignored.
Remove the assert now that this path is known to be used.
BUG=skia:5788
Review-Url: https://codereview.chromium.org/
2359923004
brianosman [Thu, 22 Sep 2016 19:31:58 +0000 (12:31 -0700)]
Support for color-spaces with multi-stop (texture) gradients
Texture is F16 linear, unless that's not supported. In that
case, we pack down to sRGB.
Added more test patches to the gamut GM with many stops,
to test this case. Now they render correctly.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2343253002
Review-Url: https://codereview.chromium.org/
2343253002
mtklein [Thu, 22 Sep 2016 18:51:24 +0000 (11:51 -0700)]
GN: build skiaserve
I trimmed the libmicrohttpd sources and defines down to the minimum needed to build and run. This builds and runs on Linux and Android for me.
Request.h was missing an include for SkTypes.h, which supplies the default for SK_GPU_SUPPORTED if not otherwise defined.
To build on Android, exit() -> _exit().
build.py was unused.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2367513002
NOTREECHECKS=true
Review-Url: https://codereview.chromium.org/
2367513002
caryclark [Thu, 22 Sep 2016 17:24:59 +0000 (10:24 -0700)]
Reland of ix for conic fuzz (patchset #1 id:1 of https://codereview.chromium.org/
2361473004/ )
Reason for revert:
Landed suppression in Chrome's LayoutTests/TestExpectations
Original issue's description:
> Revert of fix for conic fuzz (patchset #3 id:40001 of https://codereview.chromium.org/
2350263003/ )
>
> Reason for revert:
> See if this fixes the layout tests.
>
> Original issue's description:
> > fix for conic fuzz
> >
> > A fuzzer generates a conic that hangs when drawn.
> > The quads that approximate the conics move up and down
> > in y, confusing the renderer.
> >
> > This fix ensures that the split conic maintains the
> > same y direction as the original conic.
> >
> > R=reed@google.com
> > BUG=647922
> > GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2350263003
> >
> > Committed: https://skia.googlesource.com/skia/+/
ac78863acdef4b428aaf66985b80c76d1be0fdea
>
> TBR=reed@google.com
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=647922
>
> Committed: https://skia.googlesource.com/skia/+/
08b345588414b861af8a55950e7dc21a1bd85a28
TBR=reed@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=647922
Review-Url: https://codereview.chromium.org/
2359253002
caryclark [Thu, 22 Sep 2016 14:42:39 +0000 (07:42 -0700)]
Revert of fix for conic fuzz (patchset #3 id:40001 of https://codereview.chromium.org/
2350263003/ )
Reason for revert:
See if this fixes the layout tests.
Original issue's description:
> fix for conic fuzz
>
> A fuzzer generates a conic that hangs when drawn.
> The quads that approximate the conics move up and down
> in y, confusing the renderer.
>
> This fix ensures that the split conic maintains the
> same y direction as the original conic.
>
> R=reed@google.com
> BUG=647922
> GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2350263003
>
> Committed: https://skia.googlesource.com/skia/+/
ac78863acdef4b428aaf66985b80c76d1be0fdea
TBR=reed@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=647922
Review-Url: https://codereview.chromium.org/
2361473004
brianosman [Thu, 22 Sep 2016 14:15:37 +0000 (07:15 -0700)]
Add output format properties to SkImageFilter::Context
For now, this is just the color space (of the original
requesting device). This is used when constructing
intermediate rendering surfaces, so that we ensure we
land in a surface that's similar/compatible to the
final consumer of the DAG's output.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2357273002
Review-Url: https://codereview.chromium.org/
2357273002
msarett [Thu, 22 Sep 2016 14:02:24 +0000 (07:02 -0700)]
Make SkColorSpaceXform::New() take bare ptrs
There's no need to take sk_sp if we're not going to
ref the ptr.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2360863003
Review-Url: https://codereview.chromium.org/
2360863003
mtklein [Thu, 22 Sep 2016 13:50:24 +0000 (06:50 -0700)]
Run commandbuffer config on CommandBuffer Perf bots.
Uh, so hey, I just noticed that while the CommandBuffer Test bot runs
with --config commandbuffer, the Perf bot just runs as a vanilla GPU bot.
Shouldn't we pass --config commandbuffer?
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2364483002
CQ_INCLUDE_TRYBOTS=master.client.skia:Perf-Mac-Clang-MacMini6.2-GPU-HD4000-x86_64-Debug-CommandBuffer-Trybot
Review-Url: https://codereview.chromium.org/
2364483002
mtklein [Thu, 22 Sep 2016 13:01:17 +0000 (06:01 -0700)]
Clean up dead code.
SkOSEnvironment was used to set --enable-gl-path-rendering, and is now unused.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2365483002
Review-Url: https://codereview.chromium.org/
2365483002
cblume [Thu, 22 Sep 2016 12:25:26 +0000 (05:25 -0700)]
Is char* or uintptr_t easier to read?
Using a char* instead of uintptr_t allows us to use fewer
reinterpret_casts which may make the code easier to read.
BUG=648512
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2356703002
Review-Url: https://codereview.chromium.org/
2356703002
benjaminwagner [Thu, 22 Sep 2016 12:25:12 +0000 (05:25 -0700)]
Add recipe support for Intel HD Graphics 530.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2358183002
Review-Url: https://codereview.chromium.org/
2358183002
msarett [Thu, 22 Sep 2016 12:20:21 +0000 (05:20 -0700)]
Do not quickReject until virtual drawPatch
TBR=reed@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2358873003
Review-Url: https://codereview.chromium.org/
2358873003
caryclark [Thu, 22 Sep 2016 12:20:07 +0000 (05:20 -0700)]
fix next kevin fuzz
Fix one more fuzzer crash.
R=kjlubick@google.com
BUG=skia:5775
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2357373002
Review-Url: https://codereview.chromium.org/
2357373002
caryclark [Thu, 22 Sep 2016 12:15:14 +0000 (05:15 -0700)]
fix for conic fuzz
A fuzzer generates a conic that hangs when drawn.
The quads that approximate the conics move up and down
in y, confusing the renderer.
This fix ensures that the split conic maintains the
same y direction as the original conic.
R=reed@google.com
BUG=647922
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2350263003
Review-Url: https://codereview.chromium.org/
2350263003
caryclark [Thu, 22 Sep 2016 12:15:04 +0000 (05:15 -0700)]
speed up debug dm
SkPathMeasure can take minutes with pathological cases.
Limit the debug check to a reasonable number.
R=reed@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2356343004
Review-Url: https://codereview.chromium.org/
2356343004
csmartdalton [Thu, 22 Sep 2016 12:10:02 +0000 (05:10 -0700)]
Add hardware monitoring to skpbench
Adds a Hardware class with hooks for entering and exiting
"benchmarking" mode (e.g. locking clocks, etc.) as well as periodic
polling of hardware to verify the environment is stable.
Adds a partial implementation for generic Android hardware, but
ultimately we will need to write specific classes tailored to each
unique platform we need to test.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=
2360473002
Review-Url: https://codereview.chromium.org/
2360473002