Comment on attachment 349809[details]
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=349809&action=review> Source/WebCore/platform/graphics/cg/GradientCG.cpp:144
> + // Find first gradient position to the left of the bounding box
> + int n = CGFloor((boundingBox.origin.x - gradientStart) / width);
> + gradientStart += n * width;
> + if (!(n % 2))
> + flip = false;
I think your first version of this patch where you manually do the left side then the right side was much more understandable than this. I can't convince myself that this code is correct.