+2006-12-26 Eric Seidel <eric@webkit.org>
+
+ Reviewed by olliej.
+
+ ASSERT while loading SVG map
+ http://bugs.webkit.org/show_bug.cgi?id=11987
+
+ * svg/custom/invalid-fill-hex-expected.checksum: Added.
+ * svg/custom/invalid-fill-hex-expected.png: Added.
+ * svg/custom/invalid-fill-hex-expected.txt: Added.
+ * svg/custom/invalid-fill-hex.svg: Added.
+
2006-12-27 Nikolas Zimmermann <zimmermann@kde.org>
Reviewed by Oliver.
* editing/pasteboard/emacs-cntl-y-001.html:
-2006-09-27 Eric Seidel <eric@eseidel.com>
+2006-09-27 Eric Seidel <eric@webkit.org>
Reviewed by mitz.
* svg/custom/viewport-update2-expected.txt: Added.
* svg/custom/viewport-update2.svg: Added.
-2006-09-27 Eric Seidel <eric@eseidel.com>
+2006-09-27 Eric Seidel <eric@webkit.org>
Reviewed by darin.
* editing/pasteboard/emacs-cntl-y-001-expected.txt: Added.
* editing/pasteboard/emacs-cntl-y-001.html: Added.
-2006-09-26 Eric Seidel <eric@eseidel.com>
+2006-09-26 Eric Seidel <eric@webkit.org>
Reviewed by Tim H.
* svg/hixie/viewbox/003-expected.txt: Added.
* svg/hixie/viewbox/003.xml: Added.
-2006-09-26 Eric Seidel <eric@eseidel.com>
+2006-09-26 Eric Seidel <eric@webkit.org>
Reviewed by Tim H.
* svg/custom/createelement-expected.txt: Added.
* svg/custom/createelement.svg: Added.
-2006-09-26 Eric Seidel <eric@eseidel.com>
+2006-09-26 Eric Seidel <eric@webkit.org>
Reviewed by Tim H.
* editing/execCommand/indent-empty-root-expected.txt: Added.
* editing/execCommand/indent-empty-root.html: Added.
-2006-09-24 Eric Seidel <eric@eseidel.com>
+2006-09-24 Eric Seidel <eric@webkit.org>
Reviewed by mjs.
* svg/custom/viewport-clip-expected.txt: Added.
* svg/custom/viewport-clip.svg: Added.
-2006-09-24 Eric Seidel <eric@eseidel.com>
+2006-09-24 Eric Seidel <eric@webkit.org>
Reviewed by beth.
* svg/custom/invalid-fill-expected.txt: Added.
* svg/custom/invalid-fill.svg: Added.
-2006-09-24 Eric Seidel <eric@eseidel.com>
+2006-09-24 Eric Seidel <eric@webkit.org>
Reviewed by hyatt.
* fast/forms/file-input-direction-expected.txt: Added.
* fast/forms/file-input-direction.html: Added.
-2006-09-20 Eric Seidel <eric@eseidel.com>
+2006-09-20 Eric Seidel <eric@webkit.org>
Reviewed by hyatt.
* svg/W3C-SVG-1.1/painting-marker-02-f-expected.png:
* svg/W3C-SVG-1.1/painting-marker-02-f-expected.txt:
-2006-09-19 Eric Seidel <eric@eseidel.com>
+2006-09-19 Eric Seidel <eric@webkit.org>
Reviewed by adele.
* svg/custom/non-opaque-filters-expected.txt:
* svg/custom/text-filter-expected.txt:
-2006-09-19 Eric Seidel <eric@eseidel.com>
+2006-09-19 Eric Seidel <eric@webkit.org>
Reviewed by adele.
* fast/forms/input-appearance-height-expected.txt:
* fast/forms/input-value-expected.txt:
-2006-09-17 Eric Seidel <eric@eseidel.com>
+2006-09-17 Eric Seidel <eric@webkit.org>
Reviewed by hyatt.
* fast/js/window-properties.html: make work in Firefox (hangs IE)
-2006-09-17 Eric Seidel <eric@eseidel.com>
+2006-09-17 Eric Seidel <eric@webkit.org>
Add a textual dump of the window object property tree.
* svg/W3C-SVG-1.1/struct-dom-06-b-expected.txt:
-2006-08-26 Eric Seidel <eric@eseidel.com>
+2006-08-26 Eric Seidel <eric@webkit.org>
Reviewed by hyatt.
* fast/forms/onselect-selectall-expected.txt: Added.
* fast/forms/onselect-selectall.html: Added.
-2006-08-14 Eric Seidel <eric@eseidel.com>
+2006-08-14 Eric Seidel <eric@webkit.org>
Reviewed by mjs.
* svg/custom/text-filter-expected.txt:
* svg/custom/tref-update-expected.txt:
-2006-08-14 Eric Seidel <eric@eseidel.com>
+2006-08-14 Eric Seidel <eric@webkit.org>
Reviewed by darin and mjs.
* tables/mozilla/bugs/bug24200-expected.txt: Ditto.
* tables/mozilla/bugs/bug4527-expected.txt: Ditto.
-2006-08-11 Eric Seidel <eric@eseidel.com>
+2006-08-11 Eric Seidel <eric@webkit.org>
Reviewed by andersca.
* svg/W3C-SVG-1.1/struct-dom-06-b-expected.png:
* svg/W3C-SVG-1.1/struct-dom-06-b-expected.txt:
-2006-07-27 Eric Seidel <eric@eseidel.com>
+2006-07-27 Eric Seidel <eric@webkit.org>
Reviewed by darin.
* fast/encoding/denormalised-voiced-japanese-chars-expected.txt: Added.
* fast/encoding/denormalised-voiced-japanese-chars.html: Added.
-2006-07-27 Eric Seidel <eric@eseidel.com>
+2006-07-27 Eric Seidel <eric@webkit.org>
Reviewed by ap.
* svg/custom/invalid-viewbox2-expected.txt: Added.
* svg/custom/invalid-viewbox2.svg: Added.
-2006-07-27 Eric Seidel <eric@eseidel.com>
+2006-07-27 Eric Seidel <eric@webkit.org>
Reviewed by ap.
* tables/mozilla_expected_failures/bugs/bug32205-1-expected.txt:
* tables/mozilla_expected_failures/bugs/bug65372-expected.txt:
-2006-05-25 Eric Seidel <eric@eseidel.com>
+2006-05-25 Eric Seidel <eric@webkit.org>
Reviewed by andersca.
--- /dev/null
+990395834523505015e0a73e1769f835
\ No newline at end of file
--- /dev/null
+layer at (0,0) size 800x600
+ RenderView at (0,0) size 800x600
+ RenderSVGContainer {svg} at (-0.50,-0.50) size 677.50x194.50
+ RenderSVGContainer {g} at (-0.50,-0.50) size 677.50x194.50
+ RenderPath {rect} at (0,0) size 100x100 [fill={[type=SOLID] [color=#FF0000]}] [data="M0.00,0.00L100.00,0.00L100.00,100.00L0.00,100.00"]
+ RenderPath {rect} at (-0.50,-0.50) size 101x101 [fill={[type=SOLID] [color=#000000]}] [data="M0.00,0.00L100.00,0.00L100.00,100.00L0.00,100.00"]
+ RenderSVGContainer {g} at (10,116) size 667x78
+ RenderSVGText {text} at (10,130) size 800x18
+ RenderSVGInlineText {#text} at (0,-14) size 223x18
+ text run at (0,-14) width 223: "The above rect should be BLACK."
+ RenderSVGText {text} at (10,150) size 800x18
+ RenderSVGInlineText {#text} at (0,-14) size 667x18
+ text run at (0,-14) width 667: "SVG 1.1 FULL says there should be a \"highly perceivable error\" message when an invalid fill url is used."
+ RenderSVGText {text} at (10,170) size 800x18
+ RenderSVGInlineText {#text} at (0,-14) size 468x18
+ text run at (0,-14) width 468: "Browsers don't display such an error, instead use some sort of fill-fallback."
+ RenderSVGText {text} at (10,190) size 800x18
+ RenderSVGInlineText {#text} at (0,-14) size 546x18
+ text run at (0,-14) width 471: "Opera falls back to 'black' (default), Firefox to 'none', and Safari to 'black'. "
+ text run at (471,-14) width 75: "Bug 11017."
--- /dev/null
+<?xml version="1.0" encoding="UTF-8"?>
+<svg xmlns="http://www.w3.org/2000/svg">
+ <g fill="orange">
+ <rect width="100" height="100" fill="red"/>
+ <rect width="100" height="100" fill="#0" stroke="#0"/>
+ <g fill="black">
+ <text x="10" y="130">The above rect should be BLACK.</text>
+ <text x="10" y="150">SVG 1.1 FULL says there should be a "highly perceivable error" message when an invalid fill is used.</text>
+ <text x="10" y="170">Browsers don't display such an error, instead use some sort of fill-fallback.</text>
+ <text x="10" y="190">Opera falls back to 'black' (default), Firefox to 'none', and Safari to 'black'. Bug 11017.</text>
+ </g>
+ </g>
+</svg>
+2006-12-26 Eric Seidel <eric@webkit.org>
+
+ Reviewed by olliej.
+
+ * svg/svg.pltsuite: re-enable word-iso.svg after fixing http://bugs.webkit.org/show_bug.cgi?id=11987
+
2006-12-26 Eric Seidel <eric@webkit.org>
Reviewed by bradee-oh.
file:///WEBKIT_PATH/PageLoadTests/svg/files/gearflowers.svg
file:///WEBKIT_PATH/PageLoadTests/svg/files/hereGear4.svg
file:///WEBKIT_PATH/PageLoadTests/svg/files/mtsthelens.svg
-file:///WEBKIT_PATH/PageLoadTests/svg/files/world-iso.svg-disabled
+file:///WEBKIT_PATH/PageLoadTests/svg/files/world-iso.svg
file:///WEBKIT_PATH/PageLoadTests/svg/files/worldcup.svg
+2006-12-26 Eric Seidel <eric@webkit.org>
+
+ Reviewed by olliej.
+
+ ASSERT while loading SVG map
+ http://bugs.webkit.org/show_bug.cgi?id=11987
+
+ * ksvg2/css/SVGCSSParser.cpp: remove a few .impl() calls previously missed.
+ (WebCore::CSSParser::parseSVGValue):
+ (WebCore::CSSParser::parseSVGPaint):
+ (WebCore::CSSParser::parseSVGColor):
+ * ksvg2/misc/KCanvasRenderingStyle.cpp: handle invalid colors
+ (WebCore::KSVGPainterFactory::fillPaintServer):
+ (WebCore::KSVGPainterFactory::strokePaintServer):
+
2006-12-26 Eric Seidel <eric@webkit.org>
Reviewed by ggaren.
else if (id == SVGCSS_VAL_CURRENTCOLOR)
parsedValue = new SVGPaint(SVGPaint::SVG_PAINTTYPE_CURRENTCOLOR);
else if (value->unit == CSSPrimitiveValue::CSS_URI)
- parsedValue = new SVGPaint(SVGPaint::SVG_PAINTTYPE_URI, domString(value->string).impl());
+ parsedValue = new SVGPaint(SVGPaint::SVG_PAINTTYPE_URI, domString(value->string));
else
parsedValue = parseSVGPaint();
case CSS_PROP_COLOR: // <color> | inherit
if ((id >= CSS_VAL_AQUA && id <= CSS_VAL_WINDOWTEXT) ||
(id >= SVGCSS_VAL_ALICEBLUE && id <= SVGCSS_VAL_YELLOWGREEN))
- parsedValue = new SVGColor(domString(value->string).impl());
+ parsedValue = new SVGColor(domString(value->string));
else
parsedValue = parseSVGColor();
case SVGCSS_PROP_LIGHTING_COLOR:
if ((id >= CSS_VAL_AQUA && id <= CSS_VAL_WINDOWTEXT) ||
(id >= SVGCSS_VAL_ALICEBLUE && id <= SVGCSS_VAL_YELLOWGREEN))
- parsedValue = new SVGColor(domString(value->string).impl());
+ parsedValue = new SVGColor(domString(value->string));
else if (id == SVGCSS_VAL_CURRENTCOLOR)
parsedValue = new SVGColor(SVGColor::SVG_COLORTYPE_CURRENTCOLOR);
else // TODO : svgcolor (iccColor)
g = max(0, min(255, g));
b = max(0, min(255, b));
- return new SVGPaint(SVGPaint::SVG_PAINTTYPE_RGBCOLOR, String(), String::format("rgb(%d, %d, %d)", r, g, b).impl());
+ return new SVGPaint(SVGPaint::SVG_PAINTTYPE_RGBCOLOR, String(), String::format("rgb(%d, %d, %d)", r, g, b));
}
else
return 0;
{
Value* value = valueList->current();
if (!strict && value->unit == CSSPrimitiveValue::CSS_NUMBER && value->fValue >= 0. && value->fValue < 1000000.)
- return new SVGColor(String::format("%06d", (int)(value->fValue+.5)).impl());
- else if (value->unit == CSSPrimitiveValue::CSS_RGBCOLOR) {
- String str = "#" + domString(value->string);
- return new SVGColor(str.impl());
- } else if (value->unit == CSSPrimitiveValue::CSS_IDENT || (!strict && value->unit == CSSPrimitiveValue::CSS_DIMENSION))
- return new SVGColor(domString(value->string).impl());
+ return new SVGColor(String::format("%06d", (int)(value->fValue+.5)));
+ else if (value->unit == CSSPrimitiveValue::CSS_RGBCOLOR)
+ return new SVGColor("#" + domString(value->string));
+ else if (value->unit == CSSPrimitiveValue::CSS_IDENT || (!strict && value->unit == CSSPrimitiveValue::CSS_DIMENSION))
+ return new SVGColor(domString(value->string));
else if (value->unit == Value::Function && value->function->args != 0 && domString(value->function->name).lower() == "rgb(") {
ValueList* args = value->function->args;
Value* v = args->current();
g = max(0, min(255, g));
b = max(0, min(255, b));
- return new SVGColor(String::format("rgb(%d, %d, %d)", r, g, b).impl());
+ return new SVGColor(String::format("rgb(%d, %d, %d)", r, g, b));
}
else
return 0;
fillPaintServer = getPaintServerById(item->document(), AtomicString(fill->uri().substring(1)));
if (fillPaintServer && item->isRenderPath())
fillPaintServer->addClient(static_cast<const RenderPath*>(item));
- if (!fillPaintServer) {
- // default value (black), see bug 11017
- fillPaintServer = sharedSolidPaintServer();
- static_cast<SVGPaintServerSolid*>(fillPaintServer)->setColor(Color::black);
- }
} else {
fillPaintServer = sharedSolidPaintServer();
SVGPaintServerSolid* fillPaintServerSolid = static_cast<SVGPaintServerSolid*>(fillPaintServer);
fillPaintServerSolid->setColor(style->color());
else
fillPaintServerSolid->setColor(fill->color());
+ // FIXME: Ideally invalid colors would never get set on the RenderStyle and this could turn into an ASSERT
+ if (!fillPaintServerSolid->color().isValid())
+ fillPaintServer = 0;
+ }
+ if (!fillPaintServer) {
+ // default value (black), see bug 11017
+ fillPaintServer = sharedSolidPaintServer();
+ static_cast<SVGPaintServerSolid*>(fillPaintServer)->setColor(Color::black);
}
return fillPaintServer;
}
strokePaintServerSolid->setColor(style->color());
else
strokePaintServerSolid->setColor(stroke->color());
+ // FIXME: Ideally invalid colors would never get set on the RenderStyle and this could turn into an ASSERT
+ if (!strokePaintServerSolid->color().isValid())
+ strokePaintServer = 0;
}
return strokePaintServer;