Don't throw on setting lineWidth and ignore non-positive values

RESOLVED FIXED in mozilla2.0b11

Status

()

RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Tracking

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

Trunk
mozilla2.0b11
dev-doc-complete, html5
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Updated

8 years ago
Flags: in-testsuite+
Hmm, why lineWidth doesn't throw but arc should throw (Bug 630068).
Sounds like an inconsistency in the API, which should be fixed in the spec.
But perhaps there is some reason for this behavior.
(Assignee)

Comment 2

8 years ago
It seems like attributes don't throw, and methods do. That seems reasonable.
(Assignee)

Updated

8 years ago
Assignee: nobody → Ms2ger
(Assignee)

Comment 4

8 years ago
Created attachment 508735 [details] [diff] [review]
Patch for checkin

Thanks!
Attachment #508279 - Attachment is obsolete: true
(Assignee)

Updated

8 years ago
Keywords: checkin-needed
Whiteboard: [needs review]
http://hg.mozilla.org/mozilla-central/rev/22aa43f18646
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
(Assignee)

Updated

8 years ago
Target Milestone: --- → mozilla2.0b11
Keywords: dev-doc-needed
Added note to:

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

And listed the change on Firefox 4 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.