aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* Symmetric rendering when >20px with DirectWrite.Gravatar Ben Wagner2017-04-14
| | | | | | | | | | | | | If the font has a gasp table use it to determine symmetric. Otherwise, use symmetric if the the font isn't hinted or is >20px. The remaining cases use non-symmetric. BUG=chromium:706792,skia:6432 Change-Id: I91b66a9615aae27c195e1545298a9d36bc58a705 Reviewed-on: https://skia-review.googlesource.com/11400 Reviewed-by: Ben Wagner <bungeman@google.com> Commit-Queue: Ben Wagner <bungeman@google.com>
* Fix advances for aliased text with DirectWrite.Gravatar Ben Wagner2017-04-14
| | | | | | | | | | | | | The existing code assumes that GetGdiCompatibleGlyphMetrics transform parameter acts the same as the GetGlyphOutlineW lpmat2 parameter. However, it appears this is very much not true. BUG=skia:6499 Change-Id: I88c762b27acd09cc746da4d6e2b901540876908a Reviewed-on: https://skia-review.googlesource.com/13479 Reviewed-by: Ben Wagner <bungeman@google.com> Commit-Queue: Ben Wagner <bungeman@google.com>
* Revert "Reduce read/write-SurfacePixels call sites"Gravatar Robert Phillips2017-04-14
| | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 74d5a1470378dc7dae9eec38d46f9f802fc5c249. Reason for revert: erg Original change's description: > Reduce read/write-SurfacePixels call sites > > Change-Id: I657c489c56b93765567be69076fb4c3d526e55a5 > Reviewed-on: https://skia-review.googlesource.com/13068 > Reviewed-by: Brian Osman <brianosman@google.com> > Commit-Queue: Robert Phillips <robertphillips@google.com> > TBR=robertphillips@google.com,brianosman@google.com,egdaniel@google.com,reviews@skia.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Change-Id: If80f36ddae9d695ee2e5413f3eb49a1f5c32f8a4 Reviewed-on: https://skia-review.googlesource.com/13506 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
* Reduce read/write-SurfacePixels call sitesGravatar Robert Phillips2017-04-14
| | | | | | | Change-Id: I657c489c56b93765567be69076fb4c3d526e55a5 Reviewed-on: https://skia-review.googlesource.com/13068 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
* Add jobs for Samsung Chromebook XE303C12Gravatar Kevin Lubick2017-04-14
| | | | | | | | | Bug: skia:6507 NOTRY=true Change-Id: I957a72d496ff521972fbc505fcb85cb30c019a93 Reviewed-on: https://skia-review.googlesource.com/13477 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Kevin Lubick <kjlubick@google.com>
* sk_sp-ify opList creation & storageGravatar Robert Phillips2017-04-14
| | | | | | | Change-Id: Idd4d81cd248ad2b2169028ac2e269a66c9cad26b Reviewed-on: https://skia-review.googlesource.com/13400 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
* tools/check-headers-self-sufficient: optionally test one file at a timeGravatar Hal Canary2017-04-14
| | | | | | | Change-Id: Ie7703e9b6abb7229cd8c2dbcdf305fa4240f416c Reviewed-on: https://skia-review.googlesource.com/11385 Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
* Add documentation for color correct skia to websiteGravatar Matt Sarett2017-04-14
| | | | | | | | | | | | BUG=skia: NOTRY=true DOCS_PREVIEW= https://skia.org/?cl=10043 Change-Id: I025bf296518470dd2d5f62d7024a0f5d0fcc91c4 Reviewed-on: https://skia-review.googlesource.com/10043 Commit-Queue: Matt Sarett <msarett@google.com> Reviewed-by: Brian Osman <brianosman@google.com>
* Improve accuracy of cubic classificationGravatar Chris Dalton2017-04-14
| | | | | | | | | | | | | | | | | | - Updates the logic to reflect the Loop-Blinn paper instead of the GPU gems website. - Removes the threshold for detecting local cusps. The serpentine codepath works for these cusps anyway, so what we really want to know is whether the discriminant is negative. - Makes sure to not scale the inflection function by 1/0. - Shifts the inflection function coefficients in d[] so they match the paper. - Stores the cubic discriminant in d[0]. Bug: skia: Change-Id: I909a522a0fd27c9c8dfbc27d968bc43eeb7a416f Reviewed-on: https://skia-review.googlesource.com/13304 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Chris Dalton <csmartdalton@google.com>
* Add documentation for android-o-roll.skia.orgGravatar Ravi Mistry2017-04-14
| | | | | | | | | | | | | BUG=skia:6438 NOTRY=true DOCS_PREVIEW= https://skia.org/?cl=13475 Change-Id: I7d76d66b9b665fafc5c89bd4620517ebb9d2b076 Change-Id: I7d76d66b9b665fafc5c89bd4620517ebb9d2b076 Reviewed-on: https://skia-review.googlesource.com/13475 Reviewed-by: Matt Sarett <msarett@google.com> Reviewed-by: Derek Sollenberger <djsollen@google.com> Commit-Queue: Ravi Mistry <rmistry@google.com>
* fix pathops_unittest test stringsGravatar Cary Clark2017-04-14
| | | | | | | | | | | | | Using std::string is tons faster than SkString; multiple callers to std::string don't run into thread contention but SkString does. R=csmartdalton@google.com Change-Id: I0357c6a9c73856bfffbb76e65c275acdfe7d8159 Reviewed-on: https://skia-review.googlesource.com/13471 Reviewed-by: Chris Dalton <csmartdalton@google.com> Commit-Queue: Cary Clark <caryclark@google.com>
* Let region decoders query the type and color spaceGravatar Matt Sarett2017-04-14
| | | | | | | | | | | | | b/36905374, b/32984164 Test: CtsGraphicsTestCases Written by: romainguy@ Change-Id: Ief4bba6ec3cd3446224199d05f06ffcffaaa72b5 Reviewed-on: https://skia-review.googlesource.com/13472 Reviewed-by: Matt Sarett <msarett@google.com> Reviewed-by: Derek Sollenberger <djsollen@google.com> Commit-Queue: Matt Sarett <msarett@google.com>
* Do non-deterministic FP opt testing on all botsGravatar Brian Osman2017-04-14
| | | | | | | | Bug: skia:6395 Change-Id: Ic5065bfc91cac83495836cd6d2a7cc059423c96e Reviewed-on: https://skia-review.googlesource.com/13412 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
* refine .type ...,%functionGravatar Mike Klein2017-04-13
| | | | | | | | | | | | | | | | .type is an ELF thing, not understood by Mach. So do the same sort of #define trick we do for HIDDEN. This expands the use of .type ...,%function to everywhere that supports it, rather than just where we needed it. Feels cozier this way. CQ_INCLUDE_TRYBOTS=skia.primary:Build-Mac-Clang-arm-Debug-iOS,Build-Mac-Clang-arm-Release-iOS,Test-ChromeOS-Clang-Chromebook_C100p-GPU-MaliT764-arm-Release,Test-ChromeOS-Clang-Chromebook_C100p-GPU-MaliT764-arm-Debug Change-Id: Iaff01b0f3f70ceedf743d7a553915792cdd7e569 Reviewed-on: https://skia-review.googlesource.com/13469 Commit-Queue: Mike Klein <mtklein@chromium.org> Reviewed-by: Herb Derby <herb@google.com>
* Update Skia milestone to 60Gravatar Heather Miller2017-04-13
| | | | | | | | | BUG=skia: Change-Id: I1354ff96f5ff62c02af1d9cf39f0a971da077b17 Reviewed-on: https://skia-review.googlesource.com/13465 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com>
* jumper, explicitly tag functions as functionsGravatar Mike Klein2017-04-13
| | | | | | | | | | | | | | | | | | This helps the linker decide to rewrite bl -> blx when linking Thumb2 SkJumper.o code with ARM SkJumper_generated.o. The reason Android wasn't failing is because it somehow figured out to do this without these .type directives. We use a different toolchain for ChromeOS builds that I guess needs more handholding. BUG=skia:6471 CQ_INCLUDE_TRYBOTS=skia.primary:Test-ChromeOS-Clang-Chromebook_C100p-GPU-MaliT764-arm-Release,Test-ChromeOS-Clang-Chromebook_C100p-GPU-MaliT764-arm-Debug Change-Id: I4a5c50b6ab7683512776c70aec6e9a75a0999787 Reviewed-on: https://skia-review.googlesource.com/13464 Reviewed-by: Herb Derby <herb@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
* remove code directly involved in lock/unlock in SkPixelRefGravatar Mike Reed2017-04-13
| | | | | | | | Bug: skia:6481 Change-Id: I3c301ad42b082e04e233aa32d77862036fe998fa Reviewed-on: https://skia-review.googlesource.com/13463 Reviewed-by: Matt Sarett <msarett@google.com> Commit-Queue: Mike Reed <reed@google.com>
* make picture-imagegenerator more robust on requested infosGravatar Mike Reed2017-04-13
| | | | | | | | | | This new unittest would assert before this fix. Bug: skia:6501 Change-Id: I351ad03f29bccc054f72bfcb838174830dbd008c Reviewed-on: https://skia-review.googlesource.com/13413 Commit-Queue: Mike Reed <reed@google.com> Reviewed-by: Matt Sarett <msarett@google.com>
* Further unify logic in readSurfacePixels and writeSurfacePixelsGravatar Brian Osman2017-04-13
| | | | | | | | | | | | | | | | | | | | Hoist simple failure checks to the beginning, apply logic in the same sequence in both functions. Factor out some common logic. Remove redundant checking inside other helper functions (that are only used from here). I'm inching towards better sRGB and color-conversion support in these functions, but I don't want that intertwined with the legacy premul software fallback. Next step is probably to diverge at the top-level, and keep the current code as "legacy" versions, with new versions that don't have the fallback logic, only used for configs and color space scenarios where it doesn't matter. Bug: skia:5853 Change-Id: I327be5f0186a16ceda9440670fb0646acaef7766 Reviewed-on: https://skia-review.googlesource.com/13337 Commit-Queue: Brian Osman <brianosman@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com> Reviewed-by: Matt Sarett <msarett@google.com>
* Move GrAtlasGlyphCache over to SkArenaAlloc from SkVarAlloc.Gravatar Herb Derby2017-04-13
| | | | | | | Change-Id: Ibeee7235d30c68be9fd4f68ecf30a65c7ecb78f5 Reviewed-on: https://skia-review.googlesource.com/13020 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Herb Derby <herb@google.com>
* c100p bots: put a bandaid on broken ThumbGravatar Mike Klein2017-04-13
| | | | | | | | | | | BUG=skia:6471 CQ_INCLUDE_TRYBOTS=skia.primary:Test-ChromeOS-Clang-Chromebook_C100p-GPU-MaliT764-arm-Release,Test-ChromeOS-Clang-Chromebook_C100p-GPU-MaliT764-arm-Debug Change-Id: I3faa28c69510fbb33d40abbfb59bf3a7ba5ddb77 Reviewed-on: https://skia-review.googlesource.com/13407 Reviewed-by: Kevin Lubick <kjlubick@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
* remove old-pixelref-api flag from androidGravatar Mike Reed2017-04-13
| | | | | | | | | call sites have been updated in master Bug: skia:6481 Change-Id: I7f341cb511126a71fe3077bed3eaa55dc7ff76c3 Reviewed-on: https://skia-review.googlesource.com/13440 Reviewed-by: Mike Reed <reed@google.com>
* disable this test on NoGPU botsGravatar Mike Klein2017-04-13
| | | | | | | | | | It fails at line 336: REPORTER_ASSERT(reporter, configs[i]->getBackend().equals(expectedConfigs[i].backend)); Change-Id: Iab71162b9d7b5454d200cb373026992eac92adf8 Reviewed-on: https://skia-review.googlesource.com/13406 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
* Add infra/bots/Makefile to simplify test/trainGravatar Eric Boren2017-04-13
| | | | | | | | | | | | | | | eg. $ make test or $ make train Update docs accordingly. Bug: skia: Change-Id: I069fa686833f0a78f398bd85eb8bb44025cae0d5 Reviewed-on: https://skia-review.googlesource.com/13327 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* make ShadowUtils unit test pass on no-GPU buildsGravatar Mike Klein2017-04-13
| | | | | | | | | It thinks it should fail, but it doesn't. Change-Id: Ie97f57dd7a9040a817b59186b2f8fa704bdda3e9 Reviewed-on: https://skia-review.googlesource.com/13404 Reviewed-by: Jim Van Verth <jvanverth@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
* Use a simpler method for calculating atanGravatar Herb Derby2017-04-13
| | | | | | | Change-Id: I675851350c02c85f3f212c214766b3e8a6761dc9 Reviewed-on: https://skia-review.googlesource.com/13402 Reviewed-by: Florin Malita <fmalita@chromium.org> Commit-Queue: Herb Derby <herb@google.com>
* re-disable lld on MSAN botsGravatar Mike Klein2017-04-13
| | | | | | | | | | | | | When using lld, calls to clock_gettime() end up in the real libc clock_gettime(). When using ld, calls to clock_gettime() end up in __interceptor_clock_gettime(). Don't know why yet. Change-Id: I41f54218eb961e07ed2ef2c37b864e9aeeb49662 Reviewed-on: https://skia-review.googlesource.com/13403 Reviewed-by: Kevin Lubick <kjlubick@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
* Change infra recipe module to use the Go assetGravatar Eric Boren2017-04-13
| | | | | | | | Bug: skia: Change-Id: Ib2fa0b971f201ea2f16afd0467d7882b66b4047c Reviewed-on: https://skia-review.googlesource.com/13331 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Remove GrRenderTargetOpList::discard & move functionality to ↵Gravatar Robert Phillips2017-04-13
| | | | | | | | | GrRenderTargetContext Change-Id: If84ffb7b01543a44179837d683c0a4c004e52f63 Reviewed-on: https://skia-review.googlesource.com/13335 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
* remove dead code around replayClipsGravatar Mike Reed2017-04-13
| | | | | | | | Bug: skia: Change-Id: I208bf41846265524d86f65de660311199fefc158 Reviewed-on: https://skia-review.googlesource.com/13338 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com>
* Set skia_enable_gpu=false on MSAN builds.Gravatar Mike Klein2017-04-13
| | | | | | | | | | | | This should get us past the crash at startup. The next problem I'd expect to see, if any, should involve HumanizeMS() and chrono::steady_clock::now(). Change-Id: I14dbe6bedd6939c1a355db8dba74ab0853db7706 Reviewed-on: https://skia-review.googlesource.com/13401 Commit-Queue: Mike Klein <mtklein@chromium.org> Reviewed-by: Kevin Lubick <kjlubick@google.com>
* Store the GrBackend we are using on GrContextGravatar Greg Daniel2017-04-13
| | | | | | | | | | | | | | As we start adding more functionality to external clients and passing data around, we will need to know what backend we are working with at a layer higher that GrGpu. TBR=bsalomon@google.com Bug: skia: Change-Id: Ifee527ea0c1046469401a39e0500f84a46bb3081 Reviewed-on: https://skia-review.googlesource.com/13275 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com>
* Make all GrRenderTargetOpList ops store a GrRenderTarget pointerGravatar Robert Phillips2017-04-13
| | | | | | | Change-Id: I39652b4b8b2ba96b5206304dfc51395283c3fe16 Reviewed-on: https://skia-review.googlesource.com/13332 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
* try -BGravatar Mike Klein2017-04-13
| | | | | | Change-Id: I4e03d668197d76feb71113f47bae24fdd2fc6b6f Reviewed-on: https://skia-review.googlesource.com/13334 Reviewed-by: Kevin Lubick <kjlubick@google.com>
* Do sRGB premul/unpremul on the GPUGravatar Brian Osman2017-04-13
| | | | | | | | | | | | | | | | | | Previously, the early check would decide that sRGB pixel configs were okay (because they're 8888-unorm). Then we'd go to make the effect and decide that we didn't want them to work. This led to the software fallback. The software fallback was obviously slower, but also doing non-linear premul/unpremul operations. Eventually, whether or not the premul is linear should be dictated by the destination color space, but for now, this is an improvement (and only affects the one GM that tests this feature). Bug: skia: Change-Id: I0cf1ad5a7f552135ac1da728c6db2977652a433b Reviewed-on: https://skia-review.googlesource.com/13321 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
* Access EGL_KHR_fence_sync via eglGetProcAddressGravatar Chris Dalton2017-04-13
| | | | | | | | | Bug: skia:6497 Change-Id: Icf04de6b1facc94e8c45085575cb45189c6c649c Reviewed-on: https://skia-review.googlesource.com/13306 Reviewed-by: Kevin Lubick <kjlubick@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Chris Dalton <csmartdalton@google.com>
* Update clang_linux to version 4Gravatar Kevin Lubick2017-04-13
| | | | | | | | Bug: skia: Change-Id: Iab4fcd7b8f17fa7bf478e879a325759e0c622b1a Reviewed-on: https://skia-review.googlesource.com/13120 Commit-Queue: Mike Klein <mtklein@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
* remove unused fURI from pixelrefGravatar Mike Reed2017-04-13
| | | | | | | | Bug: skia: Change-Id: Ib25569862063b765b823b68ffdc22349a606f940 Reviewed-on: https://skia-review.googlesource.com/13324 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com>
* locking methods should only be virtual in legacy modeGravatar Mike Reed2017-04-12
| | | | | | | | | | | However, they should not be pure-virtual, otherwise the client can't migrate to the new api independent of undef-ing the flag. Bug: skia:6481 Change-Id: I3c9107b145c4c53297891224946b080ce729f9b0 Reviewed-on: https://skia-review.googlesource.com/13272 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com>
* Fix cubic KLM solvingGravatar csmartdalton2017-04-12
| | | | | | | | | | | | Fixes calc_inverse_transpose_power_basis_matrix() to look for the determinant with the largest ABSOLUTE VALUE. BUG=skia: Change-Id: I70e26cc7ab9955d2e094fb7581f0ec9f5e0ae2d9 Reviewed-on: https://skia-review.googlesource.com/13081 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Chris Dalton <csmartdalton@google.com>
* Add job for Acer chromebook (TegraK1)Gravatar Kevin Lubick2017-04-12
| | | | | | | | | | | | | Known issue: Nanobench fails with /home/chronos/user/bin/nanobench: symbol lookup error: /home/chronos/user/bin/nanobench: undefined symbol: eglCreateSyncKHR Bug: skia:6468 NOTRY=true Change-Id: Idb1291c2da835d093826022f4cab4069e28c5964 Reviewed-on: https://skia-review.googlesource.com/13188 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
* Add Win IntelHD4400 bot.Gravatar Ben Wagner2017-04-12
| | | | | | | | | | Bug: skia:6488 NOTRY=true Change-Id: Icab4bf478925fb48f7facd6c9883a2753d04aaac Reviewed-on: https://skia-review.googlesource.com/13191 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* jumper, bilinear and bicubic sampling stagesGravatar Mike Klein2017-04-12
| | | | | | | | | | | | | | | | | | | | This splits SkImageShaderContext into three parts: - SkJumper_GatherCtx: always, already done - SkJumper_SamplerCtx: when bilinear or bicubic - MiscCtx: other little bits (the matrix, paint color, tiling limits) Thanks for the snazzy allocator that allows this Herb! Both SkJumper and SkRasterPipeline_opts.h should be speaking all the same types now. I've copied the comments about bilinear/bicubic to SkJumper with little typo fixes and clarifications. Change-Id: I4ba7b7c02feba3f65f5292169a22c060e34933c6 Reviewed-on: https://skia-review.googlesource.com/13269 Reviewed-by: Herb Derby <herb@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
* Allow including of vulkan.h as a required public api when using vulkan.Gravatar Greg Daniel2017-04-12
| | | | | | | | Bug: skia: Change-Id: I3ac382edc12144dcda9ef168f752b14878e0e5ac Reviewed-on: https://skia-review.googlesource.com/13262 Commit-Queue: Mike Klein <mtklein@chromium.org> Reviewed-by: Mike Klein <mtklein@chromium.org>
* android-only api to reset the fields of a pixelrefGravatar Mike Reed2017-04-12
| | | | | | | | | BUG=skia:6494 Change-Id: I97c8a93849a13f0c5fd57c8a150ecc2ce363d8f3 Reviewed-on: https://skia-review.googlesource.com/13250 Reviewed-by: Matt Sarett <msarett@google.com> Commit-Queue: Mike Reed <reed@google.com>
* Update Android autoroller documentationGravatar Ravi Mistry2017-04-12
| | | | | | | | | | | | BUG=skia:5538 NOTRY=true DOCS_PREVIEW= https://skia.org/?cl=13266 Change-Id: I76fa8db5ea9916fab2ef0f8058157dac96d52517 Change-Id: I76fa8db5ea9916fab2ef0f8058157dac96d52517 Reviewed-on: https://skia-review.googlesource.com/13266 Reviewed-by: Matt Sarett <msarett@google.com> Commit-Queue: Ravi Mistry <rmistry@google.com>
* Add jobs for Samsung Chromebook PlusGravatar Kevin Lubick2017-04-12
| | | | | | | | | | | Also refactor the cipd package used to compile these, as it can be reused. Bug: skia: NOTRY=true Change-Id: I53cf5b7fdfcb8cafd50791a0beaaf557fafa65c3 Reviewed-on: https://skia-review.googlesource.com/13252 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* fix drawDRRect for fuzzerGravatar Cary Clark2017-04-12
| | | | | | | | | | make assert abort instead Bug: skia:6450 Change-Id: I23ff51124fa8f069f2c7e5260f800017d7475d46 Reviewed-on: https://skia-review.googlesource.com/13197 Commit-Queue: Cary Clark <caryclark@google.com> Reviewed-by: Hal Canary <halcanary@google.com>
* make all gather_*() use SkJumper_GatherCtxGravatar Mike Klein2017-04-12
| | | | | | | | | | | SkJumper_GatherCtx is a prefix of SkImageShaderContext, so this is a no-op. It helps to keep things straight, and I do want to split apart the GatherCtx from a new SamplingCtx. Change-Id: I9c5f436b096624c2809e1f810e9bcd6c6b00b883 Reviewed-on: https://skia-review.googlesource.com/13264 Reviewed-by: Mike Klein <mtklein@chromium.org> Commit-Queue: Mike Klein <mtklein@chromium.org>
* Add/edit README files for infra/recipesGravatar Eric Boren2017-04-12
| | | | | | | | | | | Bug: skia:6473 Change-Id: I050df47a5a3f633263f0ca6258e83e96b78d7957 Reviewed-on: https://skia-review.googlesource.com/13123 Commit-Queue: Eric Boren <borenet@google.com> Reviewed-by: Stephan Altmueller <stephana@google.com> Reviewed-by: Mike Klein <mtklein@chromium.org> Reviewed-by: Kevin Lubick <kjlubick@google.com> Reviewed-by: Ben Wagner <benjaminwagner@google.com>