Revert 79985 - 2011-03-01 Nikolas Zimmermann <nzimmermann@rim.com>

Source/WebCore/svg/SVGColor.cpp

 /*
  * Copyright (C) 2004, 2005 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007 Rob Buis <buis@kde.org>
  * Copyright (C) Research In Motion Limited 2011. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 32 matching lines @@
 void SVGColor::setRGBColor(const String& rgbColor, ExceptionCode& ec)
 {
     Color color = SVGColor::colorFromRGBColorString(rgbColor);
     if (!color.isValid()) {
         ec = SVGException::SVG_INVALID_VALUE_ERR;
         return;
     }
 
     m_color = color;
     m_colorType = SVG_COLORTYPE_RGBCOLOR;
-    setNeedsStyleRecalc();
+    // FIXME: A follow up patch will call valueChanged() here.
 }
 
 Color SVGColor::colorFromRGBColorString(const String& colorString)
 {
     // FIXME: Rework css parser so it is more SVG aware.
     RGBA32 color;
     if (CSSParser::parseColor(color, colorString.stripWhiteSpace()))
         return color;
     return Color();
 }
 
 void SVGColor::setRGBColorICCColor(const String& rgbColor, const String& iccColor, ExceptionCode& ec)
 {
     if (rgbColor.isEmpty() || iccColor.isEmpty()) {
         ec = SVGException::SVG_INVALID_VALUE_ERR;
         return;
     }
 
     // FIXME: No support for ICC colors. We're just ignoring it.
     setRGBColor(rgbColor, ec);
     if (ec)
         return;
 
     m_colorType = SVG_COLORTYPE_RGBCOLOR_ICCCOLOR;
-    setNeedsStyleRecalc();
+    // FIXME: A follow up patch will call valueChanged() here.
 }
 
 void SVGColor::setColor(unsigned short colorType, const String& rgbColor, const String& iccColor, ExceptionCode& ec)
 {
     if (colorType > SVG_COLORTYPE_CURRENTCOLOR) {
         ec = SVGException::SVG_WRONG_TYPE_ERR;
         return;
     }
 
     bool requiresRGBColor = false;
@@ skipping 19 matching lines @@
         ec = SVGException::SVG_INVALID_VALUE_ERR;
         return;
     }
 
     // Spec: If colorType requires an SVGICCColor, then iccColor must be a string that matches <icccolor>; otherwise, iccColor must be null.
     if (requiresICCColor && iccColor.isEmpty()) {
         ec = SVGException::SVG_INVALID_VALUE_ERR;
         return;
     }
 
-    setNeedsStyleRecalc();
+    // FIXME: A follow up patch will call valueChanged() here.
     m_colorType = type;
     if (!requiresRGBColor) {
         ASSERT(!requiresICCColor);
         m_color = Color();
         return;
     }
 
     if (requiresICCColor)
         setRGBColorICCColor(rgbColor, iccColor, ec);
     else
@@ skipping 15 matching lines @@
         return "currentColor";
     }
 
     ASSERT_NOT_REACHED();
     return String();
 }
 
 }
 
 #endif // ENABLE(SVG)