[Webkit-unassigned] [Bug 135215] Turn x/y to presentation attributes

[bugzilla-daemon at webkit.org]

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





--- Comment #3 from Dirk Schulze <krit at webkit.org>  2014-07-24 15:54:28 PST ---
(In reply to comment #2)
> (From update of attachment 235444 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=235444&action=review
> 
> > LayoutTests/svg/css/parse-length-expected.txt:7
> > +CONSOLE MESSAGE: Error: Invalid value for <rect> attribute width="  100"
> > +CONSOLE MESSAGE: Error: Invalid value for <rect> attribute width="100   "
> > +CONSOLE MESSAGE: Error: Invalid value for <rect> attribute x="  100"
> > +CONSOLE MESSAGE: Error: Invalid value for <rect> attribute x="100   "
> > +CONSOLE MESSAGE: Error: Invalid value for <rect> attribute y="  100"
> > +CONSOLE MESSAGE: Error: Invalid value for <rect> attribute y="100   "
> 
> Why are these invalid?

The problem is that SVGLength parsing does not accept new parsing rules. Cleaning this up is one of the next patches.

> 
> > LayoutTests/svg/css/parse-length.html:12
> > +description("Test that 'with' presentation attribute is parsed with CSS presentation rules.");
> 
> width not with
> 
> > Source/WebCore/css/CSSComputedStyleDeclaration.cpp:2902
> > +        // New positioning and sizing properties for SVG
> 
> Nit: missing .

Will fix both. Thanks!

> 
> > Source/WebCore/rendering/style/SVGRenderStyleDefs.h:294
> > +    // Positioning and sizing properties.
> > +    class StyleLayoutData : public RefCounted<StyleLayoutData> {
> 
> It's just positioning.

For now. It will have more information in the next patch. Just fear that I could forget to update the comment.

> 
> > Source/WebCore/svg/SVGAnimateElement.cpp:6
> > + * Copyright (C) 2014 Adobe Systems Incorporated. All rights reserved.
> 
> There were no other changes in this file.

These were missing from the last patch. Did add a comment to the change log.

> 
> > Source/WebCore/svg/SVGFilterElement.cpp:7
> > + * Copyright (C) 2014 Adobe Systems Incorporated. All rights reserved.
> 
> Or this one.

Ditto.

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