Closed Bug 745498 Opened 12 years ago Closed 12 years ago

Add testcases for DOM4 exception types

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla15

People

(Reporter: emk, Assigned: emk)

References

Details

Attachments

(1 file, 2 obsolete files)

      No description provided.
Attached patch patch (obsolete) — Splinter Review
Assignee: nobody → VYV03354
Status: NEW → ASSIGNED
Attachment #615099 - Flags: review?(jonas)
Comment on attachment 615099 [details] [diff] [review]
patch

Review of attachment 615099 [details] [diff] [review]:
-----------------------------------------------------------------

r=me with that change. Yay!

::: content/base/test/file_bug416317.xhtml
@@ +494,5 @@
>            return root[select](q, resolver);
>          } catch(e){
> +          if ( e.message.indexOf("ERR") > -1 ||
> +              e.name == "NamespaceError" && e.code == DOMException.NAMESPACE_ERR ||
> +              e.name == "SyntaxError" && e.code == DOMException.SYNTAX_ERR)

Please don't rely on the fact that && has higher precedence than || since that's not obvious to everyone. So add some parenthesis here.

@@ +548,5 @@
>              try {
>                results = query(q, resolver);
>              } catch(e) {
> +              pass = (e.message === "bad ERROR" ||
> +                      e.name == "NamespaceError" && e.code == DOMException.NAMESPACE_ERR);

Same here
Attachment #615099 - Flags: review?(jonas) → review+
Attached patch patch for check in (obsolete) — Splinter Review
Attachment #615099 - Attachment is obsolete: true
Attachment #618253 - Flags: review+
Keywords: checkin-needed
content/test/unit/test_range.js appears to be bitrotted. Please rebase.
Keywords: checkin-needed
Conflicted with bug 702948. Rebased.
Attachment #618253 - Attachment is obsolete: true
Attachment #618651 - Flags: review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/bafdd54189ee
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: