Changeset 162537 in webkit


Ignore:
Timestamp:
Jan 22, 2014 11:10:51 AM (10 years ago)
Author:
commit-queue@webkit.org
Message:

Refactor calculation of hasRx and hasRy values in SVGPathData
https://bugs.webkit.org/show_bug.cgi?id=127423

Patch by Peter Molnar <pmolnar.u-szeged@partner.samsung.com> on 2014-01-22
Reviewed by Darin Adler.

This is a follow-up on https://bugs.webkit.org/show_bug.cgi?id=127337

  • rendering/svg/SVGPathData.cpp:

(WebCore::updatePathFromRectElement):

Location:
trunk/Source/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r162534 r162537  
     12014-01-22  Peter Molnar  <pmolnar.u-szeged@partner.samsung.com>
     2
     3        Refactor calculation of hasRx and hasRy values in SVGPathData
     4        https://bugs.webkit.org/show_bug.cgi?id=127423
     5
     6        Reviewed by Darin Adler.
     7
     8        This is a follow-up on https://bugs.webkit.org/show_bug.cgi?id=127337
     9
     10        * rendering/svg/SVGPathData.cpp:
     11        (WebCore::updatePathFromRectElement):
     12
    1132014-01-21  Joseph Pecoraro  <pecoraro@apple.com>
    214
  • trunk/Source/WebCore/rendering/svg/SVGPathData.cpp

    r162438 r162537  
    116116    float x = rect->x().value(lengthContext);
    117117    float y = rect->y().value(lengthContext);
    118     bool hasRx = rect->rx().value(lengthContext) > 0;
    119     bool hasRy = rect->ry().value(lengthContext) > 0;
     118    float rx = rect->rx().value(lengthContext);
     119    float ry = rect->ry().value(lengthContext);
     120    bool hasRx = rx > 0;
     121    bool hasRy = ry > 0;
    120122    if (hasRx || hasRy) {
    121         float rx = rect->rx().value(lengthContext);
    122         float ry = rect->ry().value(lengthContext);
    123123        if (!hasRx)
    124124            rx = ry;
Note: See TracChangeset for help on using the changeset viewer.