[Webkit-unassigned] [Bug 71811] Eliminate CSSMutableValue

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Nov 9 07:21:54 PST 2011


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


Nikolas Zimmermann <zimmermann at kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #114268|review+                     |review-
               Flag|                            |




--- Comment #8 from Nikolas Zimmermann <zimmermann at kde.org>  2011-11-09 07:21:53 PST ---
(From update of attachment 114268)
View in context: https://bugs.webkit.org/attachment.cgi?id=114268&action=review

r- for various tests issues:

> LayoutTests/svg/custom/SVGPaint-mutate-attribute.svg:-1
> -<?xml version="1.0" encoding="ISO-8859-1" standalone="no"?>

This test shouldn't die - along the others, see below.

> LayoutTests/svg/custom/SVGPaint-mutate-inline-style.svg:-2
> -<?xml version="1.0" encoding="ISO-8859-1" standalone="no"?>
> -<!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 20010904//EN" "http://www.w3.org/TR/2001/REC-SVG-20010904/DTD/svg10.dtd">

This should test that SVGPaint mutations don't work for inline styles anymore.

> LayoutTests/svg/custom/getPresentationAttribute-modify.svg:-3
> -<?xml version="1.0" encoding="ISO-8859-1" standalone="no"?>
> -<!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 20010904//EN" "http://www.w3.org/TR/2001/REC-SVG-20010904/DTD/svg10.dtd">
> -<svg xmlns="http://www.w3.org/2000/svg" onload="runTest()">

I'd rather modify the tests to demonstrate that getPA does NOT modify anymore.

> LayoutTests/svg/custom/getPresentationAttribute.svg:27
> -        var fill = rect.getPresentationAttribute('fill');
> -        fill.setRGBColor("green");
> +        rect.setAttribute("fill", "green");

This makes the test useless, it's supposed to check that rect.getPresentationAttribute('fill') returns the right value.
You should make sure that fill is a SVGPaint, and check its color is not-green. Then call rect.setAttribute("fill",, "green", refetch the SVGPaint using getPA, and check its now green.

> LayoutTests/svg/dom/SVGColor-expected.txt:38
> -PASS stopColor.setColor(SVGColor.SVG_COLORTYPE_RGBCOLOR_ICCCOLOR, 'rgb(77,0,77)', 'icc-color(myRGB, 0, 1, 2)') is undefined.
> -PASS stopColor.colorType is SVGColor.SVG_COLORTYPE_RGBCOLOR_ICCCOLOR
> -PASS stopColor.colorType is SVGColor.SVG_COLORTYPE_RGBCOLOR_ICCCOLOR
> +FAIL stopColor.setColor(SVGColor.SVG_COLORTYPE_RGBCOLOR_ICCCOLOR, 'rgb(77,0,77)', 'icc-color(myRGB, 0, 1, 2)') should be undefined. Threw exception Error: NO_MODIFICATION_ALLOWED_ERR: DOM Exception 7

You should update these tests, that they still say PASS. Otherwhise this will lead to confusion. aka. tests that all of them throw now.

> LayoutTests/svg/dom/SVGPaint-expected.txt:47
> +FAIL fillPaint.setPaint(SVGPaint.SVG_PAINTTYPE_NONE, '', '', '') should be undefined. Threw exception Error: NO_MODIFICATION_ALLOWED_ERR: DOM Exception 7
> +FAIL fillPaint.paintType should be 101. Was 1.

Also not good, this should be changed to report PASSes.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list