1452010 - Remove nsIDOMDOMException

Status: RESOLVED FIXED

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

Comment 1

[Boris Zbarsky [:bzbarsky, bz on IRC]]

Attachment: part 1. Remove uses of nsIDOMDOMException

MozReview-Commit-ID: K5zUjh25WJG

Comment 2

[Boris Zbarsky [:bzbarsky, bz on IRC]]

Attachment: part 2. Remove nsIDOMDOMException members

MozReview-Commit-ID: Bp5ablEtcHA

Comment 3

[Boris Zbarsky [:bzbarsky, bz on IRC]]

Attachment: part 3. Remove nsIDOMDOMException

MozReview-Commit-ID: AFKzyxCyriJ

Comment 4

[Boris Zbarsky [:bzbarsky, bz on IRC]]

Attachment: part 1. Remove uses of nsIDOMDOMException

MozReview-Commit-ID: K5zUjh25WJG

Comment 5

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

Comment on attachment 8965674 [details] [diff] [review]
part 1.  Remove uses of nsIDOMDOMException

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

::: devtools/server/actors/call-watcher.js
@@ +621,5 @@
>    let parsedStack = parseStack(stack);
>    let { fileName, lineNumber, columnNumber } = parsedStack[parsedStack.length - 1];
>    let error;
>  
> +  let isDOMException = ChromeUtils.getClassName(e) == "DOMException";

===

::: devtools/server/actors/object/previewers.js
@@ +732,5 @@
>      return true;
>    },
>  
>    function DOMException({obj, hooks}, grip, rawObj) {
> +    if (isWorker || !rawObj || obj.class != "DOMException") {

!==

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e52e0d29bb1a
part 1.  Remove uses of nsIDOMDOMException.  r=qdot
https://hg.mozilla.org/integration/mozilla-inbound/rev/0b4a43745b4a
part 2.  Remove nsIDOMDOMException members.  r=qdot
https://hg.mozilla.org/integration/mozilla-inbound/rev/ace85b988b45
part 3.  Remove nsIDOMDOMException.  r=qdot

Comment 7

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/e52e0d29bb1a
https://hg.mozilla.org/mozilla-central/rev/0b4a43745b4a
https://hg.mozilla.org/mozilla-central/rev/ace85b988b45