Commit Graph

33543 Commits

Author SHA1 Message Date
Yuqian Li
755778cda0 Update trans limit after zoom changes in viewer
Previously, the trans limit assumes that only touch gesture can
change the zoom level. That's not true on desktops so we need this
fix. Otherwise, we won't be able to translate when we zoomed in.

Bug: skia:
Change-Id: I5901600a0044639a47514ab76b7e1914f04137a2
Reviewed-on: https://skia-review.googlesource.com/116987
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Yuqian Li <liyuqian@google.com>
2018-03-28 20:51:44 +00:00
Ravi Mistry
470d829731 Whitespace change
NoTry: true
Bug: skia:
Change-Id: I6dc0ba66aca94f7f99cb851dda6503c0cb9c302b
Reviewed-on: https://skia-review.googlesource.com/114407
Reviewed-by: Ben Wagner <benjaminwagner@google.com>
Commit-Queue: Ravi Mistry <rmistry@google.com>
2018-03-28 20:43:34 +00:00
Ben Wagner
97182cc8f6 Add color fonts.
BUG=skia:7624

Change-Id: I68a6bb4c6fa541ffad565d970438d93ff1bd8ed6
Reviewed-on: https://skia-review.googlesource.com/109521
Commit-Queue: Ben Wagner <bungeman@google.com>
Reviewed-by: Mike Klein <mtklein@google.com>
Reviewed-by: Herb Derby <herb@google.com>
Reviewed-on: https://skia-review.googlesource.com/116541
Reviewed-on: https://skia-review.googlesource.com/116984
Reviewed-by: Ben Wagner <bungeman@google.com>
2018-03-28 20:14:44 +00:00
skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com
a11bfe5a21 Roll skia/third_party/externals/skcms/ 59fdb4e14..ed6a9bd48 (2 commits)
https://skia.googlesource.com/skcms/+log/59fdb4e14c97..ed6a9bd48726

$ git log 59fdb4e14..ed6a9bd48 --date=short --no-merges --format='%ad %ae %s'
2018-03-28 brianosman Allow anonymous struct/union in MSVC
2018-03-28 mtklein remove -ffp-contract mentions

Created with:
  roll-dep skia/third_party/externals/skcms
BUG=skia:


The AutoRoll server is located here: https://skcms-skia-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=mtklein@google.com

Change-Id: Ieabe5c8f257a4a52118698e3ccd8d4fcade21451
Reviewed-on: https://skia-review.googlesource.com/117000
Commit-Queue: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Reviewed-by: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
2018-03-28 20:14:24 +00:00
Stephen White
050c86768a GrTessellator: hang fix.
Some edges are not coincident with their own endpoints (because floating
point). If this happens for an edge which is a right-enclosing-edge
during the Bentley-Ottman simplify() pass, we end up an infinite loop
attempting to split the edge, since the edge is never to the right of its
endpoint.

The easiest fix is to simply remove the right-enclosing-edge splitting
code. This code was originally added before we had proper
active-edge-list rewinding, and should no longer be necessary.

BUG=802896

Change-Id: Id9f2942b73f01152af8c0088e8c6b1389891d827
Reviewed-on: https://skia-review.googlesource.com/116920
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Stephen White <senorblanco@chromium.org>
2018-03-28 19:55:34 +00:00
Robert Phillips
6eba063b63 Add new GrResourceCache::purgeUnlockedResources variant
TBR=bsalomon@google.com
Change-Id: I05bef1f8a271474db878a046cc1f6ac7b60a15f1
Reviewed-on: https://skia-review.googlesource.com/116801
Reviewed-by: Greg Daniel <egdaniel@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
2018-03-28 18:56:14 +00:00
Ben Wagner
aa5da735fa Check that paint is not null in OveridePaintFilterCanvas.
SkCanvas::drawPicture may provide nullptr for a paint.

BUG=skia:7756

Change-Id: I46651f7e8f0d859a7e75206d93ed1f3b3c0a4bac
Reviewed-on: https://skia-review.googlesource.com/116902
Reviewed-by: Yuqian Li <liyuqian@google.com>
Commit-Queue: Ben Wagner <bungeman@google.com>
2018-03-28 18:02:24 +00:00
Robert Phillips
e47f0a0f16 Remove old method of drawing DDL SKPs from DM (in favor of ViaDDL method)
Change-Id: I6bb270d9df1c76b9d2e384abde603cdda91c9bb6
Reviewed-on: https://skia-review.googlesource.com/116550
Commit-Queue: Mike Klein <mtklein@google.com>
Reviewed-by: Mike Klein <mtklein@google.com>
2018-03-28 17:21:24 +00:00
skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com
282f9daa82 Roll skia/third_party/externals/skcms/ 680ab5a31..59fdb4e14 (3 commits)
https://skia.googlesource.com/skcms/+log/680ab5a31cd2..59fdb4e14c97

$ git log 680ab5a31..59fdb4e14 --date=short --no-merges --format='%ad %ae %s'
2018-03-28 mtklein pack skcms_Curve a bit more, to 32 bytes
2018-03-28 mtklein no need to ship 1D/2D clut stages
2018-03-28 mtklein pack skcms_Curve a bit

Created with:
  roll-dep skia/third_party/externals/skcms


The AutoRoll server is located here: https://skcms-skia-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=mtklein@google.com

Change-Id: I5d6b133f50895611f8f39fc547b70b316957cc95
Reviewed-on: https://skia-review.googlesource.com/116882
Commit-Queue: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Reviewed-by: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
2018-03-28 16:40:44 +00:00
angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com
a9713340c7 Roll skia/third_party/externals/angle2/ c7575da54..78c5eb2ab (1 commit)
c7575da546..78c5eb2ab3

$ git log c7575da54..78c5eb2ab --date=short --no-merges --format='%ad %ae %s'
2018-03-28 geofflang Skip RobustResourceInitTestES3.MaskedStencilClearBuffer on Intel Mac.

Created with:
  roll-dep skia/third_party/externals/angle2


The AutoRoll server is located here: https://angle-skia-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=skia.primary:Perf-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUC5i7RYH-GPU-IntelIris6100-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE,Test-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE,Test-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE,Test-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE,Test-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE,Build-Debian9-Clang-x86_64-Release-ANGLE
TBR=mtklein@google.com

Change-Id: Ifa8887bc3e2f6a98cdac06b6e13c0ee4de607820
Reviewed-on: https://skia-review.googlesource.com/116845
Reviewed-by: <angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: <angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
2018-03-28 16:28:24 +00:00
skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com
ce92cb612d Roll skia/third_party/externals/skcms/ fc0d0c4c7..680ab5a31 (1 commit)
https://skia.googlesource.com/skcms/+log/fc0d0c4c7f9c..680ab5a31cd2

$ git log fc0d0c4c7..680ab5a31 --date=short --no-merges --format='%ad %ae %s'
2018-03-28 mtklein trim profile fields we ignore

Created with:
  roll-dep skia/third_party/externals/skcms


The AutoRoll server is located here: https://skcms-skia-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=mtklein@google.com

Change-Id: I665e43e89a4823c975469a6585ec744c766c1f91
Reviewed-on: https://skia-review.googlesource.com/116844
Commit-Queue: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Reviewed-by: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
2018-03-28 16:09:24 +00:00
Ben Wagner
3aa8bfe0d5 Roll FreeType.
Change-Id: I3fe4e27e70270f4dc31610f98741c63b3574cf96
Reviewed-on: https://skia-review.googlesource.com/116628
Commit-Queue: Mike Klein <mtklein@chromium.org>
Reviewed-by: Mike Klein <mtklein@chromium.org>
2018-03-28 16:06:34 +00:00
Hal Canary
a567e3315b SkPDF: Accept non-monotonic text cluster indices
BUG=skia:7055
Change-Id: Ic1540a15118dbe99db5bcff26dd1235adac5468a
Reviewed-on: https://skia-review.googlesource.com/116725
Reviewed-by: Florin Malita <fmalita@chromium.org>
Commit-Queue: Hal Canary <halcanary@google.com>
2018-03-28 15:29:24 +00:00
Chris Dalton
a466228a61 Delete GPU alpha clip masks
The cost of switching render targets on each draw to make a custom
clip is enormous. There are virtually no circumstances where this will
outperform our cached, multi-threaded software mask generator. The
tried-and-true approach to clipping on-GPU is with analytic FPs. And
now that we support CCPR clip FPs, there ulitmately should be very few
clip stacks that even require a mask as long as they don't use
deprecated SkClipOps.

Bug: skia:
Change-Id: I79c5558c93c1b99179f1e933d029f69b14ad1ce3
Reviewed-on: https://skia-review.googlesource.com/116724
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Chris Dalton <csmartdalton@google.com>
2018-03-28 15:01:04 +00:00
Jim Van Verth
41964ed46e Rename SkInsetConvexPolygon to SkOffsetPolygon.
Prep for adding new offset routines.

Change-Id: I261c22d9998e5ae4567b697c5f20a31f20777ac1
Reviewed-on: https://skia-review.googlesource.com/116800
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Jim Van Verth <jvanverth@google.com>
2018-03-28 14:49:14 +00:00
Robert Phillips
3a9710b1e4 Add handling for allocation failure in GrTextureOpList
I believe this is what has been causing the intermittent TSAN crash

Change-Id: Ie7170a3eb0495cebe8b1398a00f98ad099f78944
Reviewed-on: https://skia-review.googlesource.com/116629
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
2018-03-28 14:25:54 +00:00
angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com
6cf20044d8 Roll skia/third_party/externals/angle2/ e858cb1db..c7575da54 (1 commit)
e858cb1db3..c7575da546

$ git log e858cb1db..c7575da54 --date=short --no-merges --format='%ad %ae %s'
2018-03-27 fjhenigman Skip ReuploadingClearsTexture test on Mac.

Created with:
  roll-dep skia/third_party/externals/angle2


The AutoRoll server is located here: https://angle-skia-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=skia.primary:Perf-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUC5i7RYH-GPU-IntelIris6100-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE,Test-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE,Test-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE,Test-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE,Test-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE,Build-Debian9-Clang-x86_64-Release-ANGLE
TBR=mtklein@google.com

Change-Id: I269e6e7ef638646fe1921ce36ea0998f1b2846aa
Reviewed-on: https://skia-review.googlesource.com/116760
Reviewed-by: <angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: <angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
2018-03-28 07:09:44 +00:00
skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com
13e3ddd069 Roll skia/third_party/externals/skcms/ 80ac04dc4..fc0d0c4c7 (1 commit)
https://skia.googlesource.com/skcms/+log/80ac04dc4706..fc0d0c4c7f9c

$ git log 80ac04dc4..fc0d0c4c7 --date=short --no-merges --format='%ad %ae %s'
2018-03-27 mtklein force inline where feasible

Created with:
  roll-dep skia/third_party/externals/skcms


The AutoRoll server is located here: https://skcms-skia-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=mtklein@google.com

Change-Id: I46629965c100feba21f18404437bf12badf07aa5
Reviewed-on: https://skia-review.googlesource.com/116730
Commit-Queue: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Reviewed-by: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
2018-03-27 23:06:24 +00:00
angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com
d7ee596a17 Roll skia/third_party/externals/angle2/ 15c7d0a64..e858cb1db (12 commits)
15c7d0a649..e858cb1db3

$ git log 15c7d0a64..e858cb1db --date=short --no-merges --format='%ad %ae %s'
2018-03-27 jmadill Split VAO dirty bits to speed iteration.
2018-03-27 courtneygo Update license and readme for use downstream
2018-03-22 lucferron dEQP: Enable depth/stencil backbuffer.
2018-03-27 fjhenigman Vulkan: flush StreamingBuffer before unmap.
2018-03-21 lucferron Vulkan: Reallocate only the right uniform descriptor set when needed
2018-03-21 lfy GLES1: glAlphaFunc
2018-03-15 lucferron Vulkan: Dynamic update of uniforms
2018-03-27 lucferron Revert "Vulkan: Enable depth and stencil dEQP tests"
2018-03-27 jmadill Buffer11: Refactor Subject/Observer pattern.
2018-03-27 jmadill D3D11: Add primitive type dirty bits.
2018-03-16 jiajia.qin ES31: Add DispatchCompute perf tests
2018-03-21 jiawei.shao Use packed enums on shader types in ANGLE renderer

Created with:
  roll-dep skia/third_party/externals/angle2


The AutoRoll server is located here: https://angle-skia-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=skia.primary:Perf-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUC5i7RYH-GPU-IntelIris6100-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE,Test-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE,Test-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE,Test-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE,Test-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE,Build-Debian9-Clang-x86_64-Release-ANGLE
TBR=mtklein@google.com

Change-Id: I0659b33b9361e42222755feebd0785d1e230702c
Reviewed-on: https://skia-review.googlesource.com/116727
Reviewed-by: <angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: <angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
2018-03-27 22:31:23 +00:00
Herb Derby
39e45df22d Various cleanup to remote_demo
Integrate SkRemoteGlyphCacheRenderer into SkStrikeServer
Add a Timer.
Remove unused code.

BUG=skia:7515

Change-Id: Idffb477af71cbcc4035df190e29e8910b61aa6e5
Reviewed-on: https://skia-review.googlesource.com/116485
Reviewed-by: Herb Derby <herb@google.com>
Commit-Queue: Herb Derby <herb@google.com>
2018-03-27 21:51:44 +00:00
skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com
d24c325452 Roll skia/third_party/externals/skcms/ 6793c6ab0..80ac04dc4 (2 commits)
https://skia.googlesource.com/skcms/+log/6793c6ab038e..80ac04dc4706

$ git log 6793c6ab0..80ac04dc4 --date=short --no-merges --format='%ad %ae %s'
2018-03-27 mtklein split gather functions
2018-03-27 brianosman Include XYZ encoding factor in the mAB matrix if PCS is XYZ

Created with:
  roll-dep skia/third_party/externals/skcms


The AutoRoll server is located here: https://skcms-skia-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=mtklein@google.com

Change-Id: Ie13faa897fed25352bcc290bb903251ee46ca2eb
Reviewed-on: https://skia-review.googlesource.com/116723
Reviewed-by: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
2018-03-27 21:41:24 +00:00
Robert Phillips
b7fafba851 Reland "Switch bots to use ViaDDL (i.e., ddl-gl) instead of old --ddl flag"
This reverts commit 7a11abd4f0.

Reason for revert: Fixes have landed

Original change's description:
> Revert "Switch bots to use ViaDDL (i.e., ddl-gl) instead of old --ddl flag"
> 
> This reverts commit d234c9e6b9.
> 
> Reason for revert: GM/TSAN unhappiness
> 
> Original change's description:
> > Switch bots to use ViaDDL (i.e., ddl-gl) instead of old --ddl flag
> > 
> > Change-Id: I9c5f27bacf7d9fa7a9ac123379e2556913810cba
> > Reviewed-on: https://skia-review.googlesource.com/116464
> > Reviewed-by: Eric Boren <borenet@google.com>
> > Commit-Queue: Robert Phillips <robertphillips@google.com>
> 
> TBR=borenet@google.com,robertphillips@google.com
> 
> Change-Id: I95388ee582bdd9561b8ebf24250f58b361657b7c
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Reviewed-on: https://skia-review.googlesource.com/116621
> Reviewed-by: Robert Phillips <robertphillips@google.com>
> Commit-Queue: Robert Phillips <robertphillips@google.com>

TBR=borenet@google.com,robertphillips@google.com

Change-Id: Idcd8cc2337657131fb9b1785bec44afefacda634
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://skia-review.googlesource.com/116681
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
2018-03-27 20:49:34 +00:00
skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com
f42e548d91 Roll skia/third_party/externals/skcms/ aa8c0f3d2..6793c6ab0 (1 commit)
https://skia.googlesource.com/skcms/+log/aa8c0f3d24e7..6793c6ab038e

$ git log aa8c0f3d2..6793c6ab0 --date=short --no-merges --format='%ad %ae %s'
2018-03-27 mtklein prefer A2B0 over A2B1

Created with:
  roll-dep skia/third_party/externals/skcms


The AutoRoll server is located here: https://skcms-skia-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=mtklein@google.com

Change-Id: I61343efbfb8ea936d49ee5d0cecdfd1a9f7de5a4
Reviewed-on: https://skia-review.googlesource.com/116700
Reviewed-by: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
2018-03-27 19:50:23 +00:00
Ethan Nicholas
26a9aad63b initial SkSLJIT checkin
Docs-Preview: https://skia.org/?cl=112204
Bug: skia:
Change-Id: I10042a0200db00bd8ff8078467c409b1cf191f50
Reviewed-on: https://skia-review.googlesource.com/112204
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
Reviewed-by: Mike Klein <mtklein@chromium.org>
2018-03-27 18:39:13 +00:00
Ben Wagner
3560b58de3 Revert "Add color fonts."
This reverts commit d3b933f3e0.

Reason for revert: we're not mirroring experimental/ everywhere.

Original change's description:
> Add color fonts.
> 
> BUG=skia:7624
> 
> Change-Id: I1d0974282adce2fd7b13a003309e63593b6e1a9c
> Reviewed-on: https://skia-review.googlesource.com/109521
> Commit-Queue: Ben Wagner <bungeman@google.com>
> Reviewed-by: Mike Klein <mtklein@google.com>
> Reviewed-by: Herb Derby <herb@google.com>
> Reviewed-on: https://skia-review.googlesource.com/116541

TBR=mtklein@google.com,bungeman@google.com,herb@google.com

Change-Id: Ic2d763244cbce663a23eb53321ac45201c466501
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:7624
Reviewed-on: https://skia-review.googlesource.com/116680
Reviewed-by: Ben Wagner <bungeman@google.com>
Commit-Queue: Ben Wagner <bungeman@google.com>
2018-03-27 18:16:04 +00:00
Mike Klein
83ee463c31 exclude SkColorSpaceXform_skcms.cpp from Google3
We're globbing it up right now, but can't build without skcms.h.

Change-Id: I2a9b29b7a91a0c8cccc73b1e20f64c1f6181eabd
Reviewed-on: https://skia-review.googlesource.com/116623
Reviewed-by: Brian Osman <brianosman@google.com>
2018-03-27 17:25:23 +00:00
skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com
68a1515c08 Roll skia/third_party/externals/skcms/ 07da72a48..aa8c0f3d2 (1 commit)
https://skia.googlesource.com/skcms/+log/07da72a48a72..aa8c0f3d24e7

$ git log 07da72a48..aa8c0f3d2 --date=short --no-merges --format='%ad %ae %s'
2018-03-27 mtklein gather directly into vectors

Created with:
  roll-dep skia/third_party/externals/skcms


The AutoRoll server is located here: https://skcms-skia-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=mtklein@google.com

Change-Id: I270fc88a3879d021460f23c44a9d561539884635
Reviewed-on: https://skia-review.googlesource.com/116660
Reviewed-by: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
2018-03-27 17:17:24 +00:00
Ben Wagner
d3b933f3e0 Add color fonts.
BUG=skia:7624

Change-Id: I1d0974282adce2fd7b13a003309e63593b6e1a9c
Reviewed-on: https://skia-review.googlesource.com/109521
Commit-Queue: Ben Wagner <bungeman@google.com>
Reviewed-by: Mike Klein <mtklein@google.com>
Reviewed-by: Herb Derby <herb@google.com>
Reviewed-on: https://skia-review.googlesource.com/116541
2018-03-27 16:49:14 +00:00
Ben Wagner
6ce482aed4 Remove include/svg from public.bzl includes.
It's strange to include it since public.bzl currently excludes all of
the source files that define the symbols declared in the headers in this
directory.

This also fixes the two files which needed to put some of these includes
behind the SK_XML macro. The public.bzl never defines the SK_XML macro,
so there is no need to ever have the include/svg directory on the
include path, even for DM.

Change-Id: I6cc18908aa16cfc914ed9b7ab174d03a0a242aa4
Reviewed-on: https://skia-review.googlesource.com/116547
Commit-Queue: Ben Wagner <bungeman@google.com>
Reviewed-by: Mike Klein <mtklein@google.com>
2018-03-27 16:36:29 +00:00
skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com
f80c8ecdaa Roll skia/third_party/externals/skcms/ 7e5292bb2..07da72a48 (1 commit)
https://skia.googlesource.com/skcms/+log/7e5292bb229e..07da72a48a72

$ git log 7e5292bb2..07da72a48 --date=short --no-merges --format='%ad %ae %s'
2018-03-27 mtklein Don't use toXZYD50 matrix for A2B sources.

Created with:
  roll-dep skia/third_party/externals/skcms


The AutoRoll server is located here: https://skcms-skia-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=mtklein@google.com

Change-Id: I472d3f3047dbf566c67f81f5ff7afc53397ac5c7
Reviewed-on: https://skia-review.googlesource.com/116615
Reviewed-by: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
2018-03-27 16:35:44 +00:00
Brian Osman
5d7e1cf98f Remove duplicate entry for SkTDPQueue.h in gni
This was causing the generated MSVC project to not load, leading to
confusing behavior in the IDE.

Change-Id: I7ec8f1391c0c3b53e7b5c473571a062e6406b358
Reviewed-on: https://skia-review.googlesource.com/116467
Commit-Queue: Brian Osman <brianosman@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>
Reviewed-by: Mike Klein <mtklein@google.com>
2018-03-27 16:31:24 +00:00
Mike Klein
b412f5bd06 cache SkMatrix44 type in makeColorSpin()
This helps prevent races on getType() later, e.g.
https://luci-milo.appspot.com/swarming/task/3c80715df637a810/steps/symbolized_dm/0/stdout

Change-Id: I3dbc9f6bf2fddff9ca416b0001bfd5da123e6b25
Reviewed-on: https://skia-review.googlesource.com/116466
Commit-Queue: Mike Klein <mtklein@chromium.org>
Commit-Queue: Brian Osman <brianosman@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
2018-03-27 16:29:14 +00:00
Robert Phillips
cc47335ded Fix flushing of context in ViaDDL
Change-Id: I9f84f4081861e2789b45787405e0f1a5aa87824d
Reviewed-on: https://skia-review.googlesource.com/116546
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
2018-03-27 16:02:44 +00:00
Chris Dalton
8dfc70f7e2 ccpr: Various cleanups
Renames GrCCCoverageProcessor::RenderPass to PrimitiveType and handles
corners as subpasses instead. Various touchups to coverage processors
now that the overhaul is complete. This change should be strictly a
refactor.

Bug: skia:
Change-Id: I52852463330d5ec71fae7e19fadccd9ede8b2c16
Reviewed-on: https://skia-review.googlesource.com/116169
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Chris Dalton <csmartdalton@google.com>
2018-03-27 15:38:34 +00:00
skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com
9b685238f6 Roll skia/third_party/externals/skcms/ 286108ae1..7e5292bb2 (1 commit)
https://skia.googlesource.com/skcms/+log/286108ae15ae..7e5292bb229e

$ git log 286108ae1..7e5292bb2 --date=short --no-merges --format='%ad %ae %s'
2018-03-27 mtklein invert inverse-CMYK to CMYK

Created with:
  roll-dep skia/third_party/externals/skcms


The AutoRoll server is located here: https://skcms-skia-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=mtklein@google.com

Change-Id: Id6e7de8b155973447ff1fc582530c416f7c6a3d5
Reviewed-on: https://skia-review.googlesource.com/116609
Reviewed-by: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
2018-03-27 14:56:24 +00:00
Brian Osman
052ef69570 Pin color type when a cross-context image is constructed
Fixes issues with gray images that may be incorrectly re-interpreted as
alpha when re-wrapped. (https://github.com/flutter/flutter/issues/15600)

Change-Id: I4a78466073e14d212108d168eed0b2df1bc92ffe
Reviewed-on: https://skia-review.googlesource.com/116484
Commit-Queue: Brian Osman <brianosman@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>
2018-03-27 14:45:14 +00:00
Ben Wagner
f830443814 Update doc for connecting to GCE bots.
No-Try: true
Docs-Preview: https://skia.org/?cl=116544
Change-Id: Ia10e638513907efa368411dfb7c8d90247a4ffee
Reviewed-on: https://skia-review.googlesource.com/116544
Reviewed-by: Ravi Mistry <rmistry@google.com>
Reviewed-by: Eric Boren <borenet@google.com>
Commit-Queue: Ben Wagner <benjaminwagner@google.com>
2018-03-27 14:44:24 +00:00
Robert Phillips
d3b37a1a7c Fix TSAN race in SkPaintToGrPaintNoShader
Change-Id: Id906e139db80e0c3b426a0e2318e77477989cf8d
Reviewed-on: https://skia-review.googlesource.com/116545
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
2018-03-27 14:33:54 +00:00
Robert Phillips
7a11abd4f0 Revert "Switch bots to use ViaDDL (i.e., ddl-gl) instead of old --ddl flag"
This reverts commit d234c9e6b9.

Reason for revert: GM/TSAN unhappiness

Original change's description:
> Switch bots to use ViaDDL (i.e., ddl-gl) instead of old --ddl flag
> 
> Change-Id: I9c5f27bacf7d9fa7a9ac123379e2556913810cba
> Reviewed-on: https://skia-review.googlesource.com/116464
> Reviewed-by: Eric Boren <borenet@google.com>
> Commit-Queue: Robert Phillips <robertphillips@google.com>

TBR=borenet@google.com,robertphillips@google.com

Change-Id: I95388ee582bdd9561b8ebf24250f58b361657b7c
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://skia-review.googlesource.com/116621
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
2018-03-27 14:03:31 +00:00
angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com
6671a7b81c Roll skia/third_party/externals/angle2/ 84fdc62c6..15c7d0a64 (1 commit)
84fdc62c61..15c7d0a649

$ git log 84fdc62c6..15c7d0a64 --date=short --no-merges --format='%ad %ae %s'
2018-03-27 lucferron Vulkan: Enable depth and stencil dEQP tests

Created with:
  roll-dep skia/third_party/externals/angle2


The AutoRoll server is located here: https://angle-skia-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=skia.primary:Perf-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUC5i7RYH-GPU-IntelIris6100-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE,Test-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE,Test-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE,Test-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE,Test-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE,Build-Debian9-Clang-x86_64-Release-ANGLE
TBR=mtklein@google.com

Change-Id: I947cdd97a90b276cb0c53fc6d47e3a43528d4746
Reviewed-on: https://skia-review.googlesource.com/116603
Commit-Queue: <angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Reviewed-by: <angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
2018-03-27 13:47:24 +00:00
Kaloyan Donev
dfffb39ca8 Fixed warnings for casting large type to smaller one.
Change-Id: Ib3e2661d0383bf154bc9178dac070dfd910a393c
Reviewed-on: https://skia-review.googlesource.com/115200
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>
2018-03-27 13:28:24 +00:00
Robert Phillips
d234c9e6b9 Switch bots to use ViaDDL (i.e., ddl-gl) instead of old --ddl flag
Change-Id: I9c5f27bacf7d9fa7a9ac123379e2556913810cba
Reviewed-on: https://skia-review.googlesource.com/116464
Reviewed-by: Eric Boren <borenet@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
2018-03-27 13:23:04 +00:00
Brian Osman
094fba9cd6 Add SkColorSpaceXform_skcms
Currently only enabled in Skia dev builds. Has some diffs
in GMs and images, but (hopefully) nothing major.

Bug: skia:
Change-Id: Ifdf5d2804e59f555a3dc84f657e438dd589a2751
Reviewed-on: https://skia-review.googlesource.com/116520
Commit-Queue: Mike Klein <mtklein@google.com>
Reviewed-by: Mike Klein <mtklein@chromium.org>
2018-03-27 13:15:24 +00:00
Yuqian Li
c26be9c50f Reland "Use DAA for small cubics and non-convex paths that fit into a mask"
This reverts commit 107d53971d.

Reason for revert: Ready to rebaseline

Original change's description:
> Revert "Use DAA for small cubics and non-convex paths that fit into a mask"
> 
> This reverts commit 1875e05384.
> 
> Reason for revert:
> 
> I don't think there's anything wrong with this, but it looks like Yuqian is out today and there is a large number of GM, SKP, and SVG images to triage from this.  This is just a triage-by-revert... should be fine to reland when you're ready to triage.
> 
> Original change's description:
> > Use DAA for small cubics and non-convex paths that fit into a mask
> > 
> > I forgot to benchmark svgs and it turns out that DAA is specifically
> > good for the small cubics and small non-convex paths in svgs. This
> > should make our svg performance fast again:
> > 
> >     2.84% faster in svgparse_Florida-StateSeal.svg_1
> >     2.90% faster in svgparse_NewYork-StateSeal.svg_1
> >     2.95% faster in svgparse_Seal_of_Texas.svg_1
> >     3.05% faster in car.svg_1
> >     3.53% faster in svgparse_Vermont_state_seal.svg_1
> >     3.68% faster in svgparse_Wyoming-StateSeal.svg_1
> >     4.88% faster in svgparse_Minnesota-StateSeal.svg_1
> >     5.22% faster in svgparse_NewMexico-StateSeal.svg_1
> >     6.49% faster in svgparse_fsm.svg_1
> > 
> > 
> > Bug: skia:
> > Change-Id: Ia149944443d72c12c3dda178cb5ebc89d6d0bf18
> > Reviewed-on: https://skia-review.googlesource.com/116185
> > Reviewed-by: Cary Clark <caryclark@google.com>
> > Commit-Queue: Yuqian Li <liyuqian@google.com>
> 
> TBR=caryclark@google.com,liyuqian@google.com,reed@google.com,caryclark@skia.org
> 
> # Not skipping CQ checks because original CL landed > 1 day ago.
> 
> Bug: skia:
> Change-Id: I232f34dcea1cdabef768879a261fe6796f3e4a79
> Reviewed-on: https://skia-review.googlesource.com/116400
> Reviewed-by: Mike Klein <mtklein@google.com>
> Commit-Queue: Mike Klein <mtklein@google.com>

TBR=mtklein@google.com,caryclark@google.com,liyuqian@google.com,reed@google.com,caryclark@skia.org

Change-Id: I6a413e3a2f1ce9182f9e209f6e2654a602170378
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:
Reviewed-on: https://skia-review.googlesource.com/116620
Commit-Queue: Yuqian Li <liyuqian@google.com>
Reviewed-by: Yuqian Li <liyuqian@google.com>
2018-03-27 12:41:05 +00:00
Robert Phillips
33f02edb14 Switch DDL rendering to be a Via in DM (take 2)
This will let us also render the GMs via DDLs.

TBR=mtklein@google.com
Change-Id: If7c2460d964822a6decc33cf5e8e685e67923127
Reviewed-on: https://skia-review.googlesource.com/116463
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
2018-03-27 12:37:44 +00:00
Herb Derby
2ef4525daf Remove SkAutoGlyphCache
Change-Id: Ia485ce5acd70730d461ce81104595a6b961f5ca6
Reviewed-on: https://skia-review.googlesource.com/116183
Reviewed-by: Ben Wagner <bungeman@google.com>
Commit-Queue: Herb Derby <herb@google.com>
2018-03-27 02:58:44 +00:00
angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com
32ccf165ae Roll skia/third_party/externals/angle2/ a571f28d4..84fdc62c6 (6 commits)
a571f28d40..84fdc62c61

$ git log a571f28d4..84fdc62c6 --date=short --no-merges --format='%ad %ae %s'
2018-03-19 jiawei.shao Fix a compile error when ANGLE_D3D9EX == ANGLE_DISABLED
2017-10-03 geofflang Implement robust resource initialization for OpenGL.
2018-03-20 oetuaho Clean up checkCanBeLValue
2018-01-27 kbr Update stencil validation rules for WebGL
2018-03-19 oetuaho Rely on hash to check for some mangled name matches
2018-03-23 oetuaho Clarify aliasing rules in CHROMIUM_bind_uniform_location

Created with:
  roll-dep skia/third_party/externals/angle2


The AutoRoll server is located here: https://angle-skia-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=skia.primary:Perf-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUC5i7RYH-GPU-IntelIris6100-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE,Test-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE,Test-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE,Test-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE,Test-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE,Build-Debian9-Clang-x86_64-Release-ANGLE
TBR=mtklein@google.com

Change-Id: I55d5e7111b00110f0c671e559f4e2f6525cd341f
Reviewed-on: https://skia-review.googlesource.com/116584
Commit-Queue: <angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Reviewed-by: <angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
2018-03-27 02:04:24 +00:00
Ben Wagner
e579f1e70f [Bazel] Add missing include dir.
No-Try: true
Change-Id: If27fb8532d186f96d5cf15f4099113713319693d
Reviewed-on: https://skia-review.googlesource.com/116525
Reviewed-by: Ben Wagner <benjaminwagner@google.com>
Commit-Queue: Ben Wagner <benjaminwagner@google.com>
2018-03-26 22:44:34 +00:00
skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com
132d039956 Roll skia/third_party/externals/skcms/ db50cadbb..286108ae1 (1 commit)
https://skia.googlesource.com/skcms/+log/db50cadbbc70..286108ae15ae

$ git log db50cadbb..286108ae1 --date=short --no-merges --format='%ad %ae %s'
2018-03-26 mtklein refactor source premul handling for A2B

Created with:
  roll-dep skia/third_party/externals/skcms


The AutoRoll server is located here: https://skcms-skia-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=mtklein@google.com

Change-Id: Iaa431c7b1ad978b408f324514e87f233276b6898
Reviewed-on: https://skia-review.googlesource.com/116515
Commit-Queue: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Reviewed-by: skcms-skia-autoroll <skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
2018-03-26 22:02:44 +00:00
Chris Dalton
b709f27bc0 ccpr: Use Manhattan distance for curve AA
Manhattan distance has the nice property that every pixel crossed
anywhere by the curve gets a fractional coverage value, whereas pixels
not touched by the curve will have zero coverage. This arguably looks
better on its own merits, but it also helps the curves blend more
seamlessly with their adjoining edges, which are already using Manhattan
distance.

Bug: skia:
Change-Id: I31cf28171d0b74512c74dca1088e50f0f442b924
Reviewed-on: https://skia-review.googlesource.com/115781
Commit-Queue: Chris Dalton <csmartdalton@google.com>
Reviewed-by: Jim Van Verth <jvanverth@google.com>
2018-03-26 21:58:44 +00:00