aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
...
* SkPDF: clean up PDFDevice.Gravatar Hal Canary2017-06-27
| | | | | | | | | | | | | | | Motivation: factor out some code for later re-use; clean up. - mask_to_greyscale_image() - addSMaskGraphicState() - clearMaskOnGraphicState() - stop using bare pointer to indicate ownership. - add ScopedContentEntry::stream() Change-Id: I7abe7ff9eab89e1002692017000cda2ca7642631 Reviewed-on: https://skia-review.googlesource.com/20978 Reviewed-by: Ben Wagner <bungeman@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
* 2pt conical stage for concentric caseGravatar Florin Malita2017-06-27
| | | | | | | | | | If the circles are concentric, we can use the radial stage and post-massage t with a 2x3 stage. Change-Id: I744bc889d509d5f9ff162c179e8ec7ca35410e3c Reviewed-on: https://skia-review.googlesource.com/20972 Reviewed-by: Mike Klein <mtklein@chromium.org> Commit-Queue: Florin Malita <fmalita@chromium.org>
* Roll recipe dependencies (trivial).Gravatar recipe-roller2017-06-27
| | | | | | | | | | | | | | | | | | | | | | | | 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/96e1d788304830a9e405925d3c0a38953502558c gclient flatten: add support for hooks_os (phajdan.jr@chromium.org) https://crrev.com/aee6c8527222f719c55eb9d3e1dcdd6bcd808e95 git-cl-issue: respect --gerrit/--rietveld with integer argument (agable@chromium.org) recipe_engine: https://crrev.com/097a0dbe9f6b8eae39e690de848aea5c3d8ea38b [test] minor refactoring. (iannucci@chromium.org) TBR=borenet@google.com Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Change-Id: Ic6d97eeb4356ca28e0f3c6e507616b96cc6dfad6 Reviewed-on: https://skia-review.googlesource.com/20973 Reviewed-by: Recipe Roller <recipe-roller@chromium.org> Commit-Queue: Recipe Roller <recipe-roller@chromium.org>
* Adjust path_stroke_with_zero_length to draw on pixel centersGravatar Brian Osman2017-06-27
| | | | | | | | Bug: skia: Change-Id: I1183574de4e3a429fe00c5a6a0f71745f1f6f5b4 Reviewed-on: https://skia-review.googlesource.com/20969 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
* Turn on verbose logging on flakily dying bots.Gravatar Ben Wagner2017-06-27
| | | | | | | | | No-Try: true Change-Id: I2c1b598f03d8536aa3ef118630a0aed63fe54f9e Reviewed-on: https://skia-review.googlesource.com/10526 Commit-Queue: Ben Wagner <benjaminwagner@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Re-land sksl fragment processor supportGravatar Ethan Nicholas2017-06-27
| | | | | | | | | | This reverts commit ed50200682e0de72c3abecaa4d5324ebcd1ed9f9. Bug: skia: Change-Id: I9caa7454b391450620d6989dc472abb3cf7a2cab Reviewed-on: https://skia-review.googlesource.com/20965 Reviewed-by: Ben Wagner <benjaminwagner@google.com> Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
* Expand 2pt conical gradient gm with repeat/mirrorGravatar Florin Malita2017-06-27
| | | | | | | Change-Id: Iabb67dbf133690c649e9687233c4bfcb396db0cf Reviewed-on: https://skia-review.googlesource.com/20971 Reviewed-by: Mike Klein <mtklein@chromium.org> Commit-Queue: Florin Malita <fmalita@chromium.org>
* Upgrade android build (and infrastructure) to use ndk r15.Gravatar Derek Sollenberger2017-06-27
| | | | | | | | | | Bug: 6672 Change-Id: Ia6c6bae8a9adfd75172d7116487dfa63c3d69670 Reviewed-on: https://skia-review.googlesource.com/20904 Commit-Queue: Derek Sollenberger <djsollen@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org> Reviewed-by: Mike Klein <mtklein@chromium.org> Reviewed-by: Eric Boren <borenet@google.com>
* Fix TransferBufferTest for TegraX1Gravatar Jim Van Verth2017-06-27
| | | | | | | | | Bug: skia: Change-Id: Ia64f48581e8003baec2eeaa4821354350b3333f4 Reviewed-on: https://skia-review.googlesource.com/20504 Reviewed-by: Robert Phillips <robertphillips@google.com> Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Jim Van Verth <jvanverth@google.com>
* SkDevice::drawBitmap takes x,y, not matrixGravatar Hal Canary2017-06-27
| | | | | | | | | Motivation: a simpler call should make it easier for SkPDF to learn how to do drawBitmap with A8 bitmap and a maskfilter. Change-Id: I1a5d190b40b0e9e08fa8876d265a9835a3e9987d Reviewed-on: https://skia-review.googlesource.com/20961 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
* add dumbest possible 32-bit SkJumper backendGravatar Mike Klein2017-06-27
| | | | | | | | | | | | | | | | Everything uses a ton of stack, nothing tail calls, and for now this is non-Windows only. But, it does run faster than the portable serial code. On my trashcan, running `monobench SkRasterPipeline_compile`: - Normal 64-bit AVX build: 43.6ns - Before this CL, 32-bit: 707.9ns - This CL: 147.5ns Change-Id: I4a8929570ace47193ed8925c58b70bb22d6b1447 Reviewed-on: https://skia-review.googlesource.com/20964 Reviewed-by: Mike Reed <reed@google.com> Reviewed-by: Herb Derby <herb@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
* Revert "sksl fragment processor support"Gravatar Ethan Nicholas2017-06-27
| | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit ccf59917d3fe7aaf59de714acfbd0596503f324f. Reason for revert: breaking iOS bots Original change's description: > sksl fragment processor support > > Bug: skia: > Change-Id: Ia3b0305c2b0c78074303831f628fb01852b90d34 > Reviewed-on: https://skia-review.googlesource.com/17843 > Commit-Queue: Ethan Nicholas <ethannicholas@google.com> > Reviewed-by: Ben Wagner <benjaminwagner@google.com> > Reviewed-by: Mike Klein <mtklein@google.com> TBR=mtklein@google.com,benjaminwagner@google.com,ethannicholas@google.com Change-Id: I0a33060c7c42c7b44c5c13d443ac42958291c2f1 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: skia: Reviewed-on: https://skia-review.googlesource.com/20962 Reviewed-by: Ethan Nicholas <ethannicholas@google.com> Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
* sksl fragment processor supportGravatar Ethan Nicholas2017-06-27
| | | | | | | | | Bug: skia: Change-Id: Ia3b0305c2b0c78074303831f628fb01852b90d34 Reviewed-on: https://skia-review.googlesource.com/17843 Commit-Queue: Ethan Nicholas <ethannicholas@google.com> Reviewed-by: Ben Wagner <benjaminwagner@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
* Load FreeType glyph bitmap before emboldening.Gravatar Ben Wagner2017-06-27
| | | | | | | | | | | | | If a bitmap glyph was loaded with FT_LOAD_BITMAP_METRICS_ONLY then the glyph must be re-loaded without this flag before accessing the bitmap. BUG=chromium:725975 Change-Id: If5e5a6844e9c32238560135e141fea7f77ad7fac Reviewed-on: https://skia-review.googlesource.com/20830 Reviewed-by: Herb Derby <herb@google.com> Reviewed-by: Florin Malita <fmalita@chromium.org> Commit-Queue: Ben Wagner <bungeman@google.com>
* add _hsw lowp backendGravatar Mike Klein2017-06-27
| | | | | | | | | | CQ_INCLUDE_TRYBOTS=skia.primary:Build-Ubuntu-Clang-x86_64-Debug-MSAN Change-Id: Id53279c17589b3434629bb644358ee238af8649f Reviewed-on: https://skia-review.googlesource.com/20269 Commit-Queue: Mike Klein <mtklein@chromium.org> Reviewed-by: Herb Derby <herb@google.com> Reviewed-by: Mike Reed <reed@google.com>
* Add shading language version to GL dumpGravatar Jim Van Verth2017-06-27
| | | | | | | | Bug: skia:6759 Change-Id: I021a8487ccb1d127b2005285e439be34d644f333 Reviewed-on: https://skia-review.googlesource.com/20905 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Jim Van Verth <jvanverth@google.com>
* 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>