aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* GM: add test for type1 fontGravatar halcanary2016-09-20
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2350453002 Review-Url: https://codereview.chromium.org/2350453002
* Disable framebuffer fetch on the Adreno5xx devices.Gravatar egdaniel2016-09-20
| | | | | | | | | | This should help with some of the rendering bugs currently in GL for the S7. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2348293004 Review-Url: https://codereview.chromium.org/2348293004
* Fix possible divide by zero in vulkan granularity adjustmentsGravatar egdaniel2016-09-20
| | | | | | | | | This should fix the divide by zero crash on the nexus player bots BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2358613002 Review-Url: https://codereview.chromium.org/2358613002
* Stop closing filled paths in GrShapeGravatar bsalomon2016-09-20
| | | | | | | This can force a copy of the path data and isn't really being taken advantage of. It complicates a forthcoming change to key small paths based on their data rather than gen id. GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2355833002 Review-Url: https://codereview.chromium.org/2355833002
* [SVGDom] Opacity optimizationGravatar fmalita2016-09-20
| | | | | | | | | | Apply opacity as fill/stroke paint alpha instead of saveLayer, when possible. R=robertphillips@google.com,stephana@google.com,reed@google.com GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2353503005 Review-Url: https://codereview.chromium.org/2353503005
* Disable failing unit tests on adreno vulkanGravatar egdaniel2016-09-20
| | | | | | | | | | As mentioned in the attached bug, these failures all seem to be caused by driver issues that have no workarounds at this time. BUG=skia:5777 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2355883002 Review-Url: https://codereview.chromium.org/2355883002
* Use STDArray to for tracked resources in vulkan command bufferGravatar egdaniel2016-09-20
| | | | | | | | | | This is a minor speed improvement to help with the number of times we are freeing and re-allocating the tracked resource arrays BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2338963004 Review-Url: https://codereview.chromium.org/2338963004
* Revert of Skia recipe: Use [depot_tools]/gsutil.py instead of ↵Gravatar borenet2016-09-20
| | | | | | | | | | | | | | | | | | | | | | | | | [depot_tools]/third_party/gsutil/gsutil (patchset #1 id:1 of https://codereview.chromium.org/2298293002/ ) Reason for revert: Broke all nanobench uploads, eg. https://uberchromegw.corp.google.com/i/client.skia.android/builders/Perf-Android-Clang-Nexus6p-CPU-Snapdragon810-arm64-Release-GN_Android/builds/154/steps/Upload%20perf%20results/logs/stdio Original issue's description: > Skia recipe: Use [depot_tools]/gsutil.py instead of [depot_tools]/third_party/gsutil/gsutil > > The latter is being removed in https://codereview.chromium.org/2280023003/ > Is also pinned to 3.25, whereas the former is updated more often. > > BUG=skia: > GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2298293002 > > Committed: https://skia.googlesource.com/skia/+/f15590f803596f51d0950994b2aeb623d2d1fc24 TBR=hinoka@google.com NOTREECHECKS=true NOTRY=true BUG=skia: Review-Url: https://codereview.chromium.org/2355943002
* GN: add skia_enable_gpu (default true)Gravatar mtklein2016-09-20
| | | | | | | | | | | This lets us replace the NoGPU bot. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2356853002 NOTREECHECKS=true Review-Url: https://codereview.chromium.org/2356853002
* GN: compiler_prefix -> cc_wrapperGravatar mtklein2016-09-20
| | | | | | | | | | | | | | | | I was a bit mixed up thinking Chrome called this feature compiler_prefix. They actually call it cc_wrapper. So let's switch to their name before we get too used to this. No bots are affected. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2355003002 DOCS_PREVIEW= https://skia.org/?cl=2355003002 NOTREECHECKS=true Review-Url: https://codereview.chromium.org/2355003002
* Snazzier dump log step on Android bots.Gravatar mtklein2016-09-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (The failing NexusPlayer bot is a demo.) This should make stack traces more useful, turning this sort of thing 09-20 11:29:39.536 2978 2978 F DEBUG : #00 pc 00970fd0 /data/local/tmp/dm into something like this 09-20 11:29:39.536 2978 2978 F DEBUG : #00 pc 00970fd0 adjust_bounds_to_granularity(SkIRect*, SkIRect const&, VkExtent2D const&, int, int) at /b/work/skia/out/Build-Ubuntu-Clang-x86-Debug-GN_Android_Vulkan/Debug/../../../src/gpu/vk/GrVkGpu.cpp:1803 /data/local/tmp/dm Some bots like the S7 already have good enough stack traces, e.g. 09-20 11:35:12.567 936 936 F DEBUG : #00 pc 00000000000bed6c /system/vendor/lib64/hw/vulkan.msm8996.so (_ZN13QglManagedBuf14ConfirmEntriesEv+108) 09-20 11:35:12.567 936 936 F DEBUG : #01 pc 00000000000b098c /system/vendor/lib64/hw/vulkan.msm8996.so (_ZN9QglBltLib6FillHwEPK10QglBltFillPK15QglBltColorFillPK15QglBltDepthFillPjP12QglBltStatusSB_+588) 09-20 11:35:12.567 936 936 F DEBUG : #02 pc 00000000000b23bc /system/vendor/lib64/hw/vulkan.msm8996.so (_ZN9QglBltLib9FillImageEPK10QglBltFillPK15QglBltColorFillPK15QglBltDepthFillPjP12QglBltStatusSB_+348) 09-20 11:35:12.567 936 936 F DEBUG : #03 pc 000000000009bb00 /system/vendor/lib64/hw/vulkan.msm8996.so (_ZN16A5xCommandBuffer22PerformConditionalFillEiiP10QglBltFillPK15QglBltColorFillP15QglBltDepthFillP12QglBltStatusS8_+256) 09-20 11:35:12.567 936 936 F DEBUG : #04 pc 000000000009c0c0 /system/vendor/lib64/hw/vulkan.msm8996.so (_ZN16A5xCommandBuffer23HwWriteSubpassInitClearEP10QglBltFillPK15QglBltColorFillP15QglBltDepthFillP12QglBltStatusS8_+224) 09-20 11:35:12.567 936 936 F DEBUG : #05 pc 0000000000072610 /system/vendor/lib64/hw/vulkan.msm8996.so (_ZN16QglCommandBuffer18WriteSubpassClearsEv+464) 09-20 11:35:12.567 936 936 F DEBUG : #06 pc 0000000000073ae0 /system/vendor/lib64/hw/vulkan.msm8996.so (_ZN16QglCommandBuffer12BeginSubpassEv+32) 09-20 11:35:12.567 936 936 F DEBUG : #07 pc 0000000000063120 /system/vendor/lib64/hw/vulkan.msm8996.so (vkCmdBeginRenderPass+224) 09-20 11:35:12.567 936 936 F DEBUG : #08 pc 0000000000635f60 /data/local/tmp/nanobench (_ZN24GrVkPrimaryCommandBuffer15beginRenderPassEPK7GrVkGpuPK14GrVkRenderPassjPK12VkClearValueRK16GrVkRenderTargetRK7SkIRectb+132) or 09-20 11:42:24.557 937 937 F DEBUG : backtrace: 09-20 11:42:24.557 937 937 F DEBUG : #00 pc 0000000000069404 /system/lib64/libc.so (tgkill+8) 09-20 11:42:24.557 937 937 F DEBUG : #01 pc 0000000000066b94 /system/lib64/libc.so (pthread_kill+68) 09-20 11:42:24.557 937 937 F DEBUG : #02 pc 0000000000023a28 /system/lib64/libc.so (raise+28) 09-20 11:42:24.557 937 937 F DEBUG : #03 pc 000000000001e358 /system/lib64/libc.so (abort+60) 09-20 11:42:24.557 937 937 F DEBUG : #04 pc 000000000076430c /data/local/tmp/dm (_Z17sk_abort_no_printv+8) These won't be affected. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2351243002 NOTREECHECKS=true Review-Url: https://codereview.chromium.org/2351243002
* Cleanup some sRGB TODOs, mostly around setting GrPaint flags correctlyGravatar brianosman2016-09-20
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2358433004 Review-Url: https://codereview.chromium.org/2358433004
* Support use of non-coherent memory allocations in Vulkan.Gravatar jvanverth2016-09-20
| | | | | | | BUG=skia:5034 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2348523002 Review-Url: https://codereview.chromium.org/2348523002
* Stop flattening GrCoordTransforms in parent GrFragmentProcessors.Gravatar bsalomon2016-09-20
| | | | | | | | This changes moves to a model that iterates over GrCTs in a GrFP hierarchy when inserting transformations by GrGLSLPrimitiveProcessors. GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2339203002 Committed: https://skia.googlesource.com/skia/+/d91237ee051523f439238042674ade99207fe4a6 Review-Url: https://codereview.chromium.org/2339203002
* Workaround for Adreno INITIALIZATION_FAILED bugGravatar egdaniel2016-09-20
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2352083002 Review-Url: https://codereview.chromium.org/2352083002
* Fix a memory leak reported by clang static analyzer.Gravatar sdefresne2016-09-20
| | | | | | | | | | | | | The CFStringCreateWithCString method respect the "Create Rule" [1] regarding the ownership of the returned reference. This means that the object needs to be deallocated explicitly by calling CFRelease. [1]: https://developer.apple.com/library/content/documentation/CoreFoundation/Conceptual/CFMemoryMgmt/Concepts/Ownership.html#//apple_ref/doc/uid/20001148-103029 BUG=648210 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2354963002 Review-Url: https://codereview.chromium.org/2354963002
* abstract name of clipping ops, to transtion to a more restricted setGravatar reed2016-09-20
| | | | | | | | | | | SkRegion::Op --> SkCanvas::ClipOp (alias) --> SkClipOp pre-CL needed in chrome : https://codereview.chromium.org/2355583002/ BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2355483002 Review-Url: https://codereview.chromium.org/2355483002
* skia_enable_android_framework_defines in all code, public and private, ↵Gravatar mtklein2016-09-20
| | | | | | | | | | | | | | | optional and required. The bug fixed here can be demonstrated by $ ninja -C out pdf -v before and after this CL. Before you won't see the Android defines, afterwards you do. Previously the defines applied only privately to Skia, and only to its non-optional components. TBR=reed@google.com BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2356783002 Review-Url: https://codereview.chromium.org/2356783002
* Auto-generate much of the help text for --config to reduce maintenanceGravatar brianosman2016-09-20
| | | | | | | | | | The list of configs, and the details about all predefined configs are now built directly from the gPredefinedConfigs array. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2354943002 Review-Url: https://codereview.chromium.org/2354943002
* Fix Win toolchain asset, fix scriptGravatar borenet2016-09-20
| | | | | | | BUG=skia:5764 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2351033004 Review-Url: https://codereview.chromium.org/2351033004
* Harden LinearGradient4fContextGravatar fmalita2016-09-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | Three distinct issues exposed by Fuzzer: 1) truncating the flags to 8 bits may trip the assert in SkToU8 Since clients can pass in any garbage, we cannot assume it fits in 8 bits. Just static_cast<> instead. 2) vertical gradients with dx == -0.0 don't trigger interval inversion. For dx < 0 we want reversed intervals. But, alas, -0.0 < 0 == false (thanks, Ob^WIEEE 754!). Use signbit() instead. 3) half closed range checking needs to be inverted for reversed intervals. Normally we check for [p0, p1), but for reversed intervals (p1 < p0) we want to check for [p1, p0), and not (p1, p0]. Adjust the logic accordingly. BUG=skia:5647 R=reed@google.com,kjlubick@google.com GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2349153004 Review-Url: https://codereview.chromium.org/2349153004
* Blacklist incomplete pngs on CGGravatar msarett2016-09-20
| | | | | | | BUG=skia:5774 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2351083002 Review-Url: https://codereview.chromium.org/2351083002
* SkASSERT firing because pointer wraps negative.Gravatar cblume2016-09-19
| | | | | | | | | | | | | | | | | Some pointers are being cast to intptr_t so they can be used with SkAlign8(). However, a large pointer value might become a negative integer since intptr_t is signed. When comparing these intptr_ts, we expect the larger pointer value to be greater. But it might be so large that it becomes negative, causing it to be less than. A SkASSERT is firing for this exact reason. BUG=648452 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2353453004 Review-Url: https://codereview.chromium.org/2353453004
* Add ability to pick a different gamut in GPU f16 or sRGB configsGravatar brianosman2016-09-19
| | | | | | | | | | | | | | | | | | | | | | | For now, the only options are sRGB or WideGamutRGB Basically, the color option to the gpu config is now of the form: 8888|srgb[_gamut]|f16[_gamut] color=8888 still implies legacy behavior srgb implies 8-bit gamma-correct rendering (via sRGB format) f16 implies 16-bit gamma-correct rendering (via F16 format) Either of the last two options can then optionally include a gamut specifier, either _srgb or _wide. _srgb selects the (default) sRGB gamut _wide selects the Adobe Wide Gamut RGB gamut, which is nice for testing, in that it's significantly wider than sRGB, so rendering differences are obvious. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2350003003 Review-Url: https://codereview.chromium.org/2350003003
* Reland of Run SVGs through nanobench on all Perf bots (patchset #1 id:1 of ↵Gravatar rmistry2016-09-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/2348563003/ ) Reason for revert: skbug/5770 has been fixed. Trying out trybots to see if this can be relanded. Original issue's description: > Revert of Run SVGs through nanobench on all Perf bots (patchset #5 id:120001 of https://codereview.chromium.org/2343903002/ ) > > Reason for revert: > Causing failures: > https://uberchromegw.corp.google.com/i/client.skia/builders/Perf-Mac-Clang-MacMini4.1-GPU-GeForce320M-x86_64-Debug-GN/builds/812 > > Original issue's description: > > Run SVGs through nanobench on all Perf bots. > > > > Tested with: > > python infra/bots/recipes.py run --workdir=../ swarm_trigger path_config=kitchen buildername=Perf-Ubuntu-GCC-ShuttleA-GPU-GTX660-x86_64-Release-GN-Trybot mastername=client.skia slavename=skiabot-linux-swarm-010 buildnumber=1 revision=d8c2476a8b1e1e1a1771b17e8dd4db8645914f8c issue=2343903002 rietveld=https://codereview.chromium.org patchset=80001 > > > > Example output is here: > > https://luci-milo.appspot.com/swarming/task/3147c9789b2d0410 > > Nanobench did run on SVGs. > > > > BUG=skia:5757 > > GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2343903002 > > > > Committed: https://skia.googlesource.com/skia/+/1c58dcb909642454d91ba36f71e55597511b5d52 > > TBR=jcgregorio@google.com,fmalita@chromium.org,mtklein@google.com > # Skipping CQ checks because original CL landed less than 1 days ago. > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG=skia:5757 > > Committed: https://skia.googlesource.com/skia/+/be7e3c0db3caa7a86ae233a6bd26281817b83882 TBR=jcgregorio@google.com,fmalita@chromium.org,mtklein@google.com # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=skia:5757 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2350233002 Review-Url: https://codereview.chromium.org/2350233002
* Create GrVkCopyPipeline classGravatar egdaniel2016-09-19
| | | | | | | | | | This creates a special subclass of GrVkPipeline that we can use for doing copies on draws. Currently this is not hooked into the system. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2353563003 Review-Url: https://codereview.chromium.org/2353563003
* remove dead code around SK_ENABLE_CLIP_QUICKREJECTGravatar reed2016-09-19
| | | | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2349383002 TBR= Review-Url: https://codereview.chromium.org/2349383002
* Add FrameworkDefs bot to the CQ.Gravatar mtklein2016-09-19
| | | | | | | | | | Word is that this bot only exists so it can be on the CQ... but we forgot to actually put it on the CQ until last week. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2352703002 Review-Url: https://codereview.chromium.org/2352703002
* Skia recipe: Use [depot_tools]/gsutil.py instead of ↵Gravatar hinoka2016-09-19
| | | | | | | | | | | | [depot_tools]/third_party/gsutil/gsutil The latter is being removed in https://codereview.chromium.org/2280023003/ Is also pinned to 3.25, whereas the former is updated more often. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2298293002 Review-Url: https://codereview.chromium.org/2298293002
* Split SkFontConfigInterface globals and factory.Gravatar bungeman2016-09-19
| | | | | | | | | | Chromium needs to be able to set up their build such that the globals continue existing but the SkFontMgr::Factory can be defined separately. GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2346333002 CQ_INCLUDE_TRYBOTS=master.client.skia.compile:Build-Ubuntu-GCC-x86_64-Release-CMake-Trybot,Build-Mac-Clang-x86_64-Release-CMake-Trybot Review-Url: https://codereview.chromium.org/2346333002
* Add adb support to skpbenchGravatar csmartdalton2016-09-19
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2350003002 Review-Url: https://codereview.chromium.org/2350003002
* Remove unnecessary assert in GrBatchAtlas.Gravatar jvanverth2016-09-19
| | | | | | | BUG=skia:5770 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2354653002 Review-Url: https://codereview.chromium.org/2354653002
* skpbenchGravatar csmartdalton2016-09-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | skpbench is a benchmarking suite for skps that aims to generate 100% repeatable results. The initial commit consists of three parts: skpbench A minimalist program whose sole purpose is to open an skp file, benchmark it on a single config, and exit. No tiling, looping, or other fanciness is used; it just draws the skp whole into a size- matched render target and syncs the GPU after each draw. Limiting the entire process to a single config/skp pair helps to keep the results repeatable. skpbench.py A wrapper to execute the skpbench binary with various configs and skps. It also monitors the output in order to filter out and re-run results with an unacceptable stddev. In the future this script will lock down and monitor clocks and temperatures. parseskpbench.py A utility for parsing skpbench output into a spreadsheet. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2341823002 Review-Url: https://codereview.chromium.org/2341823002
* SkPDF: SkBitSet gets reset() to make drop() better.Gravatar halcanary2016-09-19
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2354583002 Review-Url: https://codereview.chromium.org/2354583002
* Move vulkan spir-v compile call to GrVkUtilGravatar egdaniel2016-09-19
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2355493002 Review-Url: https://codereview.chromium.org/2355493002
* Refactor vulkan buffer mapping and unmappingGravatar egdaniel2016-09-19
| | | | | | | | | | | | | | A lot of this is so we don't have duplicated code in both the map/unmap and updateData functions of GrVkBuffer. Also there were slightly differences in how we handled various things in the two cases that this now unifies. Also I added a barrier after the vkUpdateBuffer call which I believe was missing. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2344323002 Review-Url: https://codereview.chromium.org/2344323002
* Revert of Stop flattening GrCoordTransforms in parent GrFragmentProcessors. ↵Gravatar bsalomon2016-09-19
| | | | | | | | | | | | | | | | | | | | | | | | (patchset #3 id:40001 of https://codereview.chromium.org/2339203002/ ) Reason for revert: Crashing blink https://codereview.chromium.org/2351743002/ Original issue's description: > Stop flattening GrCoordTransforms in parent GrFragmentProcessors. > > This changes moves to a model that iterates over GrCTs in a GrFP hierarchy when inserting transformations by GrGLSLPrimitiveProcessors. > GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2339203002 > > Committed: https://skia.googlesource.com/skia/+/d91237ee051523f439238042674ade99207fe4a6 TBR=egdaniel@google.com,robertphillips@google.com # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review-Url: https://codereview.chromium.org/2351753002
* To prep for removing CMake bots, take them off the CQ and presubmit.Gravatar mtklein2016-09-19
| | | | | | | | | | | | | | | | | | GN is now far more capable than our CMake builds. Time to officially end that experiment. Reminder how this lands: 1) this CL 2) remove the bots from the masters 3) remove the CMake recipe files 4) remove the CMake files I'm willing to wait on 4) until we can do GN -> CMake generation, but I don't want the bots to hold us back. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2350073002 Review-Url: https://codereview.chromium.org/2350073002
* Stop flattening GrCoordTransforms in parent GrFragmentProcessors.Gravatar bsalomon2016-09-19
| | | | | | | This changes moves to a model that iterates over GrCTs in a GrFP hierarchy when inserting transformations by GrGLSLPrimitiveProcessors. GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2339203002 Review-Url: https://codereview.chromium.org/2339203002
* GN: remove old Android recipe code.Gravatar mtklein2016-09-19
| | | | | | | | | | | All the Android builders, testers, and perfers are on gn_android_flavor now. Not as scary as it looks... all the big line count changes are deletes. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2352653002 Review-Url: https://codereview.chromium.org/2352653002
* add 'g' to fatbitsGravatar reed2016-09-19
| | | | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2350613002 TBR= Review-Url: https://codereview.chromium.org/2350613002
* Delete unused coverage recipe code.Gravatar mtklein2016-09-19
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2353483003 Review-Url: https://codereview.chromium.org/2353483003
* Add F16 to the list of allowed formats in GaussianBlurGravatar brianosman2016-09-19
| | | | | | | | | Fixes asserts that fire in some GMs for gpuf16 config BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2350723002 Review-Url: https://codereview.chromium.org/2350723002
* [SVGDom] Only apply opacity when neededGravatar fmalita2016-09-19
| | | | | | | | | Opacity: 1 is a no-op, but it shows up quite a bit in existing documents. R=robertphillips@google.com,stephana@google.com,reed@google.com GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2349143002 Review-Url: https://codereview.chromium.org/2349143002
* Remove release asserts.Gravatar cblume2016-09-19
| | | | | | | | | | A handful of changes were added to investigate a bug occurring rarely. The bug has been found and those changes can be reverted. BUG=643845 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2347423002 Review-Url: https://codereview.chromium.org/2347423002
* fix fuzzer bugsGravatar caryclark2016-09-19
| | | | | | | | | | | | Add a couple more cases where Op() fails and returns false when the out of range input values make the internal numeric unstable. TBR=reed@google.com BUG=647834, 648068 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2348263002 Review-Url: https://codereview.chromium.org/2348263002
* Whitespace changeGravatar rmistry2016-09-18
| | | | | | | | | TBR= NOTRY=true BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2347393002 Review-Url: https://codereview.chromium.org/2347393002
* Update SKP versionGravatar update-skps2016-09-18
| | | | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= NO_MERGE_BUILDS GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2349953002 Review-Url: https://codereview.chromium.org/2349953002
* [SVGDom] Improved DM sizingGravatar fmalita2016-09-17
| | | | | | | | | | | | Rather than always using a 1000x1000 canvas, observe the SVG intrinsic size when available. If the intrinsic size is < 128x128, scale uniformly. R=robertphillips@google.com,stephana@google.com GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2342313003 Review-Url: https://codereview.chromium.org/2342313003
* Remove SK_DECLARE_LEGACY_CREATE_FROM_FONTDATA.Gravatar bungeman2016-09-16
| | | | | | | | | This is no longer defined by any users, so remove it. TBR=reed This just removes no longer used API. Review-Url: https://codereview.chromium.org/2345353002
generated by cgit v1.2.3 (git 2.39.1) at 2024-06-28 05:42:49 +0000