Changeset 64197 in webkit


Ignore:
Timestamp:
Jul 28, 2010 4:09:06 AM (14 years ago)
Author:
Nikolas Zimmermann
Message:

2010-07-28 Nikolas Zimmermann <​nzimmermann@rim.com>

Reviewed by Dirk Schulze.

SVGMaskElement/SVGForeignObjectElement should not inherit from SVGURIReference
​https://bugs.webkit.org/show_bug.cgi?id=43113

Remove wrong SVGURIReference inheritance for SVGMaskElement/SVGForeignObjectElement. Old copy/paste bug.

  • svg/SVGForeignObjectElement.cpp: (WebCore::SVGForeignObjectElement::synchronizeProperty):
  • svg/SVGForeignObjectElement.h:
  • svg/SVGMaskElement.cpp: (WebCore::SVGMaskElement::SVGMaskElement): (WebCore::SVGMaskElement::parseMappedAttribute): (WebCore::SVGMaskElement::svgAttributeChanged): (WebCore::SVGMaskElement::synchronizeProperty):
  • svg/SVGMaskElement.h:
Location:
trunk/WebCore
Files:
5 edited

Legend:

Unmodified
Added
Removed

  • trunk/WebCore/ChangeLog

    r64196 r64197  
     12010-07-28  Nikolas Zimmermann  <nzimmermann@rim.com>
     2
     3        Reviewed by Dirk Schulze.
     4
     5        SVGMaskElement/SVGForeignObjectElement should not inherit from SVGURIReference
     6        https://bugs.webkit.org/show_bug.cgi?id=43113
     7
     8        Remove wrong SVGURIReference inheritance for SVGMaskElement/SVGForeignObjectElement. Old copy/paste bug.
     9
     10        * svg/SVGForeignObjectElement.cpp:
     11        (WebCore::SVGForeignObjectElement::synchronizeProperty):
     12        * svg/SVGForeignObjectElement.h:
     13        * svg/SVGMaskElement.cpp:
     14        (WebCore::SVGMaskElement::SVGMaskElement):
     15        (WebCore::SVGMaskElement::parseMappedAttribute):
     16        (WebCore::SVGMaskElement::svgAttributeChanged):
     17        (WebCore::SVGMaskElement::synchronizeProperty):
     18        * svg/SVGMaskElement.h:
     19
    1202010-07-28  Nikolas Zimmermann  <nzimmermann@rim.com>
    221

  • trunk/WebCore/svg/SVGForeignObjectElement.cpp

    r63531 r64197  
    127127    else if (SVGExternalResourcesRequired::isKnownAttribute(attrName))
    128128        synchronizeExternalResourcesRequired();
    129     else if (SVGURIReference::isKnownAttribute(attrName))
    130         synchronizeHref();
    131129}
    132130

  • trunk/WebCore/svg/SVGForeignObjectElement.h

    r62488 r64197  
    3434                                    public SVGTests,
    3535                                    public SVGLangSpace,
    36                                     public SVGExternalResourcesRequired,
    37                                     public SVGURIReference {
     36                                    public SVGExternalResourcesRequired {
    3837    public:
    3938        SVGForeignObjectElement(const QualifiedName&, Document*);

  • trunk/WebCore/svg/SVGMaskElement.cpp

    r62488 r64197  
    4141SVGMaskElement::SVGMaskElement(const QualifiedName& tagName, Document* doc)
    4242    : SVGStyledLocatableElement(tagName, doc)
    43     , SVGURIReference()
    4443    , SVGTests()
    4544    , SVGLangSpace()
    … …  
    8180        setHeightBaseValue(SVGLength(LengthModeHeight, attr->value()));
    8281    else {
    83         if (SVGURIReference::parseMappedAttribute(attr))
    84             return;
    8582        if (SVGTests::parseMappedAttribute(attr))
    8683            return;
    … …  
    109106        || attrName == SVGNames::maskUnitsAttr
    110107        || attrName == SVGNames::maskContentUnitsAttr
    111         || SVGURIReference::isKnownAttribute(attrName)
    112108        || SVGTests::isKnownAttribute(attrName)
    113109        || SVGLangSpace::isKnownAttribute(attrName)
    … …  
    127123        synchronizeY();
    128124        synchronizeExternalResourcesRequired();
    129         synchronizeHref();
    130125        return;
    131126    }
    … …  
    141136    else if (SVGExternalResourcesRequired::isKnownAttribute(attrName))
    142137        synchronizeExternalResourcesRequired();
    143     else if (SVGURIReference::isKnownAttribute(attrName))
    144         synchronizeHref();
    145138}
    146139

  • trunk/WebCore/svg/SVGMaskElement.h

    r62488 r64197  
    2727#include "SVGStyledLocatableElement.h"
    2828#include "SVGTests.h"
    29 #include "SVGURIReference.h"
    3029
    3130namespace WebCore {
    … …  
    3433
    3534    class SVGMaskElement : public SVGStyledLocatableElement,
    36                            public SVGURIReference,
    3735                           public SVGTests,
    3836                           public SVGLangSpace,
    … …  
    4139        SVGMaskElement(const QualifiedName&, Document*);
    4240        virtual ~SVGMaskElement();
     41
    4342        virtual bool isValid() const { return SVGTests::isValid(); }
    4443
    … …  
    6160        DECLARE_ANIMATED_PROPERTY(SVGMaskElement, SVGNames::heightAttr, SVGLength, Height, height)
    6261
    63         // SVGURIReference
    64         DECLARE_ANIMATED_PROPERTY(SVGMaskElement, XLinkNames::hrefAttr, String, Href, href)
    65 
    6662        // SVGExternalResourcesRequired
    6763        DECLARE_ANIMATED_PROPERTY(SVGMaskElement, SVGNames::externalResourcesRequiredAttr, bool, ExternalResourcesRequired, externalResourcesRequired)
Note: See TracChangeset for help on using the changeset viewer.