egdaniel
4bcd62e331
Add some copy support for vulkan msaa
...
This allows us to do copies from:
msaa->msaa with same sample count
msaa->no-msaa with a resolve
Still missing support for no-msaa to msaa which will require a copyAsDraw
which is currently stalled and fixing possible driver bugs.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2294533002
Review-Url: https://codereview.chromium.org/2294533002
2016-08-31 07:37:31 -07:00
mtklein
cd01b03df8
GN: add sources_when_disabled to optional
...
Also make sources_when_disabled and public_defines optional arguments.
These are the two mechanisms code uses to turn itself off...
probably at most one is ever used per optional.
Update fiddle to not crash when there's no PDF backend.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2292343003
Review-Url: https://codereview.chromium.org/2292343003
2016-08-31 04:58:19 -07:00
halcanary
4f0a23a8d5
SkTextBlob: Begin implementing Extended TextBlob API
...
BUG=skia:5434
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2084533004
Review-Url: https://codereview.chromium.org/2084533004
2016-08-30 11:58:33 -07:00
rmistry
fd7f867a05
Increase run_benchmarks timeout because the two Chromium page sets can take a while
...
BUG=skia:5700
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2298453002
# Trybots do not exercise this code.
NOTRY=true
TBR=borenet
Review-Url: https://codereview.chromium.org/2298453002
2016-08-30 11:50:23 -07:00
brianosman
0e22eb8e6e
Some tests around surface creation and snapshotting with color space
...
Verify the rules that we're converging on for surfaces:
- For 8888, we only support sRGB-like gamma, or no color space at all.
- For F16, we require a color space, with linear gamma.
- For all other formats, we do not support color spaces.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2270823002
Review-Url: https://codereview.chromium.org/2270823002
2016-08-30 07:07:59 -07:00
msarett
0ac1bece48
Add width, height, unique ID, alpha to debugger DrawImageCommand
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2290683002
Review-Url: https://codereview.chromium.org/2290683002
2016-08-29 09:15:33 -07:00
brianosman
e3031b607c
Update ANGLE to latest as of August 25, 2016
...
With this version, it's now exposing GL_EXT_color_buffer_half_float, but
it's (incorrectly) using GL_HALF_FLOAT rather than GL_HALF_FLOAT_OES.
In addition, release builds were crashing in EGLImageTest due to an
apparent disagreement about calling convention on function-pointer to
exported symbol. There is a proper typedef for that in one of their
headers, but I can't seem to include it without creating more problems,
so I just fixed the decoration on our local typedef.
With those changes, all tests pass, and (on my Windows machine), all
GMs and SKPs produce identical results vs. master in angle and angle-gl.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2282733003
Review-Url: https://codereview.chromium.org/2282733003
2016-08-26 10:37:57 -07:00
liyuqian
1c9f55b468
Do not spam the debugging information
...
Currently, whenever there's a frame being rendered, we log such an
invalidation event. This is very spammy for animations or frame rates
test.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2276303003
Review-Url: https://codereview.chromium.org/2276303003
2016-08-25 13:14:54 -07:00
halcanary
f412f09884
SkDrawCommand: hinting
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2274373003
Review-Url: https://codereview.chromium.org/2274373003
2016-08-25 11:10:41 -07:00
mtklein
f419781d37
update Android auto-detection.
...
Clang defines __ANDROID__
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2278483004
No public API changes.
TBR=reed@google.com
Review-Url: https://codereview.chromium.org/2278483004
2016-08-25 08:44:49 -07:00
vjiaoblack
e6f5d56231
Made shadows blurry (thru implementing variance mapping)
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2224163005
Review-Url: https://codereview.chromium.org/2224163005
2016-08-25 06:30:23 -07:00
liyuqian
23d24587ee
Fix touch event parameter order
...
The issue is so obvious... However, all newer Android devices (e.g.,
Nexus 6P, 5X, 9) are immune to this bug. So I didn't notice this until I
tested Viewer app on Nexus 7 yesterday...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2254223002
Review-Url: https://codereview.chromium.org/2254223002
2016-08-22 11:31:09 -07:00
borenet
a629166d6e
Build command buffer with is_component_build=false
...
BUG=skia:5630
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2265913002
Review-Url: https://codereview.chromium.org/2265913002
2016-08-22 07:31:43 -07:00
bsalomon
ac3aa245ac
Plumb drawArc to SkDevice.
...
Plumbs the drawArc canvas method down to SkDevice without converting to a path. Plumbs through the various recording canvas classes.
BUG=skia:5227
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2257023003
Review-Url: https://codereview.chromium.org/2257023003
2016-08-19 11:25:20 -07:00
egdaniel
bf63e616a6
Add mssa configs for vulkan, and simple bug fix
...
BUG=skia:5127
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2251473002
Review-Url: https://codereview.chromium.org/2251473002
2016-08-17 06:26:16 -07:00
bsalomon
300178b1c8
Only open X Display once for all test GLX contexts
...
Saves about 4 seconds of runtime for 'dm --src gm --config gpu' on my system.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2240673002
Review-Url: https://codereview.chromium.org/2240673002
2016-08-11 14:58:34 -07:00
bsalomon
90b5cc31f3
Improvements to build_command_buffer.py
...
Allows a shortcut, -t, for --chrome-build-type and clarifies meaning of the value in help.
Removes --fetch. This used to work with gyp builds but since it doesn't run gn it currently fails to build after fetching the src. Also, it wasn't being used.
Adds --no-hooks. Like --no-sync it but also doesn't run gclient runhooks.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2236953002
Review-Url: https://codereview.chromium.org/2236953002
2016-08-11 08:25:41 -07:00
egdaniel
f5fe4b5567
Fix crash in viewer on certain android devices
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2239493002
Review-Url: https://codereview.chromium.org/2239493002
2016-08-11 08:15:12 -07:00
jvanverth
8c67ebe4c4
Clean up Xlib viewer
...
* Make event handling closer to GLFW's
* Fix Vulkan refreshing
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2226413002
Review-Url: https://codereview.chromium.org/2226413002
2016-08-11 07:03:04 -07:00
rmistry
d71fe83ac2
Update VERSION of "svg" asset.
...
Removing the SVG due to the reason in https://docs.google.com/document/d/1kYRvUxZTnm1tI_0bTU0BX9jqSSTqPUhGXJVcD3Rcg2c/edit?disco=AAAAAqQSPSc
BUG=skia:5628
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2232003002
Review-Url: https://codereview.chromium.org/2232003002
2016-08-10 16:36:15 -07:00
bsalomon
80e38ac117
Add more error printing to know why command buffer lib failed to load.
...
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2233073002
Review-Url: https://codereview.chromium.org/2233073002
2016-08-10 12:26:00 -07:00
csmartdalton
f9635999a4
Add flag for window rectangles to GrRenderTarget
...
Adds a flag to GrRenderTarget that indicates whether it can be used
with window rectangles. Also attempts to clean up some of the mixed
samples API.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2225303002
Review-Url: https://codereview.chromium.org/2225303002
2016-08-10 11:09:07 -07:00
rmistry
d8a620b173
SVG tool that downloads SVGs from a txt file into a specified dir
...
BUG=skia:5628
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2234823002
Review-Url: https://codereview.chromium.org/2234823002
2016-08-10 07:00:43 -07:00
jvanverth
a489e3f1fb
Add RasterWindowContext_mac
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2228883003
Review-Url: https://codereview.chromium.org/2228883003
2016-08-09 10:53:11 -07:00
martina.kollarova
fc3ea41ceb
Check more GLES versions when creating context
...
Unifies the context creation for GL and GLES into a single loop.
BUG=skia:5403
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2201033003
Review-Url: https://codereview.chromium.org/2201033003
2016-08-09 01:41:55 -07:00
mtklein
4e97607d9a
Use sse4.2 CRC32 instructions to hash when available.
...
About 9x faster than Murmur3 for long inputs.
Most of this is a mechanical change from SkChecksum::Murmur3(...) to SkOpts::hash(...).
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2208903002
CQ_INCLUDE_TRYBOTS=master.client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot;master.client.skia.compile:Build-Ubuntu-GCC-x86_64-Release-CMake-Trybot,Build-Mac-Clang-x86_64-Release-CMake-Trybot
Review-Url: https://codereview.chromium.org/2208903002
2016-08-08 09:06:28 -07:00
borenet
1413d52c53
Speculative fix for SKP partner-dir upload
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2229433002
Review-Url: https://codereview.chromium.org/2229433002
2016-08-08 06:26:00 -07:00
robertphillips
401c1968a1
Add new GM for occluded blurmask filter draws
...
This splits out just the GM portion of https://codereview.chromium.org/2201993003/ (Add GM to test out blurmaskfilter occluders) which included some API changes
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2214033002
Review-Url: https://codereview.chromium.org/2214033002
2016-08-04 12:35:46 -07:00
jvanverth
1ba1d372c2
Get Mac viewer working with SDL
...
Also fixes SkiaSDLExample.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2210603003
Review-Url: https://codereview.chromium.org/2210603003
2016-08-04 12:30:31 -07:00
borenet
538d5b68e8
Move builder_spec, [dm|nanobench]_flags, builder_name_schema to recipes
...
- builder_name_schema becomes its own recipe module.
- builder_spec, dm, and nanobench flags move into vars module.
- recipe expectation diffs include:
- no more buildbot_spec.py step
- "real" dm and nanobench flags, instead of --dummy-flags
- some inconsequential stuff in visualbench, which is removed anyway.
BUG=skia:5578
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2215803002
Review-Url: https://codereview.chromium.org/2215803002
2016-08-04 11:07:10 -07:00
robertphillips
276d3286b3
Add new bench for occluded blurmaskfilter draws
...
w/ occluders
44/44 MB 6 497us 500us 500us 502us 0% .oOOooooOO gpu bluroccludedrrect
w/o occluders
41/41 MB 5 1.08ms 1.09ms 1.12ms 1.47ms 11% .........O gpu bluroccludedrrect
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2203153002
Review-Url: https://codereview.chromium.org/2203153002
2016-08-04 09:03:20 -07:00
bungeman
ffae30db4a
Convert SkAutoTUnref<SkData> to sk_sp<SkData>.
...
With the move from SkData::NewXXX to SkData::MakeXXX most
SkAutoTUnref<SkData> were changed to sk_sp<SkData>. However,
there are still a few SkAutoTUnref<SkData> around, so clean
them up.
Review-Url: https://codereview.chromium.org/2212493002
2016-08-03 13:32:32 -07:00
halcanary
13cba49982
SkShaper: optionally disable harfbuzz
...
also, re-enable warnings.
motivation: used by me for PDF testing.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2201153002
Review-Url: https://codereview.chromium.org/2201153002
2016-08-03 10:43:55 -07:00
borenet
9e3d3d8b36
Fix fake_specs.py, update recipe expectation
...
BUG=skia:5578
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2200763002
Review-Url: https://codereview.chromium.org/2200763002
2016-08-03 10:10:07 -07:00
bungeman
38d909ec28
Move off SK_SUPPORT_LEGACY_DATA_FACTORIES.
...
This moves Skia code off of SK_SUPPORT_LEGACY_DATA_FACTORIES.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2206633004
Review-Url: https://codereview.chromium.org/2206633004
2016-08-02 14:40:46 -07:00
mtklein
5dbd274846
gn: fix fiddle
...
- __SK_FORCE_IMAGE_DECODER_LINKING makes sure the PNG encoder is linked
- updated path to make working with sample draw.cpp easier.
No public API changes.
TBR=reed@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2202353002
Review-Url: https://codereview.chromium.org/2202353002
2016-08-02 11:13:49 -07:00
hstern
0b401ce663
Add code to lua paths to get the fill path and get lists of verbs and
...
points
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2199823002
Review-Url: https://codereview.chromium.org/2199823002
2016-08-02 09:17:59 -07:00
borenet
86baf3d8de
Fix build_command_buffer for Windows
...
BUG=skia:5597
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2207563002
Review-Url: https://codereview.chromium.org/2207563002
2016-08-02 08:37:50 -07:00
borenet
b6aafe66a0
Fix Chrome build for RecreateSKPs and CommandBuffer bots for GYP->GN
...
BUG=skia:5597
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2199273002
Review-Url: https://codereview.chromium.org/2199273002
2016-08-02 07:02:52 -07:00
reed
320a40d773
Always return ImageShader, even from SkShader::MakeBitmapShader
...
Lessons learned
1. ImageShader (correctly) always compresses (typically via PNG) during serialization. This has the surprise results of
- if the image was marked opaque, but has some non-opaque pixels (i.e. bug in blitter or caller), then compressing may "fix" those pixels, making the deserialized version draw differently. bug filed.
- 565 compressess/decompresses to 8888 (at least on Mac), which draws differently (esp. under some filters). bug filed.
2. BitmapShader did not enforce a copy for mutable bitmaps, but ImageShader does (since it creates an Image). Thus the former would see subsequent changes to the pixels after shader creation, while the latter does not, hence the change to the BlitRow test to avoid this modify-after-create pattern. I sure hope this prev. behavior was a bug/undefined-behavior, since this CL changes that.
BUG=skia:5595
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2195893002
Review-Url: https://codereview.chromium.org/2195893002
2016-08-02 06:12:06 -07:00
borenet
f5b675efbe
Stop running nanobench on Test.*Debug bots
...
We'll need to add Perf.*Debug bots to achieve the same coverage.
BUG=skia:4768
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2189393002
NOTREECHECKS=true
Review-Url: https://codereview.chromium.org/2189393002
2016-08-01 05:08:52 -07:00
csmartdalton
e0d362929d
Add test configs for instanced rendering
...
Adds the following configs and enables them on select bots:
glinst, glinst4, glinstdit4, glinst16, glinstdit16,
esinst, esinst4, esinstdit4
Makes general changes to GrContextOptions, GrCaps, etc. to facilitate
this.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2182783004
Review-Url: https://codereview.chromium.org/2182783004
2016-07-29 08:14:20 -07:00
msarett
50ce1f28ff
Add color space xform support to SkJpegCodec (includes F16!)
...
Also changes SkColorXform to support:
RGBA->RGBA
RGBA->BGRA
Instead of:
RGBA->SkPMColor
TBR=reed@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2174493002
CQ_INCLUDE_TRYBOTS=master.client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot
Committed: https://skia.googlesource.com/skia/+/73d55332e2846dd05e9efdaa2f017bcc3872884b
Review-Url: https://codereview.chromium.org/2174493002
2016-07-29 06:23:33 -07:00
mtklein
4e41364b1e
Add another GM to skip in serialize mode.
...
See https://luci-milo.appspot.com/swarming/task/304ced7bada3b310/steps/dm/0/stdout
Also update .json.
CQ_INCLUDE_TRYBOTS=master.client.skia:Test-Win-MSVC-GCE-CPU-AVX2-x86_64-Debug-Trybot
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2196543002
Review-Url: https://codereview.chromium.org/2196543002
2016-07-28 19:14:08 -07:00
msarett
39979d8c6b
Revert of Add color space xform support to SkJpegCodec (includes F16!) (patchset #9 id:260001 of https://codereview.chromium.org/2174493002/ )
...
Reason for revert:
Breaking MSAN
Original issue's description:
> Add color space xform support to SkJpegCodec (includes F16!)
>
> Also changes SkColorXform to support:
> RGBA->RGBA
> RGBA->BGRA
>
> Instead of:
> RGBA->SkPMColor
>
> TBR=reed@google.com
> BUG=skia:
> GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2174493002
> CQ_INCLUDE_TRYBOTS=master.client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot
>
> Committed: https://skia.googlesource.com/skia/+/73d55332e2846dd05e9efdaa2f017bcc3872884b
TBR=mtklein@google.com ,reed@google.com,herb@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:
Review-Url: https://codereview.chromium.org/2195523002
2016-07-28 17:11:18 -07:00
msarett
73d55332e2
Add color space xform support to SkJpegCodec (includes F16!)
...
Also changes SkColorXform to support:
RGBA->RGBA
RGBA->BGRA
Instead of:
RGBA->SkPMColor
TBR=reed@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2174493002
CQ_INCLUDE_TRYBOTS=master.client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot
Review-Url: https://codereview.chromium.org/2174493002
2016-07-28 15:06:16 -07:00
mtklein
20c27d6d6f
Make coverage.py optional for dm and nanobench _flags.py
...
It is better to test partially than not test at all.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2184423002
Review-Url: https://codereview.chromium.org/2184423002
2016-07-28 14:58:08 -07:00
reed
9b84f8c812
turn bitmaps into images during recording
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2185563003
Review-Url: https://codereview.chromium.org/2185563003
2016-07-28 14:26:13 -07:00
bsalomon
4484d6ac78
Remove unused render target members from GL and VK sk_app::WindowContext classes
...
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2191673002
Review-Url: https://codereview.chromium.org/2191673002
2016-07-28 11:49:12 -07:00
csmartdalton
a26efc9bad
Include NVPR configs in dm runs on TegraX1
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2174353008
Review-Url: https://codereview.chromium.org/2174353008
2016-07-28 11:48:13 -07:00