cjacek [Wed, 10 Sep 2014 21:28:06 +0000 (14:28 -0700)]
Fixed void* to uint64_t cast on win64 in TraceID.
On win64, long is 32-bit. Found by GCC (mingw-w64) build, where this causes an error.
This is the Skia version of https://codereview.chromium.org/
374043002 .
R=bungeman@google.com, humper@google.com
Author: cjacek@gmail.com
Review URL: https://codereview.chromium.org/
510923002
sugoi [Wed, 10 Sep 2014 20:40:18 +0000 (13:40 -0700)]
Added test for non cached ETC1
BUG=skia:
R=bsalomon@chromium.org, robertphillips@chromium.org, robertphillips@google.com
Author: sugoi@chromium.org
Review URL: https://codereview.chromium.org/
564493002
mtklein [Wed, 10 Sep 2014 19:19:30 +0000 (12:19 -0700)]
Measure picture recording speed in nanobench.
Today we measure SkPicture playback speed, but not the time it takes to record
the SkPicture. This fixes that by reading SKPs from disk and re-recording them.
On the console, recording shows up first as the nonrendering skp benches,
followed later by the usual playback benches:
maxrss loops min median mean max stddev samples config bench
51M 2 165µs 168µs 169µs 178µs 3% ▆▄▃█▂▄▁▂▁▁ nonrendering tabl_slashdot.skp
57M 1 9.72ms 9.77ms 9.79ms 9.97ms 1% █▂▂▅▃▂▁▄▂▁ nonrendering desk_pokemonwiki.skp
57M 32 2.92µs 2.96µs 3.03µs 3.46µs 6% ▅▁▁▁▁▁▁█▂▁ nonrendering desk_yahoosports.skp
...
147M 1 3.86ms 3.87ms 3.97ms 4.81ms 7% █▁▁▁▁▁▁▁▁▁ 8888 tabl_slashdot.skp_1
147M 1 4.54ms 4.56ms 4.55ms 4.56ms 0% █▅▇▅█▅▂▁▅▁ 565 tabl_slashdot.skp_1
147M 2 3.08ms 3.24ms 4.17ms 8.18ms 50% █▁▁█▁▁▁▁▁▁ gpu tabl_slashdot.skp_1
147M 1 1.61ms 1.62ms 1.69ms 2.33ms 13% █▁▁▁▁▁▁▁▁▁ 8888 desk_pokemonwiki.skp_1
147M 1 1.44ms 1.44ms 1.45ms 1.47ms 1% ▅▂█▂▂▅▁▁▂▁ 565 desk_pokemonwiki.skp_1
...
On skiaperf.com, they'll also be separated out from playback benches by bench_type.
BUG=skia:
R=reed@google.com, mtklein@google.com, jcgregorio@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
559153002
mtklein [Wed, 10 Sep 2014 19:11:22 +0000 (12:11 -0700)]
Clean up old custom tools now subsumed by nanobench.
BUG=skia:
R=mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
551263003
mtklein [Wed, 10 Sep 2014 19:05:59 +0000 (12:05 -0700)]
Distinguish common and unique names for skiaperf.com.
Turns out we tack on the size post-facto in ResultsWriter::bench(), so the only
place we need getUniqueName() to differ from getName() is SKPBench.
BUG=skia:
R=jcgregorio@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
552303004
tomhudson [Wed, 10 Sep 2014 17:45:42 +0000 (10:45 -0700)]
Fix iteration bounds of Android GL API initialization
A dormant error in https://codereview.chromium.org/
319043005 was
exposed on some new platforms; this makes sure loop iterations
are bounded.
BUG=
R=joshualitt@google.com, joshualitt@chromium.org
Author: tomhudson@google.com
Review URL: https://codereview.chromium.org/
557363002
reed [Wed, 10 Sep 2014 17:20:24 +0000 (10:20 -0700)]
make set3DMask virtual, so we can safely notify the shadercontext
also boost preallocated storage to account for this combo of bitmapshader + emboss + colorfilter
BUG=skia:
R=djsollen@google.com
Author: reed@google.com
Review URL: https://codereview.chromium.org/
563563002
Mike Klein [Wed, 10 Sep 2014 15:10:48 +0000 (11:10 -0400)]
Expectation file update for bot changes.
BUG=skia:
Review URL: https://codereview.chromium.org/
560873002
bungeman [Wed, 10 Sep 2014 14:49:05 +0000 (07:49 -0700)]
Detect presence of dwrite_1.h.
This introduces the SK_HAS_DWRITE_1_H define which may be set at
build time or will be true when WINVER_MAXVER >= 0x0602 .
The dwrite_1.h header is available starting in Windows SDK 8.0.
This change supports users who must still use Windows SDK 7.0.
It also allows for easier local testing of the older interfaces
on newer versions of Windows.
See also: https://bugzilla.mozilla.org/show_bug.cgi?id=1053652
R=george@mozilla.com, mtklein@google.com
Author: bungeman@google.com
Review URL: https://codereview.chromium.org/
552383002
reed [Wed, 10 Sep 2014 14:23:32 +0000 (07:23 -0700)]
disable conservative rasterclip on gpu, until we understand perspective
BUG=skia:
R=bsalomon@google.com
Author: reed@google.com
Review URL: https://codereview.chromium.org/
561763002
borenet [Wed, 10 Sep 2014 04:14:51 +0000 (21:14 -0700)]
Update SKP version to 97
Automatic commit by the RecreateSKPs bot.
TBR=
Author: borenet@google.com
Review URL: https://codereview.chromium.org/
560743002
qiankun.miao [Wed, 10 Sep 2014 02:24:36 +0000 (19:24 -0700)]
Fix format of nanobench result
Column of samples is too wide. This makes config is not align with the
'config' title. Pad 'samples' tilte with some whitespaces to fix this
issue.
BUG=skia:
R=mtklein@google.com
Author: qiankun.miao@intel.com
Review URL: https://codereview.chromium.org/
549153005
reed [Wed, 10 Sep 2014 02:22:37 +0000 (19:22 -0700)]
correct prev suppression
TBR=
NOTRY=True
Author: reed@chromium.org
Review URL: https://codereview.chromium.org/
543903005
reed [Wed, 10 Sep 2014 02:08:28 +0000 (19:08 -0700)]
suppress multipicturedraw_pathclip_tiled_nvprmsaa4 after the conservative-rasterclip cl
TBR=
NOTRY=True
Author: reed@chromium.org
Review URL: https://codereview.chromium.org/
555403002
reed [Wed, 10 Sep 2014 01:46:22 +0000 (18:46 -0700)]
Revert of Revert of allow canvas to force conservative clips (for speed) (patchset #1 id:1 of https://codereview.chromium.org/
554033003/)
Reason for revert:
May just rebaseline, plus want to see the results of the chrome tests, so re-trying this CL.
Original issue's description:
> Revert of allow canvas to force conservative clips (for speed) (patchset #7 id:120001 of https://codereview.chromium.org/
541593005/)
>
> Reason for revert:
> multipicturedraw failed on nvprmsaa -- don't know why yet
>
> Original issue's description:
> > Allow SkCanvas to be initialized to force conservative rasterclips. This has the following effects:
> >
> > 1. Queries to the current clip will be conservatively large. This can mean the quickReject may return false more often.
> >
> > 2. The conservative clips mean less work is done.
> >
> > 3. Enabled by default for Gpu, Record, and NoSaveLayer canvases.
> >
> > 4. API is private for now.
> >
> > Committed: https://skia.googlesource.com/skia/+/
27a5e656c3d6ef22f9cb34de18e1b960da3aa241
>
> TBR=robertphillips@google.com,bsalomon@google.com,mtklein@google.com,junov@google.com
> NOTREECHECKS=true
> NOTRY=true
>
> Committed: https://skia.googlesource.com/skia/+/
6f09709519b79a1159f3826645f1c5fbc101ee11
R=robertphillips@google.com, bsalomon@google.com, mtklein@google.com, junov@google.com, reed@google.com
TBR=bsalomon@google.com, junov@google.com, mtklein@google.com, reed@google.com, robertphillips@google.com
NOTREECHECKS=true
NOTRY=true
Author: reed@chromium.org
Review URL: https://codereview.chromium.org/
560713002
bungeman [Wed, 10 Sep 2014 01:38:04 +0000 (21:38 -0400)]
Rebaselines for
6bc2c94de33 (fontmgr_iter).
Rebaselines for "Add font fallback support to SkFontMgr_fontconfig."
humper [Tue, 9 Sep 2014 22:57:33 +0000 (15:57 -0700)]
disable webtry seccmp on non-linux build
BUG=skia:
R=jcgregorio@google.com
Author: humper@google.com
Review URL: https://codereview.chromium.org/
555993002
humper [Tue, 9 Sep 2014 22:57:26 +0000 (15:57 -0700)]
simple build script for webtry that re-generates the code template
before compilation
BUG=skia:
R=jcgregorio@google.com
Author: humper@google.com
Review URL: https://codereview.chromium.org/
552343003
Mike Klein [Tue, 9 Sep 2014 21:52:40 +0000 (17:52 -0400)]
rebase a couple oddballs
BUG=skia:
Review URL: https://codereview.chromium.org/
555193004
Mike Klein [Tue, 9 Sep 2014 20:00:58 +0000 (16:00 -0400)]
Rebase intel rhb gpu GMs.
Looks like I accidentally re-enabled GPU mode in GM for Intel RHB.
Rebaseline it so it's green today before we turn it off tomorrow.
BUG=skia:
Review URL: https://codereview.chromium.org/
552413002
reed [Tue, 9 Sep 2014 19:51:10 +0000 (12:51 -0700)]
Revert of allow canvas to force conservative clips (for speed) (patchset #7 id:120001 of https://codereview.chromium.org/
541593005/)
Reason for revert:
multipicturedraw failed on nvprmsaa -- don't know why yet
Original issue's description:
> Allow SkCanvas to be initialized to force conservative rasterclips. This has the following effects:
>
> 1. Queries to the current clip will be conservatively large. This can mean the quickReject may return false more often.
>
> 2. The conservative clips mean less work is done.
>
> 3. Enabled by default for Gpu, Record, and NoSaveLayer canvases.
>
> 4. API is private for now.
>
> Committed: https://skia.googlesource.com/skia/+/
27a5e656c3d6ef22f9cb34de18e1b960da3aa241
R=robertphillips@google.com, bsalomon@google.com, mtklein@google.com, junov@google.com
TBR=bsalomon@google.com, junov@google.com, mtklein@google.com, robertphillips@google.com
NOTREECHECKS=true
NOTRY=true
Author: reed@google.com
Review URL: https://codereview.chromium.org/
554033003
bungeman [Tue, 9 Sep 2014 19:50:36 +0000 (12:50 -0700)]
Add font fallback support to SkFontMgr_fontconfig.
R=mtklein@google.com, djsollen@google.com
Author: bungeman@google.com
Review URL: https://codereview.chromium.org/
489733005
senorblanco [Tue, 9 Sep 2014 19:37:16 +0000 (12:37 -0700)]
When parsing arguments on Windows, check malloc() for failure.
Note: this code is only executed by Skia utilities AFAIK, not by
external clients.
BUG=skia:2642
R=reed@google.com
Author: senorblanco@chromium.org
Review URL: https://codereview.chromium.org/
548343002
jshin [Tue, 9 Sep 2014 19:30:57 +0000 (12:30 -0700)]
Update the alias mapping for Noto Sans CJK.
To make the family names 'future-proof', we decided to shorten
the name of Noto Sans CJK, which requires a change in the alias
table in Skia.
Chrome OS CL (actually updating the fonts) and Chrome CL (updating the
font preferences on CrOS) are going together with this CL.
BUG=412151
TEST=With the above two CLs in on Chrome OS, Noto Sans CJK {JP, SC} are
used when MS P Gothic / Simhei are asked for by a web page.
R=bungeman@google.com
Author: jshin@chromium.org
Review URL: https://codereview.chromium.org/
554943002
reed [Tue, 9 Sep 2014 19:19:30 +0000 (12:19 -0700)]
Allow SkCanvas to be initialized to force conservative rasterclips. This has the following effects:
1. Queries to the current clip will be conservatively large. This can mean the quickReject may return false more often.
2. The conservative clips mean less work is done.
3. Enabled by default for Gpu, Record, and NoSaveLayer canvases.
4. API is private for now.
R=robertphillips@google.com, bsalomon@google.com, mtklein@google.com, junov@google.com
Author: reed@google.com
Review URL: https://codereview.chromium.org/
541593005
mtklein [Tue, 9 Sep 2014 19:07:05 +0000 (12:07 -0700)]
Make -d x86 have the default GPU cache size.
We have no a priori reason to limit the cache for new devices.
BUG=skia:
R=djsollen@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
556863006
george [Tue, 9 Sep 2014 18:33:57 +0000 (11:33 -0700)]
Outset the stroke width when computing the bounds for drawing a stroked rect, or 1 if it's a hairline
Adds a testcase for stroke rect bug
R=reed@google.com, reed1
BUG=skia:
Author: george@mozilla.com
Review URL: https://codereview.chromium.org/
552743004
senorblanco [Tue, 9 Sep 2014 16:15:04 +0000 (09:15 -0700)]
Allow negative values in SkBlurImageFilter sigma.
There are two ways negative sigma values may occur: in
the original filter parameters, or after multiplication
by a negative scaling CTM. The former case is
invalid according to the spec, so we continue to check
for it at validation time. In the latter case, we should
interpret it as a horizontal flip in the kernel pixel
access, and simply take the absolute value (since the
filter kernel is symmetric).
Also refactor all this logic into a single place for the
CPU, GPU and onFilterBounds() paths.
BUG=https://code.google.com/p/chromium/issues/detail?id=409602
R=sugoi@google.com, reed@google.com, sugoi@chromium.org
Author: senorblanco@chromium.org
Review URL: https://codereview.chromium.org/
555603002
mtklein [Tue, 9 Sep 2014 15:42:04 +0000 (08:42 -0700)]
Back to hashing source content, not .png.
BUG=skia:
R=jcgregorio@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
560453002
fmalita [Tue, 9 Sep 2014 15:12:46 +0000 (08:12 -0700)]
Revert of Use SkBitmapCache to optimize readPixels on a texture-backed bitmap (patchset #6 id:100001 of https://codereview.chromium.org/
533323002/)
Reason for revert:
Component inversions in Blink's printing/webgl-repeated-printing.html: https://storage.googleapis.com/chromium-layout-test-archives/linux_blink_rel/24033/layout-test-results/printing/webgl-repeated-printing-diff.txt
Reverting to unblock rolls.
Original issue's description:
> Use SkBitmapCache to optimize readPixels on a texture-backed bitmap
>
> BUG=skia:2786
>
> Committed: https://skia.googlesource.com/skia/+/
95fd68e5ccd242a91e6dd827dd695f18661efbe6
R=junov@chromium.org, reed@google.com, bsalomon@google.com, piotaixr@chromium.org
TBR=bsalomon@google.com, junov@chromium.org, piotaixr@chromium.org, reed@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:2786
Author: fmalita@chromium.org
Review URL: https://codereview.chromium.org/
551523003
sugoi [Tue, 9 Sep 2014 15:12:34 +0000 (08:12 -0700)]
Adding texture uploads without cache for YUV and ETC1
BUG=skia:
R=bsalomon@google.com, robertphillips@google.com
Author: sugoi@chromium.org
Review URL: https://codereview.chromium.org/
552843004
mtklein [Tue, 9 Sep 2014 14:59:46 +0000 (07:59 -0700)]
Update DM JSON format.
Ex. dm --match patch -w bad --key arch x86 gpu nvidia model z620 --properties git_hash abcd build_number 20 ->
{
"build_number" : "20",
"git_hash" : "abcd",
"key" : {
"arch" : "x86",
"gpu" : "nvidia",
"model" : "z620"
},
"results" : [
{
"key" : {
"config" : "565",
"name" : "ninepatch-stretch"
},
"md5" : "
f78cfafcbabaf815f3dfcf61fb59acc7",
"options" : {
"source_type" : "GM"
}
},
{
"key" : {
"config" : "8888",
"name" : "ninepatch-stretch"
},
"md5" : "
3e8a42f35a1e76f00caa191e6310d789",
"options" : {
"source_type" : "GM"
}
},
...
This breaks -r, but that's okay. Going to follow up this CL with one that removes that entirely.
BUG=skia:
R=stephana@google.com, jcgregorio@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
551873003
mtklein [Tue, 9 Sep 2014 14:36:57 +0000 (07:36 -0700)]
Remove expectations / -r from DM entirely.
It's getting tricky to coordinate changes to output for bots with -r,
and -r is not widely used. The suggested alternative is to run skdiff.
BUG=skia:
R=jcgregorio@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
553583004
george [Tue, 9 Sep 2014 13:14:57 +0000 (06:14 -0700)]
SkRasterClip::quickReject doesn't need to check for emptiness as SkIRect::Intersect already does that for us
R=reed@google.com, reed1
BUG=skia:
Author: george@mozilla.com
Review URL: https://codereview.chromium.org/
554743007
borenet [Tue, 9 Sep 2014 04:08:41 +0000 (21:08 -0700)]
Update SKP version to 96
Automatic commit by the RecreateSKPs bot.
TBR=
Author: borenet@google.com
Review URL: https://codereview.chromium.org/
551263002
jbroman [Mon, 8 Sep 2014 20:24:33 +0000 (13:24 -0700)]
Remove a linear search in SkPictureRecord::addTextBlob.
It doesn't seem that harmful to have repeated text blobs in the array
if they occur within the same picture recording. And that does not seem
to be a common occurrence right now anyhow.
BUG=skia:2919
R=fmalita@chromium.org, reed@google.com, mtklein@google.com
Author: jbroman@chromium.org
Review URL: https://codereview.chromium.org/
550043003
mtklein [Mon, 8 Sep 2014 19:42:23 +0000 (12:42 -0700)]
Hash .pngs instead of SkBitmaps.
This has the nice property of being able to double-check hashes after the fact.
mtklein@mtklein ~/skia (hash-png)> md5sum bad/8888/3x3bitmaprect.png
deede70ab2f34067d461fb4a93332d4c bad/8888/3x3bitmaprect.png
mtklein@mtklein ~/skia (hash-png)> grep 3x3bitmaprect_8888 bad/dm.json
"3x3bitmaprect_8888" : "
deede70ab2f34067d461fb4a93332d4c",
I have checked that no two premultiplied colors map to the same unpremultiplied
color (math nerds: unpremultiplication is injective), so a change in
premultiplied SkBitmap will always imply a change in the encoded
unpremultiplied .png. This means, it's safe to hash .pngs; we won't miss
subtle changes.
BUG=skia:
R=jcgregorio@google.com, stephana@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
549203003
bsalomon [Mon, 8 Sep 2014 19:06:20 +0000 (12:06 -0700)]
Make GrEffectStage copy constructor work correctly with pending execution refs.
R=egdaniel@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
551083002
robertphillips [Mon, 8 Sep 2014 18:37:59 +0000 (11:37 -0700)]
Fix two SkRecord-backed layer hoisting bugs
The two bugs are/were:
The old loop to draw the hoisted layers included the saveLayer call which caused double application of the layer's paint (This is the +1 change).
The hoisted layer is intended to be drawn in device coordinates. The old code was drawing it in the coordinate space of the saveLayer thus it was misplaced (This is the setMatrix change).
Committed: https://skia.googlesource.com/skia/+/
7c0cfd4ff8f6db50a8731c886db732b106268937
R=bsalomon@google.com
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
551843002
mtklein [Mon, 8 Sep 2014 18:33:48 +0000 (11:33 -0700)]
egdaniel [Mon, 8 Sep 2014 17:48:37 +0000 (10:48 -0700)]
Remove extra proctected in GrRODrawState.h
BUG=skia:
R=bsalomon@google.com
Author: egdaniel@google.com
Review URL: https://codereview.chromium.org/
550243002
bsalomon [Mon, 8 Sep 2014 17:45:58 +0000 (10:45 -0700)]
Remove debug-only ref of GrEffectStage
BUG=skia:2889
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
553973002
caryclark [Mon, 8 Sep 2014 17:25:38 +0000 (10:25 -0700)]
fail when coincidence is too far apart
TBR=
BUG=410552
Author: caryclark@google.com
Review URL: https://codereview.chromium.org/
556433002
robertphillips [Mon, 8 Sep 2014 17:10:51 +0000 (10:10 -0700)]
Revert of Fix two SkRecord-backed layer hoisting bugs (patchset #1 id:1 of https://codereview.chromium.org/
551843002/)
Reason for revert:
turning bots red
Original issue's description:
> Fix two SkRecord-backed layer hoisting bugs
>
> The two bugs are/were:
> The old loop to draw the hoisted layers included the saveLayer call which caused double application of the layer's paint (This is the +1 change).
>
> The hoisted layer is intended to be drawn in device coordinates. The old code was drawing it in the coordinate space of the saveLayer thus it was misplaced (This is the setMatrix change).
>
> Committed: https://skia.googlesource.com/skia/+/
7c0cfd4ff8f6db50a8731c886db732b106268937
R=bsalomon@google.com
TBR=bsalomon@google.com
NOTREECHECKS=true
NOTRY=true
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
553983003
robertphillips [Mon, 8 Sep 2014 16:53:58 +0000 (09:53 -0700)]
Fix two SkRecord-backed layer hoisting bugs
The two bugs are/were:
The old loop to draw the hoisted layers included the saveLayer call which caused double application of the layer's paint (This is the +1 change).
The hoisted layer is intended to be drawn in device coordinates. The old code was drawing it in the coordinate space of the saveLayer thus it was misplaced (This is the setMatrix change).
R=bsalomon@google.com
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
551843002
mtklein [Mon, 8 Sep 2014 16:12:28 +0000 (09:12 -0700)]
Let .skps have Expectations (i.e. work with -r) too.
BUG=skia:
R=jcgregorio@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
549183002
mtklein [Mon, 8 Sep 2014 15:41:37 +0000 (08:41 -0700)]
Turn on the save-layer optimization.
BUG=skia:
R=robertphillips@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
551853002
piotaixr [Mon, 8 Sep 2014 15:07:47 +0000 (08:07 -0700)]
Use SkBitmapCache to optimize readPixels on a texture-backed bitmap
BUG=skia:2786
R=junov@chromium.org, reed@google.com, bsalomon@google.com
Author: piotaixr@chromium.org
Review URL: https://codereview.chromium.org/
533323002
mtklein [Mon, 8 Sep 2014 15:05:18 +0000 (08:05 -0700)]
Start to rework DM JSON handling.
DM's striking off into its own JSON world. This gets strawman implementations
in place for writing and reading a JSON file mapping test name to hashes.
For what it's worth, I basically want to change _all_ these pieces,
- MD5 is slow and we can replace it with something faster,
- JSON schema needs room to grow more data,
- it'd be nice to hash once instead of twice when reading and writing,
- this code wants lots of refactoring,
but this gives us a starting platform to work on these bits at our leisure.
E.x. file for now:
mtklein@mtklein ~/skia (dm)> cat good/dm.json
{
"3x3bitmaprect_565" : "
fc70d985fbfbe70e3a3c9dc626d4f5bc",
"3x3bitmaprect_8888" : "
df1591dde35907399734ea19feb76663",
"3x3bitmaprect_gpu" : "
df1591dde35907399734ea19feb76663",
"aaclip_565" : "
1862798689b838a7ab0dc0652b9ace3a",
"aaclip_8888" : "
47bb314329f0ce243f1d83fd583decb7",
"aaclip_gpu" : "
75f72412d0ef4815770202d297246e7d",
...
BUG=skia:
R=jcgregorio@google.com, stephana@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
546873002
mtklein [Mon, 8 Sep 2014 14:31:18 +0000 (07:31 -0700)]
Turn on all pixel-perfect SkRecord optimizations.
BUG=skia:
R=robertphillips@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
550083006
borenet [Mon, 8 Sep 2014 03:52:36 +0000 (20:52 -0700)]
Update SKP version to 95
Automatic commit by the RecreateSKPs bot.
TBR=
Author: borenet@google.com
Review URL: https://codereview.chromium.org/
552663002
borenet [Sun, 7 Sep 2014 03:40:51 +0000 (20:40 -0700)]
Update SKP version to 94
Automatic commit by the RecreateSKPs bot.
TBR=
Author: borenet@google.com
Review URL: https://codereview.chromium.org/
541173005
borenet [Sat, 6 Sep 2014 04:23:05 +0000 (21:23 -0700)]
Update SKP version to 93
Automatic commit by the RecreateSKPs bot.
TBR=
Author: borenet@google.com
Review URL: https://codereview.chromium.org/
543343002
reed [Sat, 6 Sep 2014 00:37:38 +0000 (17:37 -0700)]
move rasterclip_ helper into rasterclip
R=robertphillips@google.com, reed@chromium.org
Author: reed@google.com
Review URL: https://codereview.chromium.org/
546113002
bsalomon [Fri, 5 Sep 2014 21:27:43 +0000 (14:27 -0700)]
Make GrDrawState and GrEffectStage use the pending io/exec ref mechanisms.
BUG=skia:2889
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
542723004
stephana [Fri, 5 Sep 2014 20:51:24 +0000 (13:51 -0700)]
Refactored frontend for the rebaseline server.
This is going to serve as the starting point for the new front-end once the backend is rewritten.
BUG=skia:
NOTRY=true
R=jcgregorio@google.com
Author: stephana@google.com
Review URL: https://codereview.chromium.org/
538613002
bsalomon [Fri, 5 Sep 2014 20:34:00 +0000 (13:34 -0700)]
"NULL !=" = NULL
R=reed@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
544233002
stephana [Fri, 5 Sep 2014 20:21:42 +0000 (13:21 -0700)]
For certain input skpdiff segfaulted. This prevents the segfault and should fix the failed test in housekeeping.
BUG=skia:2902
R=mtklein@google.com, rmistry@google.com
Author: stephana@google.com
Review URL: https://codereview.chromium.org/
546753003
bsalomon [Fri, 5 Sep 2014 19:23:12 +0000 (12:23 -0700)]
Don't take a ref on GP in AutoEffectRestore.
BUG=skia:2889
R=joshualitt@chromium.org, robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
546043002
joshualitt [Fri, 5 Sep 2014 17:45:30 +0000 (10:45 -0700)]
Small fix for RODrawState
BUG=skia:
R=egdaniel@google.com, joshualitt@google.com
Author: joshualitt@chromium.org
Review URL: https://codereview.chromium.org/
545273002
mtklein [Fri, 5 Sep 2014 17:36:39 +0000 (10:36 -0700)]
Suppress mystery leak.
BUG=skia:2916
R=caryclark@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
543063002
bsalomon [Fri, 5 Sep 2014 16:18:51 +0000 (09:18 -0700)]
Change behavior of GrDS::AutoEffectRestort wrt GPs.
BUG=skia:2889
R=robertphillips@google.com, joshualitt@chromium.org
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
545253002
bsalomon [Fri, 5 Sep 2014 13:13:43 +0000 (06:13 -0700)]
Add pop_back() to GrAllocator and add unit test.
BUG=skia:2889
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
538183002
borenet [Fri, 5 Sep 2014 04:14:04 +0000 (21:14 -0700)]
Update SKP version to 92
Automatic commit by the RecreateSKPs bot.
TBR=
Author: borenet@google.com
Review URL: https://codereview.chromium.org/
546583003
bsalomon [Thu, 4 Sep 2014 21:13:44 +0000 (14:13 -0700)]
Add method to GrProgramResource to record that pending a execution was completed.
BUG=skia:2889
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
543833002
mtklein [Thu, 4 Sep 2014 21:12:44 +0000 (14:12 -0700)]
Implement all SkCanvas overrides that SkPictureRecord does.
Primarily this is for isDrawingToLayer(). drawData() and onNewSurface() are
for completeness.
BUG=409138
R=robertphillips@google.com, mtklein@google.com, reed@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
545613002
joshualitt [Thu, 4 Sep 2014 20:52:34 +0000 (13:52 -0700)]
Updating expectations
TBR=
BUG=skia:
Author: joshualitt@chromium.org
Review URL: https://codereview.chromium.org/
543813003
reed [Thu, 4 Sep 2014 20:18:36 +0000 (13:18 -0700)]
don't use local static bitmap -- racy and unnecessary
BUG=skia:
R=mtklein@google.com
Author: reed@google.com
Review URL: https://codereview.chromium.org/
540993003
bsalomon [Thu, 4 Sep 2014 20:12:37 +0000 (13:12 -0700)]
Add GrProgramElement base class for GrEffect with deferred exec ref.
BUG=skia:2889
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
537773004
reed [Thu, 4 Sep 2014 19:45:18 +0000 (12:45 -0700)]
add SK_API to forward declare in canvas.h
NOTRY=True
TBR=
Author: reed@google.com
Review URL: https://codereview.chromium.org/
541073002
reed [Thu, 4 Sep 2014 19:14:36 +0000 (12:14 -0700)]
remove getTotalClipAsPath, privatize getTotalClip
R=robertphillips@google.com, djsollen@google.com
TBR=robertphilips
Author: reed@google.com
Review URL: https://codereview.chromium.org/
546493002
mtklein [Thu, 4 Sep 2014 17:50:53 +0000 (10:50 -0700)]
disable flaky test
BUG=skia:2912
R=reed@google.com, mtklein@google.com, tfarina@chromium.org
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
537363003
reed [Thu, 4 Sep 2014 16:02:23 +0000 (09:02 -0700)]
cleanup rasterclip helper
R=robertphillips@google.com
TBR=robertphilips
Author: reed@google.com
Review URL: https://codereview.chromium.org/
540983002
joshualitt [Thu, 4 Sep 2014 15:56:46 +0000 (08:56 -0700)]
Initial change to create GeometryProcessor
BUG=skia:
R=bsalomon@google.com, robertphillips@google.com, egdaniel@google.com, jvanverth@google.com
Author: joshualitt@chromium.org
Review URL: https://codereview.chromium.org/
509153002
robertphillips [Thu, 4 Sep 2014 15:42:50 +0000 (08:42 -0700)]
Change SkPicture::draw to playback
R=reed@google.com
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
540963002
Mike Klein [Thu, 4 Sep 2014 15:08:07 +0000 (11:08 -0400)]
Finish off some lingering rebaselines.
BUG=skia:
Review URL: https://codereview.chromium.org/
538023002
Mike Klein [Thu, 4 Sep 2014 15:03:30 +0000 (11:03 -0400)]
rebaseline downsamplebitmaps
BUG=skia:
Review URL: https://codereview.chromium.org/
537913003
qiankun.miao [Thu, 4 Sep 2014 14:36:38 +0000 (07:36 -0700)]
Enable highQualityFilter_SSE2
With SSE2, bitmap_BGRA_8888_A_scale_rotate_bicubic gains about 40%
performance improvement on desktop i7-3770.
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/
b381fa10d8079c58928058bb8a6db32b39f05e51
CQ_EXTRA_TRYBOTS=tryserver.skia:Test-Mac10.6-MacMini4.1-GeForce320M-x86_64-Release-Trybot
R=mtklein@google.com, humper@google.com
Author: qiankun.miao@intel.com
Review URL: https://codereview.chromium.org/
525283002
robertphillips [Thu, 4 Sep 2014 14:24:18 +0000 (07:24 -0700)]
Switch Layer Hoisting over to SkRecord backend
R=bsalomon@google.com
TBR=bsalomon@google.com
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
540543002
Derek Sollenberger [Thu, 4 Sep 2014 13:51:33 +0000 (09:51 -0400)]
Temporarily disable tiling another GM that produces errors with the existing 64-bit ARM toolchain.
BUG=skia:2908
Review URL: https://codereview.chromium.org/
537343003
borenet [Thu, 4 Sep 2014 04:23:26 +0000 (21:23 -0700)]
Update SKP version to 91
Automatic commit by the RecreateSKPs bot.
TBR=
Author: borenet@google.com
Review URL: https://codereview.chromium.org/
544493002
mtklein [Wed, 3 Sep 2014 22:34:37 +0000 (15:34 -0700)]
SkThreadPool ~~> SkTaskGroup
SkTaskGroup is like SkThreadPool except the threads stay in
one global pool. Each SkTaskGroup itself is tiny (4 bytes)
and its wait() method applies only to tasks add()ed to that
instance, not the whole thread pool.
This means we don't need to bring up new thread pools when
tests themselves want to use multithreading (e.g. pathops,
quilt). We just create a new SkTaskGroup and wait for that
to complete. This should be more efficient, and allow us
to expand where we use threads to really latency sensitive
places. E.g. we can probably now use these in nanobench
for CPU .skp rendering.
Now that all threads are sharing the same pool, I think we
can remove most of the custom mechanism pathops tests use
to control threading. They'll just ride on the global pool
with all other tests now.
This (temporarily?) removes the GPU multithreading feature
from DM, which we don't use.
On my desktop, DM runs a little faster (57s -> 55s) in
Debug, and a lot faster in Release (36s -> 24s). The bots
show speedups of similar proportions, cutting more than a
minute off the N4/Release and Win7/Debug runtimes.
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/
9c7207b5dc71dc5a96a2eb107d401133333d5b6f
R=caryclark@google.com, bsalomon@google.com, bungeman@google.com, mtklein@google.com, reed@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
531653002
Mike Klein [Wed, 3 Sep 2014 21:39:25 +0000 (17:39 -0400)]
remove unused n4 expectations
BUG=skia:
Review URL: https://codereview.chromium.org/
537163002
mtklein [Wed, 3 Sep 2014 21:17:48 +0000 (14:17 -0700)]
Revert of SkThreadPool ~~> SkTaskGroup (patchset #4 id:60001 of https://codereview.chromium.org/
531653002/)
Reason for revert:
Leaks, leaks, leaks.
Original issue's description:
> SkThreadPool ~~> SkTaskGroup
>
> SkTaskGroup is like SkThreadPool except the threads stay in
> one global pool. Each SkTaskGroup itself is tiny (4 bytes)
> and its wait() method applies only to tasks add()ed to that
> instance, not the whole thread pool.
>
> This means we don't need to bring up new thread pools when
> tests themselves want to use multithreading (e.g. pathops,
> quilt). We just create a new SkTaskGroup and wait for that
> to complete. This should be more efficient, and allow us
> to expand where we use threads to really latency sensitive
> places. E.g. we can probably now use these in nanobench
> for CPU .skp rendering.
>
> Now that all threads are sharing the same pool, I think we
> can remove most of the custom mechanism pathops tests use
> to control threading. They'll just ride on the global pool
> with all other tests now.
>
> This (temporarily?) removes the GPU multithreading feature
> from DM, which we don't use.
>
> On my desktop, DM runs a little faster (57s -> 55s) in
> Debug, and a lot faster in Release (36s -> 24s). The bots
> show speedups of similar proportions, cutting more than a
> minute off the N4/Release and Win7/Debug runtimes.
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
9c7207b5dc71dc5a96a2eb107d401133333d5b6f
R=caryclark@google.com, bsalomon@google.com, bungeman@google.com, reed@google.com, mtklein@chromium.org
TBR=bsalomon@google.com, bungeman@google.com, caryclark@google.com, mtklein@chromium.org, reed@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
533393002
mtklein [Wed, 3 Sep 2014 21:06:47 +0000 (14:06 -0700)]
SkThreadPool ~~> SkTaskGroup
SkTaskGroup is like SkThreadPool except the threads stay in
one global pool. Each SkTaskGroup itself is tiny (4 bytes)
and its wait() method applies only to tasks add()ed to that
instance, not the whole thread pool.
This means we don't need to bring up new thread pools when
tests themselves want to use multithreading (e.g. pathops,
quilt). We just create a new SkTaskGroup and wait for that
to complete. This should be more efficient, and allow us
to expand where we use threads to really latency sensitive
places. E.g. we can probably now use these in nanobench
for CPU .skp rendering.
Now that all threads are sharing the same pool, I think we
can remove most of the custom mechanism pathops tests use
to control threading. They'll just ride on the global pool
with all other tests now.
This (temporarily?) removes the GPU multithreading feature
from DM, which we don't use.
On my desktop, DM runs a little faster (57s -> 55s) in
Debug, and a lot faster in Release (36s -> 24s). The bots
show speedups of similar proportions, cutting more than a
minute off the N4/Release and Win7/Debug runtimes.
BUG=skia:
R=caryclark@google.com, bsalomon@google.com, bungeman@google.com, mtklein@google.com, reed@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
531653002
bsalomon [Wed, 3 Sep 2014 21:05:49 +0000 (14:05 -0700)]
Add reference base class to GrGpuResourcewith pending IO references.
BUG=skia:2889
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
533343002
Mike Klein [Wed, 3 Sep 2014 21:02:04 +0000 (17:02 -0400)]
fix up GM suppressions
BUG=skia:2911
Review URL: https://codereview.chromium.org/
538723002
robertphillips [Wed, 3 Sep 2014 20:46:02 +0000 (13:46 -0700)]
Remove use of EXPERIMENTAL_getActiveOps from layer hoisting code
This is getting in the way of switching to the SkRecord backend and is of questionable value.
R=bsalomon@google.com
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
539693002
mtklein [Wed, 3 Sep 2014 20:40:52 +0000 (13:40 -0700)]
Turn downsamplebitmap GMs vertical to fit in 2048x2048.
BUG=skia:2911
R=bsalomon@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
536103002
djsollen [Wed, 3 Sep 2014 20:38:32 +0000 (13:38 -0700)]
Revert to enabling all of -O2 as it exposes a bug in the toolchain.
Instead as a workaround we will temporarily disable tiling the
few GMs that produce errors with the existing 64-bit ARM toolchain.
BUG=skia:2908
R=mtklein@google.com
Author: djsollen@google.com
Review URL: https://codereview.chromium.org/
537713002
mtklein [Wed, 3 Sep 2014 20:22:46 +0000 (13:22 -0700)]
On Android, skip tiling for all giantbitmap variants.
BUG=skia:
R=djsollen@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
537693002
piotaixr [Wed, 3 Sep 2014 19:34:57 +0000 (12:34 -0700)]
Remove SkBitmapCache::Find/Add(_,width,height,_)
in favor of the versions having a SkIRect as input parameter
BUG=skia:2909
R=reed@google.com, junov@chromium.org
Author: piotaixr@chromium.org
Review URL: https://codereview.chromium.org/
539643002
reed [Wed, 3 Sep 2014 18:54:58 +0000 (11:54 -0700)]
Hide fields in SkImageInfo
R=rmistry@google.com
TBR=bsalomon
Author: reed@google.com
Review URL: https://codereview.chromium.org/
536003002
piotaixr [Wed, 3 Sep 2014 18:33:13 +0000 (11:33 -0700)]
The key for SkBitmapCache can now be genID+SkIRect
BUG=skia:2894
R=junov@chromium.org, reed@google.com
Author: piotaixr@chromium.org
Review URL: https://codereview.chromium.org/
518983002
robertphillips [Wed, 3 Sep 2014 18:04:30 +0000 (11:04 -0700)]
Replace SkPictureReplacementPlayback with GrRecordReplaceDraw
I think this is sufficiently specialized to keep it in Ganesh for the time being.
R=bsalomon@google.com, mtklein@google.com
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
535953002
djsollen [Wed, 3 Sep 2014 17:44:00 +0000 (10:44 -0700)]
Disable -fexpensive-optimizations on 64-bit ARM.
BUG=skia:2908
R=mtklein@google.com
Author: djsollen@google.com
Review URL: https://codereview.chromium.org/
535113004
mtklein [Wed, 3 Sep 2014 15:57:02 +0000 (08:57 -0700)]
Revert of Enable highQualityFilter_SSE2 (patchset #1 id:1 of https://codereview.chromium.org/
525283002/)
Reason for revert:
Color order looks wrong on Macs:
Before: http://chromium-skia-gm.commondatastorage.googleapis.com/gm/bitmap-64bitMD5/filterbitmap_image_mandrill_16.png/
12823183142873462143.png
After: http://chromium-skia-gm.commondatastorage.googleapis.com/gm/bitmap-64bitMD5/filterbitmap_image_mandrill_16.png/
13683040204546320578.png
Original issue's description:
> Enable highQualityFilter_SSE2
>
> With SSE2, bitmap_BGRA_8888_A_scale_rotate_bicubic gains about 40%
> performance improvement on desktop i7-3770.
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
b381fa10d8079c58928058bb8a6db32b39f05e51
R=humper@google.com, qiankun.miao@intel.com
TBR=humper@google.com, qiankun.miao@intel.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
539523002
jvanverth [Wed, 3 Sep 2014 15:44:59 +0000 (08:44 -0700)]
Update fontcache GM to stress the font atlas.
Pulled out of https://codereview.chromium.org/
466363009/.
R=bsalomon@google.com
Author: jvanverth@google.com
Review URL: https://codereview.chromium.org/
534253003
qiankun.miao [Wed, 3 Sep 2014 15:24:12 +0000 (08:24 -0700)]
Enable highQualityFilter_SSE2
With SSE2, bitmap_BGRA_8888_A_scale_rotate_bicubic gains about 40%
performance improvement on desktop i7-3770.
BUG=skia:
R=mtklein@google.com, humper@google.com
Author: qiankun.miao@intel.com
Review URL: https://codereview.chromium.org/
525283002
bsalomon [Wed, 3 Sep 2014 14:19:50 +0000 (07:19 -0700)]
Remove GrDrawTarget::AutoRenderTargetRestore.
Pass GrRenderTarget in GrGpuG clear\bind methods.
BUG=skia:2889
R=egdaniel@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
533883004