setLineDash(null) will throw an exception in Chrome 30

Question

setLineDash(null) will throw an exception in Chrome 30

GoogleCodeExporter opened this issue 10 years ago · 3 comments

GoogleCodeExporter commented 10 years ago

Heads-up from the Chrome team!
In your javascript repository, file tests/index.js calls setLineDash(null). 
Starting with Chrome 3O, this syntax will throw an exception.  The correct way 
to reset line dashing is by using an empty array: "setLineDash([])"

Original issue reported on code.google.com by junov@chromium.org on 27 Aug 2013 at 9:46

Answer 1 · 2015-03-14T06:43:59.000Z

ok, replaced setLineDash(null) by setLineDash([]), which is compliant with the 
canvas spec.
Tested ok on FF25, chromium 28, chrome 31

Original comment by remi.tur...@gmail.com on 21 Nov 2013 at 4:06

Answer 2 · 2015-03-14T06:43:59.000Z

fixed, please close

Original comment by remi.tur...@gmail.com on 21 Nov 2013 at 4:09

Answer 3 · 2015-03-14T06:43:59.000Z

Original comment by thahlen@gmail.com on 21 Nov 2013 at 9:19

Changed state: Fixed