[Webkit-unassigned] [Bug 142516] New: background: radial-gradient draws circle instead of ellipse.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=142516

            Bug ID: 142516
           Summary: background: radial-gradient draws circle instead of
                    ellipse.
    Classification: Unclassified
           Product: WebKit
           Version: 528+ (Nightly build)
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebKit Gtk
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: oohashi at opentv.com

Created attachment 248293
  --> https://bugs.webkit.org/attachment.cgi?id=248293&action=review
a reproducible HTML file, result image and a patch for fixing

background: radial-gradient draws circle instead of ellipse.
It always shows circle on our Linux Cairo Platform. I made a patch. Please review.

I attached a file that has a HTML file, image files and a patch file.
- test.html: reproducible HTML file
- Elliptial_Gridients_issue.png: bug (showed CIRCLE incorrectly)
- Elliptial_Gridients_expected.png: expected result that was on Chrome
- GradientCairo.patch: patch file for fixing.

diff  Source/WebCore/platform/graphics/cairo/GradientCairo.cpp.2.6.5 Source/WebCore/platform/graphics/cairo/GradientCairo.c
pp.2.6.5.fixed
58c58
<         m_gradient = cairo_pattern_create_radial(m_p0.x(), m_p0.y(), m_r0, m_p1.x(), m_p1.y(), m_r1);
---
>         m_gradient = cairo_pattern_create_radial(m_p0.x(), m_p0.y() * m_aspectRatio, m_r0, m_p1.x(), m_p1.y() * m_aspectRatio, m_r1);
82a83,86
>     if (m_radial && m_aspectRatio != 1.0) {
>          cairo_matrix_scale(&matrix, 1.0, 1.0 / m_aspectRatio);
>     }
>
Thanks, Osamu

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150310/35431f64/attachment-0002.html>