Every runtime-effect shader should use main(float2 coords). The two-argument version is, thankfully, not documented in AGSL, so it's not public. The color argument is only used in a few places and is never actually necessary.
-- You received this message because: 1. The project was configured to send all issue notifications to this address
Public runtime shaders now require a signature of `half4 main(float2)`. We still have a handful of private shaders which use the input color, and keep the `half4 main(float2, half4)` signature. Those can be cleaned up in followup CLs.
This shouldn't make any difference; a*b and b*a are the same. However, this order gives us SkSL that looks closer to the original code. This might give us better results on older Pixel 2/3 devices on Vulkan. These devices started noise/garbage pixels after http://review.skia.org/580595.