aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* Remove SkImageFilter::MakeBlur entry pointGravatar Robert Phillips2017-06-27
| | | | | | | | | AFAICT none of our clients use this entry point and it is tangling up efforts to add a new parameter to the BlurImageFilter. Change-Id: I494634db98a1d246854a5e3735380fbe55f425c2 Reviewed-on: https://skia-review.googlesource.com/20837 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
* Add GM image test for blur with clamp mode.Gravatar wutao2017-06-27
| | | | | | | | | | | | | Adding a GM (Golden Master) image test in Skia to reproduce the bleed black issue in Chrome. This would allow regressions to be caught in Skia's status page before being rolled into Chrome. Bug: 622128 Change-Id: Ifd2824fff59483c8e4be48392ba467414d41ca13 TEST=imageblurclampmode.cpp Reviewed-on: https://skia-review.googlesource.com/20778 Commit-Queue: Robert Phillips <robertphillips@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com>
* attempt to fix android rollGravatar Mike Reed2017-06-27
| | | | | | | | Bug: skia: Change-Id: I78147d215b752b6622a3c96508a6f21bf0284bb8 Reviewed-on: https://skia-review.googlesource.com/20907 Commit-Queue: Mike Reed <reed@google.com> Reviewed-by: Mike Reed <reed@google.com>
* run NexusPlayer CPU tests verboselyGravatar Mike Klein2017-06-26
| | | | | | | | | | | | These were relying on a 15-minute status thread to keep them from timing out. --verbose ought to do the same. CQ_INCLUDE_TRYBOTS=skia.primary:Test-Android-Clang-NexusPlayer-CPU-Moorefield-x86-Debug-Android Change-Id: I1d1467a5e777df6ec5e79d2d9ea992e69bfaeeb7 Reviewed-on: https://skia-review.googlesource.com/20821 Reviewed-by: Mike Klein <mtklein@chromium.org> Commit-Queue: Mike Klein <mtklein@chromium.org>
* somewhat less silly tail loads and storesGravatar Mike Klein2017-06-26
| | | | | | | | | | | | | | | | | | | | No reason to keep going one at a time when we know there are generally better ways to handle loading a power-of-two number of low lanes. This strategy scales up too, with quick answers for 8 (one 8 byte load), 12 (one 8 byte, one 4 byte), etc. $ ninja -C out monobench; and out/monobench SkRasterPipeline_compile 300 Before: 46.946ns After: 43.341ns (This happens to be _lowp. Expect similar small speedups elsewhere.) Change-Id: I08f87769ea3c9f06ad13d2b1d5326e542b9b63a8 Reviewed-on: https://skia-review.googlesource.com/20903 Reviewed-by: Herb Derby <herb@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
* Restrict acceptable bounds for uploading data to mip levels on GPUGravatar Greg Daniel2017-06-26
| | | | | | | | | | | | | | The help simplify the code (and currect a current bug) inside the uploadPixel functions in GL and Vulkan, we not restrict the bounds upload/write bounds on a higher level. For non mipped uploads, we requiring the bounds to be within the bounds on the texture, and for mipped uploads we require the bounds to be the full texture. Bug: skia:6780 Change-Id: Ia1cff3ee48edd3676afce6265d60e81ccea6ca6a Reviewed-on: https://skia-review.googlesource.com/20825 Reviewed-by: Jim Van Verth <jvanverth@google.com> Commit-Queue: Greg Daniel <egdaniel@google.com>
* Roll recipe dependencies (trivial).Gravatar recipe-roller2017-06-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug. depot_tools: https://crrev.com/c9603f57350caa7f43e7fc5173c56194e81eacaf gclient flatten: add support for deps_os (phajdan.jr@chromium.org) https://crrev.com/a89d443c2eaa4d2f38ce52846dd96183ef5107d4 gclient: remove hookinfo command (phajdan.jr@chromium.org) https://crrev.com/c10a4d88a42c01d1eea007248f29bae002c01b25 gclient: extract Hook class (phajdan.jr@chromium.org) https://crrev.com/c93643954df2265d7d3febd9d6509238fffb45b4 gclient: add cwd support for hooks (phajdan.jr@chromium.org) https://crrev.com/f79f0da8c68eee72de38f5d0fdbe5faa4ea044f0 gclient: remove unused ast code (phajdan.jr@chromium.org) https://crrev.com/1c5c118cfd533f2242f95aafbf9aa275031e535a Clean up a confusing gclient_utils.py message (brucedawson@chromium.org) https://crrev.com/c7f8791a0dde00b091dd6c924dd0ec1817b5076c [led] update to a05c9b0865abbfa27673528c5d95c80b55ff87bb (iannucci@chromium.org) https://crrev.com/78753da20ee812116f2cec5daed8a22b20e66352 Accept URLs in 'git cl issue' (agable@chromium.org) https://crrev.com/61373ee4bcaf9c875536cd877547725939f7d9cf [led] update led to a3890a6e0b276b35d734b13a82b164dd7fd8f2b5. (iannucci@chromium.org) https://crrev.com/44398faeeddeed1a91f2d3752afd1ed1fa2c81fc [vpython] Bump version. (dnj@chromium.org) https://crrev.com/9df9e9f8f9b218862b41817dd7604251484cc884 Dump reviewers set when failing _check_reviewers (gab@chromium.org) https://crrev.com/62619a38110f1a49f7ed40128f843cf4bf803dc1 git-cl-patch: Return to cherry-pick, but hard reset with --force (agable@chromium.org) https://crrev.com/257c9b2a70300faf1fa03f66f1b53176a0fe5c11 Use extended path syntax on windows when extracting tarball in download script (ricow@google.com) https://crrev.com/66945379c69dca334b61ebb4728b4a6ee95fde79 gclient: throw errors if values from deps_os override deps (phajdan.jr@chromium.org) https://crrev.com/4b73b02b36238fe6c8e71b647b0e1367c9ead9cc gclient: remove unused depth_first_tree (phajdan.jr@chromium.org) https://crrev.com/dce2250d14923170c281295588e3e1ac103d6a47 Presubmit: Treat changes with Gerrit-style Tbr: header as TBRed. (jbroman@chromium.org) https://crrev.com/529d6a4e4aed18b48a8fc0e9351a5d4a07ca597d gclient: include deps_os entries in dependencies (phajdan.jr@chromium.org) https://crrev.com/f1587bf5a9b8dcf8af47156dbcb553291150b0af gclient: remove unused "use strict" logic (phajdan.jr@chromium.org) https://crrev.com/806b70187d8bff7c51fb3bad730b189b505ed937 Revert "gclient: include deps_os entries in dependencies" (tandrii@chromium.org) https://crrev.com/fd0057e42be2906e79060085201494e7df1d0e8b gclient: include deps_os entries in dependencies (reland #1) (phajdan.jr@chromium.org) https://crrev.com/9756104b69447e418b55732e737f605cb1786069 Document drover process for re-landing a CL in a release branch. (agable@chromium.org) https://crrev.com/b983ac1ae28165b738aa77ad56103312f0e0fc58 Update VC++ packaging script to not package vctip.exe (brucedawson@chromium.org) https://crrev.com/032d54594781e12361cb5e644204cfb786cd3a16 gclient: implement conditions for hooks (phajdan.jr@chromium.org) https://crrev.com/1306bd03c96bf8b36b68f19f86d656e24c959df6 Support dumping json to stdout. (szager@chromium.org) https://crrev.com/7625d880d7a0f2e272dd1c9a7e9fba0de8885497 git-cl-description: use atomic edit API (agable@chromium.org) https://crrev.com/ad30de63893abcb18f4cc1d031dcc5096723fedf gclient flatten: syntax and schema fixes from actual testing (phajdan.jr@chromium.org) recipe_engine: https://crrev.com/8c0fd220f89fdee4865b016def85a4ad8d04fec5 Add experimental downstream trybots (phajdan.jr@chromium.org) TBR=borenet@google.com Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Change-Id: I1f4cf28b0dc5fee467f21f797c034c5fef8587d0 Reviewed-on: https://skia-review.googlesource.com/20901 Reviewed-by: Recipe Roller <recipe-roller@chromium.org> Commit-Queue: Recipe Roller <recipe-roller@chromium.org>
* Exclude AALinearizingConvexPathRenderer from zero size pathsGravatar Brian Osman2017-06-26
| | | | | | | | | | | | | Per SVG, stroking a zero length line with square or round caps should draw a square or circle. This path renderer has lots of baked in assumptions that prevent that behavior, so the easy solution is to let some other path renderer handle it. Bug: skia:6781 Change-Id: I9d2da94f75d96554e3cd218848aa552f63add679 Reviewed-on: https://skia-review.googlesource.com/20900 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
* remove unneeded guards for androidGravatar Mike Reed2017-06-26
| | | | | | | Bug: skia: Change-Id: I4c0116a9ec27a0c0aa853ce1585d457c7c485b67 Reviewed-on: https://skia-review.googlesource.com/20832 Reviewed-by: Mike Reed <reed@google.com>
* always touch verbose.log, even if --verboseGravatar Mike Klein2017-06-26
| | | | | | | | | | | | This lets us run bots that upload images with --verbose. Today we're getting away with --verbose on some bots because they happen to skip upload. Change-Id: If915a65987b43c945263ada7751bf36aedff605a Reviewed-on: https://skia-review.googlesource.com/20834 Reviewed-by: Ben Wagner <benjaminwagner@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
* Move threaded BMP device to samplesGravatar Brian Osman2017-06-26
| | | | | | | | | | | | Currently only referenced by SampleApp, and this squelches false positive warnings from Clang's thread safety analysis when clients build this code. Bug: skia: Change-Id: I710fdc882e05fc5b80977139237028d1408f17db Reviewed-on: https://skia-review.googlesource.com/20831 Commit-Queue: Brian Osman <brianosman@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org> Reviewed-by: Mike Klein <mtklein@chromium.org>
* lean more on the compiler in lowp stagesGravatar Mike Klein2017-06-26
| | | | | | | | | | | | | | This refactors {from,to}_{byte,8888} to lean a bit more on the compiler, and to share code between the two. The algorithm is not exactly the same, but it's comparable, and the results of course are identical. This new algorithm is a lot easier to generalize to AVX2, and parallels the full-precision {from,to}_{byte,8888} functions in _stages.cpp. Change-Id: I31ea90d65967bf4ede2497d1e2197cb0e7648bf8 Reviewed-on: https://skia-review.googlesource.com/20828 Reviewed-by: Herb Derby <herb@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
* Re-enable opList dependency trackingGravatar Robert Phillips2017-06-26
| | | | | | | Change-Id: I0fa1bbaed565a0d7de8d4475ed1d2baf200b285f Reviewed-on: https://skia-review.googlesource.com/20700 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
* Remove SkImage_Base::peekTexture callGravatar Robert Phillips2017-06-26
| | | | | | | Change-Id: Ie5b6bec8df083e0a128aa234d80f5a214cd98f0d Reviewed-on: https://skia-review.googlesource.com/20741 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
* Temporarily stop uploading to partner bucket in RecreateSKPs botGravatar Ravi Mistry2017-06-26
| | | | | | | | Bug: skia:6797 Change-Id: I812837d02b90b78b6df3980649ccf6362e939afc Reviewed-on: https://skia-review.googlesource.com/20820 Commit-Queue: Ravi Mistry <rmistry@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Revert "Remove deadcode for sRGB image shading."Gravatar Mike Reed2017-06-26
| | | | | | | | | | | | | | | | | | | | | | | This reverts commit 51f6a0f89e55b2c0f102aa7963320a23c586e71d. Reason for revert: try to fix chrome roll Original change's description: > Remove deadcode for sRGB image shading. > > Change-Id: I9d98da67d97b64ab55cf44b9ae447882dccda1ca > Reviewed-on: https://skia-review.googlesource.com/20695 > Reviewed-by: Mike Klein <mtklein@chromium.org> > Commit-Queue: Herb Derby <herb@google.com> TBR=mtklein@chromium.org,mtklein@google.com,herb@google.com,reed@google.com # Not skipping CQ checks because original CL landed > 1 day ago. Change-Id: I90409a08409177d9dc3b094d736af0fdaf7d6cdd Reviewed-on: https://skia-review.googlesource.com/20824 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com>
* Revert "remove a bit more dead code"Gravatar Mike Reed2017-06-26
| | | | | | | | | | | | | | | | | | | | | | | This reverts commit d9b1fe02a677ec44bc1c99809f3ee7eb08708137. Reason for revert: try to fix chrome roll Original change's description: > remove a bit more dead code > > Change-Id: I61484672e88d6bb4f75833ee89e7178c4f34d610 > Reviewed-on: https://skia-review.googlesource.com/20780 > Commit-Queue: Mike Klein <mtklein@google.com> > Reviewed-by: Herb Derby <herb@google.com> TBR=mtklein@chromium.org,mtklein@google.com,herb@google.com # Not skipping CQ checks because original CL landed > 1 day ago. Change-Id: I03dcd344dfb138261d9421b0692d12e4ed431100 Reviewed-on: https://skia-review.googlesource.com/20822 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com>
* Revert "Revert "Remove deadcode for sRGB image shading.""Gravatar Mike Reed2017-06-25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit e08e639063dd5476b5a1a24c0f0dc8f1139bfd8d. Reason for revert: revert was not clean Original change's description: > Revert "Remove deadcode for sRGB image shading." > > This reverts commit 51f6a0f89e55b2c0f102aa7963320a23c586e71d. > > Reason for revert: crazy attempt to fix chrome roll > > Original change's description: > > Remove deadcode for sRGB image shading. > > > > Change-Id: I9d98da67d97b64ab55cf44b9ae447882dccda1ca > > Reviewed-on: https://skia-review.googlesource.com/20695 > > Reviewed-by: Mike Klein <mtklein@chromium.org> > > Commit-Queue: Herb Derby <herb@google.com> > > TBR=mtklein@chromium.org,mtklein@google.com,herb@google.com,reed@google.com > > # Not skipping CQ checks because original CL landed > 1 day ago. > > Change-Id: Icab02936d6c18e50b8de353aa1cd4c225e15e0bf > Reviewed-on: https://skia-review.googlesource.com/20800 > Reviewed-by: Mike Reed <reed@google.com> TBR=mtklein@chromium.org,mtklein@google.com,herb@google.com,reed@google.com Change-Id: Ibc579a5a95e78cb257e8d443dbb905db3761e84a No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://skia-review.googlesource.com/20802 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com>
* Revert "Remove deadcode for sRGB image shading."Gravatar Mike Reed2017-06-25
| | | | | | | | | | | | | | | | | | | | | | This reverts commit 51f6a0f89e55b2c0f102aa7963320a23c586e71d. Reason for revert: crazy attempt to fix chrome roll Original change's description: > Remove deadcode for sRGB image shading. > > Change-Id: I9d98da67d97b64ab55cf44b9ae447882dccda1ca > Reviewed-on: https://skia-review.googlesource.com/20695 > Reviewed-by: Mike Klein <mtklein@chromium.org> > Commit-Queue: Herb Derby <herb@google.com> TBR=mtklein@chromium.org,mtklein@google.com,herb@google.com,reed@google.com # Not skipping CQ checks because original CL landed > 1 day ago. Change-Id: Icab02936d6c18e50b8de353aa1cd4c225e15e0bf Reviewed-on: https://skia-review.googlesource.com/20800 Reviewed-by: Mike Reed <reed@google.com>
* use rasterpipeline for colorfiltersGravatar Mike Reed2017-06-25
| | | | | | | | Bug: skia: Change-Id: Ia0e88f3ae29d026843ac7beb2aee3c891123ce18 Reviewed-on: https://skia-review.googlesource.com/20773 Reviewed-by: Mike Klein <mtklein@google.com> Commit-Queue: Mike Reed <reed@google.com>
* remove unused blitrowGravatar Mike Reed2017-06-24
| | | | | | | | Bug: skia: Change-Id: Id905a4b9246a986a3c5de05042e32930d881c1ed Reviewed-on: https://skia-review.googlesource.com/20772 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com>
* Revert "speculative fix for chrome roll"Gravatar Mike Reed2017-06-24
| | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 4220a929f881de79420eb3551653848678666c13. Reason for revert: did not seem to help chrome Original change's description: > speculative fix for chrome roll > > Bug: skia: > Change-Id: Ib1d7d0839093a4ced89f591bc329bbd3adc635f8 > Reviewed-on: https://skia-review.googlesource.com/20768 > Reviewed-by: Mike Reed <reed@google.com> > Commit-Queue: Mike Reed <reed@google.com> TBR=reed@google.com Change-Id: I1906664ebf4aae8bc61bd410a74231bce3f4b562 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: skia: Reviewed-on: https://skia-review.googlesource.com/20782 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com>
* speculative fix for chrome rollGravatar Mike Reed2017-06-24
| | | | | | | | Bug: skia: Change-Id: Ib1d7d0839093a4ced89f591bc329bbd3adc635f8 Reviewed-on: https://skia-review.googlesource.com/20768 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com>
* remove a bit more dead codeGravatar Mike Klein2017-06-24
| | | | | | | Change-Id: I61484672e88d6bb4f75833ee89e7178c4f34d610 Reviewed-on: https://skia-review.googlesource.com/20780 Commit-Queue: Mike Klein <mtklein@google.com> Reviewed-by: Herb Derby <herb@google.com>
* disable optimization (for now) for androidGravatar Mike Reed2017-06-24
| | | | | | | | Bug: skia: Change-Id: I5e66d6f0a1aede5239b5a51a8b51283cede2be93 Reviewed-on: https://skia-review.googlesource.com/20781 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com>
* Remove deadcode for sRGB image shading.Gravatar Herb Derby2017-06-23
| | | | | | | Change-Id: I9d98da67d97b64ab55cf44b9ae447882dccda1ca Reviewed-on: https://skia-review.googlesource.com/20695 Reviewed-by: Mike Klein <mtklein@chromium.org> Commit-Queue: Herb Derby <herb@google.com>
* optimize imageshaders to use clamp for dimension==1Gravatar Mike Reed2017-06-23
| | | | | | | | | Bug: skia: Change-Id: I12021acb5349bd139dceee8ce46de3cfd1ff4d28 Reviewed-on: https://skia-review.googlesource.com/20698 Reviewed-by: Herb Derby <herb@google.com> Reviewed-by: Mike Klein <mtklein@chromium.org> Commit-Queue: Mike Reed <reed@google.com>
* Reduce ANGLE correctness testing blacklistsGravatar Brian Salomon2017-06-23
| | | | | | | | Bug: skia: Change-Id: Ie98d574a9840dd073a396da829aee041fb6595dc Reviewed-on: https://skia-review.googlesource.com/20601 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
* run Test-iOS-... bots with --threads 0.Gravatar Mike Klein2017-06-23
| | | | | | | | | | | | | | | | Yet another attempt to get the iOS bots stable. As GPU testers, the iOS bots are already mostly single threaded, but farm a good chunk of work like image encoding out to threads. This will take them all the way to really single threaded. BUG=skia:6748 Change-Id: I222e3ca402eee9d191c8a413b29a65d19c13e88a Reviewed-on: https://skia-review.googlesource.com/20687 Reviewed-by: Ben Wagner <benjaminwagner@google.com> Commit-Queue: Mike Klein <mtklein@google.com>
* reduce exposure of SK_SUPPORT_LEGACY_TILED_BITMAPS to not replace repeats yetGravatar Mike Reed2017-06-23
| | | | | | | | | | | A large swath of skps slowed down ~5-6x, likely because of repeat_repeat tiling slowdowns. This CL reduces our exposure to keep those in legacy mode for now. Bug: skia: Change-Id: I16d03c01fe8c14a8b9b6a1a9a0c5f31f8d616b4a Reviewed-on: https://skia-review.googlesource.com/20694 Reviewed-by: Mike Klein <mtklein@chromium.org> Commit-Queue: Mike Reed <reed@google.com>
* Update Win Haswell/Broadwell driver.Gravatar Ben Wagner2017-06-23
| | | | | | | | No-Try: true Change-Id: I3bc19ac43e90de3009ca48a18284485efff21ad8 Reviewed-on: https://skia-review.googlesource.com/20690 Commit-Queue: Ben Wagner <benjaminwagner@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* remove DM status threadGravatar Mike Klein2017-06-23
| | | | | | | | | | | Let's see if this helps iOS? BUG=skia:6748 Change-Id: Iec67eacef0058dfb79c0223a955a22c676a35278 Reviewed-on: https://skia-review.googlesource.com/20683 Commit-Queue: Mike Klein <mtklein@google.com> Reviewed-by: Ben Wagner <benjaminwagner@google.com>
* Remove SK_SUPPORT_LEGACY_DITHERING dead codeGravatar Florin Malita2017-06-23
| | | | | | | | | No longer defined in Chrome. Change-Id: I5677ef39f7815e77fa52ec9aa667ae2a51e0f20b Reviewed-on: https://skia-review.googlesource.com/20692 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Florin Malita <fmalita@chromium.org>
* Add rest of paint flags to debug canvas.Gravatar Ben Wagner2017-06-23
| | | | | | | | | Allows the Skia debugger to list all the flags on the paint. Change-Id: I9068965a676809a3e23b066d804506d58910db05 Reviewed-on: https://skia-review.googlesource.com/20689 Reviewed-by: Florin Malita <fmalita@chromium.org> Commit-Queue: Ben Wagner <bungeman@google.com>
* use mul_inv instead of div for tilingGravatar Mike Reed2017-06-23
| | | | | | | | | Bug: skia: Change-Id: I5f88e7923fe204faba8dc5d87454805a4d470d52 Reviewed-on: https://skia-review.googlesource.com/20688 Reviewed-by: Mike Klein <mtklein@chromium.org> Reviewed-by: Florin Malita <fmalita@chromium.org> Commit-Queue: Mike Reed <reed@google.com>
* Finish a few MDB TODOsGravatar Robert Phillips2017-06-23
| | | | | | | Change-Id: I5579bea381ce30dc59502dbeb5841fea927002c2 Reviewed-on: https://skia-review.googlesource.com/20640 Commit-Queue: Robert Phillips <robertphillips@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
* Roll isolate binaries to pick up some auth fixes.Gravatar Vadim Shtayura2017-06-23
| | | | | | | | | | | | | | | | | | Corresponds to infra/infra@33231bcea82e2f7d65d027ebc94afc3e95db26ee Built here: https://luci-milo.appspot.com/buildbot/chromium.infra/infra-continuous-precise-64/7888 https://luci-milo.appspot.com/buildbot/chromium.infra/infra-continuous-mac-10.10-64/7131 https://luci-milo.appspot.com/buildbot/chromium.infra/infra-continuous-win-64/8030 R=rmistry@google.com BUG=skia:6611 Change-Id: I25a961bfb931daac43d544babaad0b4b468de302 Reviewed-on: https://skia-review.googlesource.com/20555 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Vadim Shtayura <vadimsh@google.com> Commit-Queue: Ravi Mistry <rmistry@google.com>
* Hack to fix MSAA GL -> Vk transition on Windows ViewerGravatar Brian Osman2017-06-23
| | | | | | | | Bug: skia: Change-Id: I49417789ab43734a1cbb4010281482970579971d Reviewed-on: https://skia-review.googlesource.com/20505 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
* remove SkDebugfsGravatar Mike Klein2017-06-23
| | | | | | | | | These are just annoying me every time I run DM. Change-Id: I8beffb0f7a479676741fe75a2066dcfd7fb06567 Reviewed-on: https://skia-review.googlesource.com/20686 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
* SkPDF: remove duplicate dict entryGravatar Hal Canary2017-06-23
| | | | | | | Change-Id: I550b4e50694b0cb3ba89c816dc54f3da9e8a9c53 Reviewed-on: https://skia-review.googlesource.com/20682 Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
* skbug5221 unit test should not require N32 support from GPUGravatar Brian Salomon2017-06-23
| | | | | | | Change-Id: I98f0120a012f0adaf0b34d11135a1210a66ecea0 Reviewed-on: https://skia-review.googlesource.com/20681 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
* Complete another MDB todoGravatar Robert Phillips2017-06-23
| | | | | | | Change-Id: Ic1287d79edc610d44ea24f40bf9facde9573d926 Reviewed-on: https://skia-review.googlesource.com/20660 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
* Move mock gpu and caps, add Gr prefix, and create via GrContext::CreateGravatar Brian Salomon2017-06-23
| | | | | | | Change-Id: I02427839f78372e337ed717b071c6baaa8ba4450 Reviewed-on: https://skia-review.googlesource.com/20622 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
* SkPDF: implement SkPDFArray::reserve.Gravatar Hal Canary2017-06-23
| | | | | | | | | | SkTArray::reserve() was implemented here: https://skia.googlesource.com/skia/+/06ca8ec%5E%21/#F28 Change-Id: Ibe78cd686506ccb424a827b37bca9c2904565f85 Reviewed-on: https://skia-review.googlesource.com/20518 Reviewed-by: Ben Wagner <bungeman@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
* Add correctness testing for ANGLE D3D11 ES3Gravatar Brian Salomon2017-06-23
| | | | | | | | | Bug=skia:6776 Change-Id: I51c851d6b80992039c87ae43037721db52958f0c Reviewed-on: https://skia-review.googlesource.com/20503 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
* Clip FreeType glyph bitmap to mask.Gravatar Ben Wagner2017-06-23
| | | | | | | | | | | | | | | | | | | Skia has for some time assumed that when using FT_Render_Glyph with one of the LCD render modes that one extra pixel would be applied to each side of the resulting bitmap. FreieType has changed to make this more conservative when possible, so the pre-allocated SkMask and the generated FT_Bitmap may no longer agree on the size and origin. This change ensures the SkMask and FT_Bitmap are the same size and their origins align. This is not an ideal long term fix, but is both simple and localized for easy and quick back-porting, should that become necessary. BUG=skia:6663 Change-Id: I49ec8f45376be8d867e8aef54eab79537731c310 Reviewed-on: https://skia-review.googlesource.com/20327 Reviewed-by: Herb Derby <herb@google.com> Commit-Queue: Ben Wagner <bungeman@google.com>
* SkPDF: add some TRACE_EVENTsGravatar Hal Canary2017-06-22
| | | | | | | Change-Id: I5f7e7ad57c1e585254378e3fdee4d33c3582d7c2 Reviewed-on: https://skia-review.googlesource.com/20517 Reviewed-by: Ben Wagner <bungeman@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
* fix repeat/mirror sampling bleedGravatar Mike Klein2017-06-22
| | | | | | | | | | | | | | | | | I think this has been broken since we tried to simplify this in https://skia-review.googlesource.com/16547 The HSW backend does still look a little wrong, but improved, and the others seem fixed. Can you see how this affects your test cases, layout tests, etc? BUG=skia:6783 Change-Id: I17957ac8100331bea5b64d674bf43105048b72f6 Reviewed-on: https://skia-review.googlesource.com/20548 Commit-Queue: Mike Klein <mtklein@google.com> Reviewed-by: Florin Malita <fmalita@chromium.org> Reviewed-by: Herb Derby <herb@google.com>
* Remove static initializer restriction for Vulkan testsGravatar Jim Van Verth2017-06-22
| | | | | | | Change-Id: I277110976844fc80271f4e86bb8474047e8abd2a Reviewed-on: https://skia-review.googlesource.com/20510 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Jim Van Verth <jvanverth@google.com>
* Blacklist TransferPixelsTest on NVIDIA ShieldGravatar Jim Van Verth2017-06-22
| | | | | | | | Bug: skia:6784 Change-Id: I7dc870e4cac4d513103f6d61f220f87cf9cc77ef Reviewed-on: https://skia-review.googlesource.com/20508 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Jim Van Verth <jvanverth@google.com>