aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* Update SKP versionGravatar UpdateSKPs2017-07-14
| | | | | | | | | | | | Automatic commit by the RecreateSKPs bot. TBR=update-skps@skia.org NO_MERGE_BUILDS Change-Id: I6625bf8978209ff55a4107ecc2fca2341f01618f Reviewed-on: https://skia-review.googlesource.com/23600 Reviewed-by: update-skps <update-skps@skia.org> Commit-Queue: update-skps <update-skps@skia.org>
* Add "findOrMakeStaticBuffer" to GrOnFlushResourceProviderGravatar Chris Dalton2017-07-14
| | | | | | | | | Bug: skia: Change-Id: If381c91f40566e6630799ea9985c48e2b738596e Reviewed-on: https://skia-review.googlesource.com/23420 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com> Commit-Queue: Chris Dalton <csmartdalton@google.com>
* Make ops in bezier_effects GMs be non-Legacy GrMeshDrawOpsGravatar Brian Salomon2017-07-14
| | | | | | | Change-Id: I1d56536820a61af293a7bce290f35b724c3a8257 Reviewed-on: https://skia-review.googlesource.com/23480 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
* Remove SK_SUPPORT_LEGACY_RECT flagGravatar Yuqian Li2017-07-14
| | | | | | | | | | The rebaseline has finished in Chromium. Bug: skia: Change-Id: Ibafb32b3683f1bfc3e451911e6fb8382bfc9b857 Reviewed-on: https://skia-review.googlesource.com/23481 Reviewed-by: Cary Clark <caryclark@google.com> Commit-Queue: Yuqian Li <liyuqian@google.com>
* fixed sksl static ifs to work for CircleEffectGravatar Ethan Nicholas2017-07-14
| | | | | | | | | | | | | | | | | | static ifs (and switches) in .fp files are a bit tricky, because they aren't necessarily static when the CPP file is being produced. They become static when the CPP file produces the final SkSL; at this point the final values of the 'in' variables are known. This change permits 'deferred' static ifs and switches. The initial compilation (.fp -> .cpp) passes the @if / @switch through, and then the final compilation (.cpp's generated SkSL -> GLSL or whatever) enforces the static test. Bug: skia: Change-Id: I0087dfe1725c8fd350507ac77f64db1d82659cdf Reviewed-on: https://skia-review.googlesource.com/23403 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
* Make ops used in OnFlushCallbackTest be non-legacy GrMeshDrawOps.Gravatar Brian Salomon2017-07-14
| | | | | | | Change-Id: Idf7f838bcdda200aeea1bfbcd5eea7cbebd76aa2 Reviewed-on: https://skia-review.googlesource.com/23005 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
* Purge cached SkPictureShader entries on shader deletionGravatar Florin Malita2017-07-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're currently adding picture shader cache entries to the resource cache, but we don't ever purge. To avoid exhausting the budget, add logic to associate cached entries with their owning picture shader, and purge when the shader is deleted. -- Side note -- The current cache key is K(pictureID, ...) so technically the cache entries are associated with the picture, not the shader. One could resonably argue we should only purge when the *picture* is deleted. Unfortunately, this doesn't work: the cache entries contain indirect refs to the picture (SkImageShader -> SkImage_Generated -> SkPictureImageGenerator -> SkPicture), so the picture is always kept alive. Associating the cache entries with the shader itself seems like a reasonable alternative, even if we give up some cache persistence in the process. Change-Id: Ia115dbb5ae627e5ee171da7c4430fecfd42f4292 Reviewed-on: https://skia-review.googlesource.com/23380 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Florin Malita <fmalita@chromium.org>
* Add metal config to toolsGravatar Greg Daniel2017-07-14
| | | | | | | | | Bug: skia: Change-Id: I84dbd56c17d4856496af4491f340ec560e29c8a6 Reviewed-on: https://skia-review.googlesource.com/23200 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Jim Van Verth <jvanverth@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com>
* init GrDrawAtlasOp::fColorGravatar Brian Salomon2017-07-14
| | | | | | | | Change-Id: Ifa474610dcb238a05ce7d688bae639d36ef4f4c1 Reviewed-on: https://skia-review.googlesource.com/23402 Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
* Publish uploaded RecreateSKPs CLGravatar Ravi Mistry2017-07-14
| | | | | | | | Bug: skia:6797 Change-Id: I02e2a3d13a52bba66f4f9d2129aae5f094828329 Reviewed-on: https://skia-review.googlesource.com/23323 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Ravi Mistry <rmistry@google.com>
* converted CircleEffect to SkSLGravatar Ethan Nicholas2017-07-14
| | | | | | | | Bug: skia: Change-Id: I93d117c22ae2b374294f6a5e961c497ac2c92b09 Reviewed-on: https://skia-review.googlesource.com/23301 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
* Delete unread SkPicture mboxGravatar Florin Malita2017-07-13
| | | | | | | | | No longer read?! Change-Id: I6258a64590bb5ee16e02a7ad923ad41cc7bd8791 Reviewed-on: https://skia-review.googlesource.com/23043 Reviewed-by: Mike Klein <mtklein@google.com> Commit-Queue: Florin Malita <fmalita@chromium.org>
* Move GrAALinearizingConvexPathRenderer off GrLegacyMeshDrawOpGravatar Brian Salomon2017-07-13
| | | | | | | Change-Id: I7d3c4053d4a833215371d77de683964cef317a5c Reviewed-on: https://skia-review.googlesource.com/20460 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
* Add support for creating a GrContext backed by Metal.Gravatar Greg Daniel2017-07-13
| | | | | | | | | | | Also adds the support code to allow our TestContext to create a Metal backend. Bug: skia: Change-Id: Ia850687019d79b897bb16e2c151f4f8526721ad9 Reviewed-on: https://skia-review.googlesource.com/22644 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Greg Daniel <egdaniel@google.com>
* Make DefaultPathOp a non-legacy GrMeshDrawOpGravatar Brian Salomon2017-07-13
| | | | | | | Change-Id: I9abc795e7c3aca7c129faee48190be749c6dc159 Reviewed-on: https://skia-review.googlesource.com/23101 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
* Revert "Re-enable uploading to partner bucket"Gravatar Ravi Mistry2017-07-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 4e91f16fdf94cab7f3361d7efa6b12bccabee25e. Reason for revert: Still does not work Original change's description: > Re-enable uploading to partner bucket > > This is a revert of > https://skia.googlesource.com/skia/+/da066b865c1e2777a84a829486a7b37a984484b4 > > NoTry: true > Bug: skia:6797 > Change-Id: I9eca7e4b2234c79e39b2c25aaf459512dd206acc > Reviewed-on: https://skia-review.googlesource.com/23202 > Commit-Queue: Ravi Mistry <rmistry@google.com> > Reviewed-by: Ravi Mistry <rmistry@google.com> > Reviewed-by: Eric Boren <borenet@google.com> TBR=borenet@google.com,rmistry@google.com Change-Id: If1abb65801ba93d409825e290e22d7a9eb3818fa No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: skia:6797 Reviewed-on: https://skia-review.googlesource.com/23004 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Ravi Mistry <rmistry@google.com>
* Make GrDrawVerticesOp a non-legacy GrMeshDrawOpGravatar Brian Salomon2017-07-13
| | | | | | | Change-Id: I0e1e6815ed41764115bc84a5967b3da3be4ce147 Reviewed-on: https://skia-review.googlesource.com/22722 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
* Add null check for non-cached shadows.Gravatar Jim Van Verth2017-07-13
| | | | | | | Change-Id: I8015a026d4bb54f04ee86fca0d4b0be3bc1d9bd8 Reviewed-on: https://skia-review.googlesource.com/23123 Reviewed-by: Derek Sollenberger <djsollen@google.com> Commit-Queue: Jim Van Verth <jvanverth@google.com>
* Re-enable uploading to partner bucketGravatar Ravi Mistry2017-07-13
| | | | | | | | | | | | | This is a revert of https://skia.googlesource.com/skia/+/da066b865c1e2777a84a829486a7b37a984484b4 NoTry: true Bug: skia:6797 Change-Id: I9eca7e4b2234c79e39b2c25aaf459512dd206acc Reviewed-on: https://skia-review.googlesource.com/23202 Commit-Queue: Ravi Mistry <rmistry@google.com> Reviewed-by: Ravi Mistry <rmistry@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Avoid unneeded picture shader color space transformsGravatar Florin Malita2017-07-13
| | | | | | | | | | Also update SkPictureImageFilter to use SkColorSpace::Equals instead of direct comparison. Change-Id: I434cd6d1afcb869d2a56601b72edf56d207d9868 Reviewed-on: https://skia-review.googlesource.com/23201 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Florin Malita <fmalita@chromium.org>
* Download gitcookies from GS for RecreateSKPs botGravatar Ravi Mistry2017-07-13
| | | | | | | | | NoTry: true Bug: skia:6797 Change-Id: Id7005ba5252c5f1e3f85a8bff92371e0562ab87b Reviewed-on: https://skia-review.googlesource.com/22726 Commit-Queue: Ravi Mistry <rmistry@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Make GrDrawAtlasOp a non-legacy GrMeshDrawOpGravatar Brian Salomon2017-07-13
| | | | | | | Change-Id: I4d517855dbe3ee6894d7d457e0fc4488f204300c Reviewed-on: https://skia-review.googlesource.com/22542 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
* remove dead code for legacy image encode apiGravatar Mike Reed2017-07-13
| | | | | | | | Bug: skia: Change-Id: Ia90d776946281473c56cd93006df1b523475696a Reviewed-on: https://skia-review.googlesource.com/23022 Reviewed-by: Leon Scroggins <scroggo@google.com> Commit-Queue: Mike Reed <reed@google.com>
* Add trace events for shader generation.Gravatar Ryan Macnak2017-07-13
| | | | | | | | | This helps explain why some frames are slow in Flutter. Change-Id: Ic8b36186f1c8c262950e5ff9e0da3b808cff8d0c Reviewed-on: https://skia-review.googlesource.com/22087 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
* New zero length path GMGravatar Brian Osman2017-07-13
| | | | | | | | | | | | | | | | | | | | | | I replaced the previous once, because I think this covers everything it was testing, and lots more. It renders many combinations of parameters, and automatically detects if the results are correct or not. Each test is color coded, and the overall image is tinted with gold's red/green to make triaging easy. Adds four new top-level GMs (single and double contour, with and without AA). Each one tests several more params: - Cap type - Stroke width - Path structure (M vs ML vs MZ vs MLZ) - Different verbs (L vs Q vs C vs A) Bug: skia:6781 Change-Id: Ie81b4417101fea0b439cb094a65257dc06b6b5c0 Reviewed-on: https://skia-review.googlesource.com/21480 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
* Revert "Revert "Experimental blur code with 32 bit fix.""Gravatar Mike Reed2017-07-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 27b3d272a8fcce24813212f5816b177090111ec6. Reason for revert: guard has landed in android Original change's description: > Revert "Experimental blur code with 32 bit fix." > > This reverts commit d4b2c537d058ad4cb890ba116d00aa86c3416c08. > > Reason for revert: speculative fix for android-roll > > java.lang.AssertionError: expected:<0> but was:<255> > at org.junit.Assert.fail(Assert.java:88) > at org.junit.Assert.failNotEquals(Assert.java:834) > at org.junit.Assert.assertEquals(Assert.java:645) > at org.junit.Assert.assertEquals(Assert.java:631) > at android.graphics.cts.BlurMaskFilterTest.verifyColor(BlurMaskFilterTest.java:79) > at android.graphics.cts.BlurMaskFilterTest.verifyQuadrants(BlurMaskFilterTest.java:72) > at android.graphics.cts.BlurMaskFilterTest.testBlurMaskFilter(BlurMaskFilterTest.java:56) > > Original change's description: > > Experimental blur code with 32 bit fix. > > > > This uses a new method of blurring that runs the three > > passes of the box filter in a single pass. This implementation > > currently only does 1x1 pixel at a time, but it should be simple > > to expand to 4x4 pixels at a time. > > > > On the blur_10_normal_high_quality benchmark, the new is 7% faster > > than the old code. For the blur_100.50_normal_high_quality > > benchmark, the new code is 11% slower. > > > > Bug: skia: > > Change-Id: I847270906b0ceac1dfbf43ab5446756689ef660f > > Reviewed-on: https://skia-review.googlesource.com/22700 > > Reviewed-by: Mike Reed <reed@google.com> > > Commit-Queue: Herb Derby <herb@google.com> > > TBR=herb@google.com,reed@google.com > > Change-Id: Ie84f6bf8872cae08c06d679f0c2f2e6c3d8a02a2 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: skia: > Reviewed-on: https://skia-review.googlesource.com/22880 > Reviewed-by: Mike Reed <reed@google.com> > Commit-Queue: Mike Reed <reed@google.com> TBR=herb@google.com,reed@google.com Change-Id: I393d1c05f83ccf98137201bc7b4e7d8e8b0e8742 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: skia: Reviewed-on: https://skia-review.googlesource.com/23121 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com>
* Give GrSimpleMeshDrawOpHelper a cpp fileGravatar Brian Salomon2017-07-13
| | | | | | | Change-Id: I1ac3f7d41d3d546801a7fc14008b801a11429472 Reviewed-on: https://skia-review.googlesource.com/22368 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
* SPIR-V support for more integer operationsGravatar Ethan Nicholas2017-07-13
| | | | | | | | Bug: skia: Change-Id: Ic4ff8dd8b08df8c44222bd7844f4a872377ff240 Reviewed-on: https://skia-review.googlesource.com/21440 Reviewed-by: Chris Dalton <csmartdalton@google.com> Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
* Fix two issues in DashOpGravatar Brian Salomon2017-07-13
| | | | | | | | | | | | 1) Bitfield was too small on Windows for signed enum type 2) Check for unused stencil settings rather than null Also makes bool bitfields adjacent for better packing on Windows. Change-Id: I8115cc31c55e83aa7bd85d99c1fca72042617aa3 Reviewed-on: https://skia-review.googlesource.com/23120 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
* Make ShadowRRectOp a non-legacy GrMeshDrawOpGravatar Brian Salomon2017-07-13
| | | | | | | | | This also removes the ability to have an arbitrary GrPaint with this op and as a consequence simplifies the op and its GrGeometryProcessor. Change-Id: I19cc1c6f73a47e8925fc826291aad42e9423164d Reviewed-on: https://skia-review.googlesource.com/22380 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Jim Van Verth <jvanverth@google.com>
* Fail to make render target context if GrContext is abandonedGravatar Brian Salomon2017-07-13
| | | | | | | Change-Id: I69eabbc0a8cd02ddd7292f81dd24e6b949bf38e5 Reviewed-on: https://skia-review.googlesource.com/23041 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com>
* don't test index8 -- no longer supportedGravatar Mike Reed2017-07-13
| | | | | | | | Bug: skia:6828 Change-Id: I7ce4d91c2fa66b3a81e5143819dd9d774fa1c116 Reviewed-on: https://skia-review.googlesource.com/23140 Commit-Queue: Mike Reed <reed@google.com> Reviewed-by: Leon Scroggins <scroggo@google.com>
* Add abandoned context check to SkImage::MakeFromDeferredTextureImageDataGravatar Brian Salomon2017-07-13
| | | | | | | Change-Id: Ibcae793339cfaf9dae8cb9122f387def535ce1bc Reviewed-on: https://skia-review.googlesource.com/23040 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
* Re-land of "converted GrSimpleTextureEffect to sksl"Gravatar Ethan Nicholas2017-07-13
| | | | | | | | | | This reverts commit baf981f71614e7a2fbe6af2726e65126d431ed8e. Bug: skia: Change-Id: I36f6bfb616f1ec2b89043e3a6f7cbdf473bc9588 Reviewed-on: https://skia-review.googlesource.com/22369 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
* Fix computeOutputColorSpaceGravatar Leon Scroggins III2017-07-13
| | | | | | | | | | | | This was accidentally broken when we added a guard for kIndex_8 (https://skia-review.googlesource.com/c/22721). Reenable the code for 32 bit color types. Change-Id: Ibb79a163dc1cffe14e19d2affa17856b549509c4 Reviewed-on: https://skia-review.googlesource.com/23020 Commit-Queue: Mike Reed <reed@google.com> Reviewed-by: Mike Reed <reed@google.com>
* add guard for api change in google3Gravatar Mike Reed2017-07-13
| | | | | | | Bug: skia: Change-Id: I2d84c546cd2bc097c809690bb423581cbd98f2a8 Reviewed-on: https://skia-review.googlesource.com/23060 Reviewed-by: Mike Reed <reed@google.com>
* Revert "add guard for recent api change"Gravatar Mike Reed2017-07-13
| | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 1c8bb8a4c63c327fb856a41e1b0b19775eac0ac1. Reason for revert: google3 now updated Original change's description: > add guard for recent api change > > Bug: skia: > Change-Id: Ief80e0eb08d091c54cacb045d074c8564adcedba > Reviewed-on: https://skia-review.googlesource.com/22640 > Commit-Queue: Mike Reed <reed@google.com> > Reviewed-by: Mike Reed <reed@google.com> TBR=benjaminwagner@google.com,reed@google.com Change-Id: If534fa0f508b5087e3e7527f822906626fdb65c0 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: skia: Reviewed-on: https://skia-review.googlesource.com/23002 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com>
* Remove renderTargetContext swapping from GaussianBlur method (take 2)Gravatar Robert Phillips2017-07-13
| | | | | | | | TBR=bsalomon@google.com Change-Id: I6c594a8dfef1c3319a56665437f1e13e746a2875 Reviewed-on: https://skia-review.googlesource.com/23001 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
* Revert "Remove renderTargetContext swapping from GaussianBlur method"Gravatar Robert Phillips2017-07-13
| | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 0284de0a215d61a849678cb04e5f21f989ee858f. Reason for revert: assert firing in nanobench Original change's description: > Remove renderTargetContext swapping from GaussianBlur method > > Change-Id: Iee3d5165a09b5760e001cade4f9d68e823ae695f > Reviewed-on: https://skia-review.googlesource.com/22725 > Reviewed-by: Brian Salomon <bsalomon@google.com> > Commit-Queue: Robert Phillips <robertphillips@google.com> TBR=bsalomon@google.com,robertphillips@google.com Change-Id: Ied1e184c7ab5de0e41814ef8be9c4c94a679d56d No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://skia-review.googlesource.com/23000 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
* Remove renderTargetContext swapping from GaussianBlur methodGravatar Robert Phillips2017-07-13
| | | | | | | Change-Id: Iee3d5165a09b5760e001cade4f9d68e823ae695f Reviewed-on: https://skia-review.googlesource.com/22725 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
* Guard to remove kIndex_8_SkColorTypeGravatar Mike Reed2017-07-13
| | | | | | | | | Bug: skia:6828 Change-Id: Ia942a36abb18213184f8d436555a658270d97d47 Reviewed-on: https://skia-review.googlesource.com/22721 Commit-Queue: Mike Reed <reed@google.com> Reviewed-by: Florin Malita <fmalita@chromium.org> Reviewed-by: Herb Derby <herb@google.com>
* Remove dead gradient perspective codeGravatar Florin Malita2017-07-13
| | | | | | | | | | | The cool bit here is that Sk4fLinearGradient is now always fast, and never calls the slow base-class impl. This means we can rip out Sk4fGradientBase::{mapTs, shadeSpan4f, etc}. Change-Id: Id3788bc810873b2a209f66efa4187c84b3397e2f Reviewed-on: https://skia-review.googlesource.com/22366 Reviewed-by: Herb Derby <herb@google.com> Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Florin Malita <fmalita@chromium.org>
* Make MSAAPathOp a non-legacy GrMeshDrawOpGravatar Brian Salomon2017-07-13
| | | | | | | Change-Id: Ibe1626a3ebecad0ccc51505c670f9a3ee98d5209 Reviewed-on: https://skia-review.googlesource.com/22361 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com>
* Revert "Revert "Temporarily stop uploading to partner bucket in RecreateSKPs ↵Gravatar Ravi Mistry2017-07-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bot"" This reverts commit 0f3942fc247cb541a7327b6335054694d7b8e186. Reason for revert: Still cannot upload to partner bucket Original change's description: > Revert "Temporarily stop uploading to partner bucket in RecreateSKPs bot" > > This reverts commit 6945c946ea800b7bff31920914a990c88a706718. > > Reason for revert: Should be able to upload to partner bucket now > > Original change's description: > > Temporarily stop uploading to partner bucket in RecreateSKPs bot > > > > 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> > > TBR=borenet@google.com,rmistry@google.com > > # Not skipping CQ checks because original CL landed > 1 day ago. > > Bug: skia:6797 > Change-Id: I145275da0050da01fca3e6e9c3c5efd84bdadea1 > Reviewed-on: https://skia-review.googlesource.com/21738 > Reviewed-by: Ravi Mistry <rmistry@google.com> > Commit-Queue: Ravi Mistry <rmistry@google.com> TBR=borenet@google.com,rmistry@google.com # Not skipping CQ checks because original CL landed > 1 day ago. Bug: skia:6797 Change-Id: I5114c2cacd239c45c6a60cb7aa795c96ef28439e Reviewed-on: https://skia-review.googlesource.com/22840 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Ravi Mistry <rmistry@google.com>
* Revert "Experimental blur code with 32 bit fix."Gravatar Mike Reed2017-07-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit d4b2c537d058ad4cb890ba116d00aa86c3416c08. Reason for revert: speculative fix for android-roll java.lang.AssertionError: expected:<0> but was:<255> at org.junit.Assert.fail(Assert.java:88) at org.junit.Assert.failNotEquals(Assert.java:834) at org.junit.Assert.assertEquals(Assert.java:645) at org.junit.Assert.assertEquals(Assert.java:631) at android.graphics.cts.BlurMaskFilterTest.verifyColor(BlurMaskFilterTest.java:79) at android.graphics.cts.BlurMaskFilterTest.verifyQuadrants(BlurMaskFilterTest.java:72) at android.graphics.cts.BlurMaskFilterTest.testBlurMaskFilter(BlurMaskFilterTest.java:56) Original change's description: > Experimental blur code with 32 bit fix. > > This uses a new method of blurring that runs the three > passes of the box filter in a single pass. This implementation > currently only does 1x1 pixel at a time, but it should be simple > to expand to 4x4 pixels at a time. > > On the blur_10_normal_high_quality benchmark, the new is 7% faster > than the old code. For the blur_100.50_normal_high_quality > benchmark, the new code is 11% slower. > > Bug: skia: > Change-Id: I847270906b0ceac1dfbf43ab5446756689ef660f > Reviewed-on: https://skia-review.googlesource.com/22700 > Reviewed-by: Mike Reed <reed@google.com> > Commit-Queue: Herb Derby <herb@google.com> TBR=herb@google.com,reed@google.com Change-Id: Ie84f6bf8872cae08c06d679f0c2f2e6c3d8a02a2 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: skia: Reviewed-on: https://skia-review.googlesource.com/22880 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com>
* Make RegionOp a non-legacy GrMeshDrawOpGravatar Brian Salomon2017-07-13
| | | | | | | | | Also add a test factory function. Change-Id: I612973312824a4ba358965ce8378160a980b86fa Reviewed-on: https://skia-review.googlesource.com/22213 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
* SkPDF: start to support AlphaOnly Images as masks.Gravatar Hal Canary2017-07-12
| | | | | | | | | Does not work when img->isAlphaOnly() && pnt.getMaskFilter(). Change-Id: I60712d035e3a505799258e24660bba30f2c0ed53 Reviewed-on: https://skia-review.googlesource.com/22723 Reviewed-by: Ben Wagner <bungeman@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
* Add arc support to gpu Obj c++ codeGravatar Greg Daniel2017-07-12
| | | | | | | | | | | | | This is mainly for getting ready to start adding lots of metal backend code. I've also update the "gpu tools" target to require ARC with involved updating one IOS file in there. Bug: skia: Change-Id: Ied22e8fe7532445cc274efb529e3450654a6614b Reviewed-on: https://skia-review.googlesource.com/22484 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
* SkPDF: simplify drawImage/Bitmap/Sprite code.Gravatar Hal Canary2017-07-12
| | | | | | | | | | | | | | | - SkImageSubset becomes SkKeyedImage - SkPDFCanvas::onDraw{Bitmap, Image} go away - Remove SkPDFCanvas: base classes now do the right thing. - SkPDFDevice::draw{Bitmap,Image}{Rect,}() simplified - 244 fewer SLOC. All but a few PDFs are identical, those rasterize almost the same. Change-Id: I3ceb3b8935c689719cedf1ad544b0407b5c1733e Reviewed-on: https://skia-review.googlesource.com/22218 Commit-Queue: Hal Canary <halcanary@google.com> Reviewed-by: Ben Wagner <bungeman@google.com>
* Add some raster pipeline perspective assertsGravatar Florin Malita2017-07-12
| | | | | | | | | | | | | | | | | | | | | I meant to add these when removing the guard, but since we landed without a guard, might as well do it now. A couple of things exposed by these asserts: 1) we need to also catch perspective in local matrices 2) we need to disallow burst mode with perspective Also tweak the predicate to hasPerspective() instead of explicit mask check. Change-Id: I099e5125fca52dccffca77c60fc800bbdf539b53 Reviewed-on: https://skia-review.googlesource.com/22483 Reviewed-by: Mike Reed <reed@google.com> Reviewed-by: Mike Klein <mtklein@google.com> Reviewed-by: Herb Derby <herb@google.com> Commit-Queue: Florin Malita <fmalita@chromium.org>
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-04 22:06:40 +0000