Throw a TYPE_MISMATCH_ERR on createPattern(null)

RESOLVED FIXED in mozilla5

Status

()

Core
Canvas: 2D
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

(1 attachment, 1 obsolete attachment)

(Assignee)

Updated

7 years ago
Depends on: 610267
Whiteboard: [needs review] → [needs landing]
(Assignee)

Comment 1

7 years ago
Created attachment 521911 [details] [diff] [review]
Patch for checkin
Attachment #508466 - Attachment is obsolete: true
(Assignee)

Updated

7 years ago
Keywords: checkin-needed
Whiteboard: [needs landing]
Whiteboard: [not-ready-for-cedar]
(Assignee)

Comment 2

7 years ago
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).
(Assignee)

Comment 4

7 years ago
We already throw NS_ERROR_NOT_AVAILABLE, this is just a change of which exception we throw.
(Assignee)

Updated

7 years ago
Whiteboard: [not-ready-for-cedar]

Comment 5

7 years ago
http://hg.mozilla.org/mozilla-central/rev/15329bcef8d0
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.2

Updated

7 years ago
No longer depends on: 610267
Keywords: dev-doc-needed
You need to log in before you can comment on or make changes to this bug.