Bug 114578

Summary: vector-effect: non-scaling-stroke not recognized when animating
Product: WebKit Reporter: Stuart P. Bentley <stuart>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: kling, krit, pdr, zimmermann
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Test case
none
Unrelated artifacts
none
Test case none

Stuart P. Bentley
Reported 2013-04-13 23:01:00 PDT
Created attachment 197972 [details] Test case Strokes with the style {vector-effect: non-scaling-stroke} are scaled when transforming an SVG by setting an otherwise-unspecified viewBox via animation. Tested on Windows 7 @ Chrome stable, Windows 8 @ Chrome dev, Arch Linux @ Chromium 26.0.1410.63, and Android 4.2.1 @ Chrome latest. See attached test case. In writing this test case, I found that the "Desired behavior" (employing a workaround by setting the viewBox attribute to the end result) produces artifacts when animating on Windows and Linux. This is not part of the desired behavior, and I will file a bug summarily.
Attachments
Test case (831 bytes, text/html)
2013-04-13 23:01 PDT, Stuart P. Bentley 		no flags
Details
Unrelated artifacts (2.37 KB, image/png)
2013-04-13 23:06 PDT, Stuart P. Bentley 		no flags
Details
Test case (824 bytes, text/html)
2013-04-13 23:59 PDT, Stuart P. Bentley 		no flags
Details
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Stuart P. Bentley
Comment 1 2013-04-13 23:06:47 PDT
Created attachment 197973 [details] Unrelated artifacts When noting the desired behavior, ignore the creation of artifacts like those outlined in red in this screenshot.
Stuart P. Bentley
Comment 2 2013-04-13 23:33:16 PDT
Unrelated artifact bug filed as Bug 114579.
Stuart P. Bentley
Comment 3 2013-04-13 23:34:53 PDT
(In reply to comment #0) > Tested on Windows 7 @ Chrome stable, Windows 8 @ Chrome dev, Arch Linux @ Chromium 26.0.1410.63, and Android 4.2.1 @ Chrome latest. s/Windows 7 @ Chrome stable, Windows 8 @ Chrome dev/Windows 7 @ Chrome dev, Windows 8 @ Chrome stable/
Stuart P. Bentley
Comment 4 2013-04-13 23:59:04 PDT
Created attachment 197978 [details] Test case Further testing has revealed that *any* value on the viewBox attribute will inhibit this bug. Uploading simplified test case with viewBox="".
Dirk Schulze
Comment 5 2013-04-14 20:41:26 PDT
I agree that this is a bug. It looks like we do not take the viewBox transform into account for reverse transforming. Thanks for reporting.
Andreas Kling
Comment 6 2013-04-25 05:56:35 PDT
Merged the fix for this from blink in <http://trac.webkit.org/changeset/149102> (thanks pdr!)
Philip Rogers
Comment 7 2013-04-25 11:55:37 PDT
(In reply to comment #6) > Merged the fix for this from blink in <http://trac.webkit.org/changeset/149102> (thanks pdr!) Aww man, I was preparing a patch for WebKit. Thanks for merging this though.
Note You need to log in before you can comment on or make changes to this bug.