| Commit message (Collapse) | Author | Age |
|
|
|
|
|
|
|
|
|
| |
hoisting key
Adding the rendering canvas' CTM to the layer hoisting key (i.e., Add support for hoisting layers in pictures drawn with a matrix - https://codereview.chromium.org/748853002/) has increased the cache miss rate due to accumulated floating point error. This CL fixes part of the issue by using the chain of operation indices leading to each saveLayer as the key. The canvas' CTM must still form part of the key but should be less subject to accumulated error.
BUG=skia:2315
Review URL: https://codereview.chromium.org/753253002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL rebaselines:
Two GMs from https://codereview.chromium.org/748853002 (Add support for hoisting layers in pictures drawn with a matrix) which are only now showing up:
S4 Debug multipicturedraw_invpathclip_tiled_gpu
S4 Debug multipicturedraw_pathclip_tiled_gpu
Two GMs most likely from https://codereview.chromium.org/747043004 (Use scratch keys for stencil buffers):
Win8 Debug dashing2_nvprmsaa4
Win8 Debug glyph_pos_n_b_msaa16
Note that the dashing2 difference fixes an erroneously set pixel in the expected.
TBR=bsalomon@google.com, tomhudson@google.com
Review URL: https://codereview.chromium.org/758023003
|
|
|
|
|
|
|
|
| |
These appear to be recurrent flakes
TBR=tomhudson@google.com
Review URL: https://codereview.chromium.org/750943003
|
|
|
|
|
|
|
|
| |
BUG=skia:2889
Committed: https://skia.googlesource.com/skia/+/91175f19664a62851da4ca4e0984a7c7c45b258f
Review URL: https://codereview.chromium.org/747043004
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add support for hoisting layers in pictures drawn with a matrix - https://codereview.chromium.org/748853002 changed the following GMs:
multipicturedraw_sierpinski_tiled
multipicturedraw_rrectclip_tiled
multipicturedraw_rectclip_tiled
Note that it also, somewhat mysteriously, "fixed" the rendering bugs in some of the S4's multipicturedraw images (see skia:3116).
TBR=tomhudson@google.com
Review URL: https://codereview.chromium.org/755183003
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit 91175f19664a62851da4ca4e0984a7c7c45b258f.
Revert "Cleanup res cache bench and split out into a unit test."
This reverts commit 4e4303f002c5958c6c958e7ba8e49b24c25f0b22.
Revert "rebaselines"
This reverts commit 65ba7b57759bfca60b24bc34dc46fc8caaf146f0.
TBR=tomhudson@google.com
Review URL: https://codereview.chromium.org/752233002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/754833003/)
Reason for revert:
Reverted causative patchset (https://codereview.chromium.org/747043004)
Original issue's description:
> More rebaselines
>
> TBR=
> NOTRY=true
> NOTREECHECKS=true
>
> Committed: https://skia.googlesource.com/skia/+/0ef7ba2a43a41e8e1e49a5187073db6dcca345b4
TBR=bsalomon@google.com
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/756633004
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/755653003/)
Reason for revert:
Reverted causative patch (https://codereview.chromium.org/747043004)
Original issue's description:
> more rebaselines
>
> TBR=
> NOTRY=true
> NOTREECHECKS=true
>
> Committed: https://skia.googlesource.com/skia/+/f2fdc8b746acac9daa94b21d3849f60a87064fd4
TBR=bsalomon@google.com
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/754103002
|
|
|
|
|
|
|
|
|
|
| |
One Android bot is failing on the rect test case.
TBR=tomhudson@google.com
NOTRY=true
NOTREECHECKS=true
Review URL: https://codereview.chromium.org/750203003
|
|
|
|
|
|
|
|
| |
TBR=tomhudson@google.com
NOTRY=true
NOTREECHECKS=true
Review URL: https://codereview.chromium.org/754783003
|
|
|
|
|
|
|
|
| |
TBR=tomhudson@google.com
NOTRY=true
NOTREECHECKS=true
Review URL: https://codereview.chromium.org/753103005
|
|
|
|
|
|
|
|
| |
TBR=
NOTRY=true
NOTREECHECKS=true
Review URL: https://codereview.chromium.org/754833003
|
|
|
|
|
|
|
|
| |
TBR=
NOTRY=true
NOTREECHECKS=true
Review URL: https://codereview.chromium.org/755653003
|
|
|
|
|
|
|
|
| |
NOTRY=true
NOTREECHECKS=true
TBR=
Review URL: https://codereview.chromium.org/753133003
|
|
|
|
|
|
|
|
| |
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/752693002
|
|
|
|
|
|
|
|
|
| |
The Ubuntu12 msaa4 image seems to flake randomly
The S4 images are covered by skia:3116 (Galaxy S4 exhibits some saveLayer bugs (both normal and with hoisting))
TBR=humper@google.com
Review URL: https://codereview.chromium.org/748773003
|
|
|
|
|
|
|
| |
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/744383002
|
|
|
|
|
|
|
|
|
|
| |
Rebaseline after https://codereview.chromium.org/712223002. The
difference is a result of using different bounding boxes to cover
gradient fills, which caused a few pixels do differ by a value of one.
BUG=skia:
Review URL: https://codereview.chromium.org/746603002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/716243005
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/726373004
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/742893002
|
|
|
|
|
|
|
| |
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/743563003
|
|
|
|
|
|
| |
BUG=skia:2049
Review URL: https://codereview.chromium.org/741533002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/738943003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/727063004
|
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=borenet
NOTRY=true
Review URL: https://codereview.chromium.org/735833002
|
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
NOTRY=True
Review URL: https://codereview.chromium.org/738743003
|
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
NOTRY=True
Review URL: https://codereview.chromium.org/741433002
|
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
NOTRY=True
Review URL: https://codereview.chromium.org/715223003
|
|
|
|
|
|
|
| |
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/734243002
|
|
|
|
|
|
|
| |
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/726103004
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/732693002
|
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
NOTRY=True
Review URL: https://codereview.chromium.org/733233002
|
|
|
|
|
|
| |
TBR=
Review URL: https://codereview.chromium.org/733133003
|
|
|
|
|
|
|
|
| |
NOTRY=true
BUG=skia:3096
Review URL: https://codereview.chromium.org/731553002
|
|
|
|
|
|
|
|
|
|
|
|
| |
Combines adjacent DrawPaths commands into one single call when a
conservative set of conditions are met. The end result is that whole
paragraphs can be drawn with a single call to
gliStencilThenCoverFillPathInstancedNV(), rather than one call for
each line.
BUG=skia:
Review URL: https://codereview.chromium.org/712223002
|
|
|
|
|
|
|
| |
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/715363006
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/717033002
|
|
|
|
|
|
|
|
| |
This reverts commit c71ffd4e76d1abcd28ac74463349970f60a3350f.
TBR=
Review URL: https://codereview.chromium.org/683003003
|
|
|
|
|
|
| |
TBR=jcgregorio@google.com
Review URL: https://codereview.chromium.org/718763002
|
|
|
|
|
|
|
| |
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/691143006
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/699023003
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL rebaselines the buggy multipicturedraw GMs related to:
Always enable layer hoisting in Skia (https://codereview.chromium.org/698493002/)
TBR=jcgregorio@google.com
BUG=skia:3116
Review URL: https://codereview.chromium.org/715823002
|
|
|
|
|
|
|
|
|
|
| |
This CL rebaselines the multipicturedraw GMs as a follow up to:
Always enable layer hoisting in Skia (https://codereview.chromium.org/698493002/)
TBR=jcgregorio@google.com
Review URL: https://codereview.chromium.org/714153002
|
|
|
|
|
|
|
| |
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/707413003
|
|
|
|
|
|
| |
TBR=jcgregorio@google.com
Review URL: https://codereview.chromium.org/684413004
|
|
|
|
|
|
| |
TBR=jcgregorio@google.com
Review URL: https://codereview.chromium.org/713893002
|