Created attachment 40300 [details] Test case revealing the problem. This is similar to other webkit problems where we set a value into the DOM and we can't get that exact value back. Attached is a test case exhibiting the problem. <svg xmlns="http://www.w3.org/2000/svg"> <polyline id="poly" fill="green" /> <script><![CDATA[ var poly = document.getElementById("poly"); var points = "10,10 100,60 10,110"; poly.setAttribute("points", points); if (poly.getAttribute("points") != points) { alert("Pass failed, points attribute equals:\n" + poly.getAttribute("points")); } ]]></script> </svg> The above passes in Firefox and Opera but not in Safari/Chrome because Webkit changes the points string into a space-separate list of numbers. This can be a problem if we need to build up the value of the 'points' attribute and then need to check it at some point. See http://code.google.com/p/svg-edit/issues/detail?id=245 P.S. The 'Version' box in Bugzilla seems to be missing many options (Safari 4?)
Seems fixed nowadays. Cheers, Rob.
Rob, would it make sense to land the test, or do you expect this to be covered already?
Hi Alexey, (In reply to comment #2) > Rob, would it make sense to land the test, or do you expect this to be covered already? Good point. I think poly-points-attribute-changes.svg comes really close. It just uses getAttributeNode instead of getAttribute. Assuming both getAttribute and getAttributeNode will use the same internal Attr (which I think is the case) then we do not need the test below. What do you think? Cheers, Rob.
(In reply to comment #1) > Seems fixed nowadays. > Cheers, > > Rob. The problem here is, that Jeff wants to get the unnormalizd (original string) back. At least if I get it correct. IIRC we don't do this, we just changed our code to comma separated the axis of a point. If you add a space or something else, you can't get the original string back right now. So it is not fixed.
We match with other browser on attached test case but I don't know whether we have test coverage for the following or not. Appreciate if someone can confirm whether it needs test coverage or not. Else it can be marked as "RESOLVED CONFIGURATION CHANGED". Thanks!