Created attachment 508466 [details] [diff] [review] Patch v1 This makes us pass http://www.w3c-test.org/html/tests/approved/canvas/2d.pattern.image.null.html http://www.w3c-test.org/html/tests/approved/canvas/2d.pattern.image.undefined.html
Attachment #508466 - Flags: review?(Olli.Pettay)
7 years ago
Attachment #508466 - Flags: review?(Olli.Pettay) → review+
Depends on: 610267
Whiteboard: [needs review] → [needs landing]
Created attachment 521911 [details] [diff] [review] Patch for checkin
Attachment #508466 - Attachment is obsolete: true
Whiteboard: [needs landing]
Boris: not-ready-for-cedar as in post 2.2?
That was my thinking on the "it makes us throw where we don't throw now" premise. But I would be open to being convinced otherwise here.... (e.g. if my premise is wrong or if we have good reason to believe that no one depends on our old behavior).
We already throw NS_ERROR_NOT_AVAILABLE, this is just a change of which exception we throw.
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.2
Keywords: dev-doc-needed → dev-doc-complete
You need to log in before you can comment on or make changes to this bug.