karl- / poly2tri

Automatically exported from code.google.com/p/poly2tri
Other
0 stars 2 forks source link

setLineDash(null) will throw an exception in Chrome 30 #83

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 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

GoogleCodeExporter commented 8 years ago
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

GoogleCodeExporter commented 8 years ago
fixed, please close

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

GoogleCodeExporter commented 8 years ago

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