Throw INDEX_SIZE_ERR instead of SYNTAX_ERR when calling CanvasGradient.addColorStop with non-finite values

RESOLVED FIXED in mozilla5

Status

()

Core
Canvas: 2D
--
trivial
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Tracking

(Blocks: 1 bug, {dev-doc-complete, html5})

Trunk
mozilla5
dev-doc-complete, html5
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

(Assignee)

Description

7 years ago
Created attachment 508247 [details] [diff] [review]
Patch v1
Flags: in-testsuite+
Attachment #508247 - Flags: review?(vladimir)
(Assignee)

Comment 1

7 years ago
Makes us pass

http://www.w3c-test.org/html/tests/submission/PhilipTaylor/canvas/2d.gradient.object.invalidoffset.html
(Assignee)

Updated

7 years ago
Whiteboard: [needs review]
(Assignee)

Updated

7 years ago
Attachment #508247 - Flags: review?(vladimir) → review?(jonas)
Attachment #508247 - Flags: review?(jonas) → review+
(Assignee)

Comment 2

7 years ago
Created attachment 524625 [details] [diff] [review]
Patch for checkin
(Assignee)

Updated

7 years ago
Keywords: checkin-needed
Whiteboard: [needs review]
Pushed: http://hg.mozilla.org/mozilla-central/rev/06be56cbd4f0
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED

Updated

7 years ago
Keywords: checkin-needed
(Assignee)

Updated

7 years ago
Target Milestone: --- → mozilla2.2
Keywords: dev-doc-needed
Added a note here:

https://developer.mozilla.org/en/DOM/CanvasGradient

And mentioned on Firefox 5 for developers.
Keywords: dev-doc-needed → dev-doc-complete
You need to log in before you can comment on or make changes to this bug.