491150 - canvas arc with negative radius doesn't throw INDEX_SIZE_ERR

Status: RESOLVED FIXED

(Core :: Graphics: Canvas2D, defect)

Description

[Dirk Schulze]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; de; rv:1.9.0.10) Gecko/2009042523 Ubuntu/9.04 (jaunty) Firefox/3.0.10
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; de; rv:1.9.0.10) Gecko/2009042523 Ubuntu/9.04 (jaunty) Firefox/3.0.10

An arc() with a negative radius doesn't throw an INDEX_SIZE_ERR (http://www.whatwg.org/specs/web-apps/current-work/#dom-context-2d-arc) but current versions of Opera or WebKit do.
This was discussed on the WhatWg mailing list http://www.nabble.com/Canvas---Exception-on-arc-with-negative-radius-td22748785.html and Hixie opposed to change the specification.

Reproducible: Always

Steps to Reproduce:
1. Visit the link above in Opera and Firefox
2. Move the molecules around

Actual Results:  
Circles stay in FF

Expected Results:  
Circles go away in Opera an throw the error but stay in FF

Comment 1

[Saint Wesonga]

Attachment: Patch for INDEX_SIZE_ERR exception

Patch to raise exception specified by spec - also, there's a simple testcase at http://philip.html5.org/tests/canvas/suite/tests/2d.path.arc.negative.html

Comment 3

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Attachment: Patch v2

Actually, this needs a test change.

Comment 4

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Attachment: Patch for checkin

Thanks!

Comment 5

[Jonathan Watt [:jwatt]]

Pushed http://hg.mozilla.org/mozilla-central/rev/263e4f35b797 but note the bug number in the patch was wrong.

Comment 6

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Oops, sorry about that.

Comment 7

[Eric Shepherd [:sheppy]]

Added note to:

https://developer.mozilla.org/en/DOM/CanvasRenderingContext2D#Notes

And listed the change on Firefox 4 for developers.