Re: [PATCH v14 0/2] MR3391: mshtml: implement window.MutationObserver with MutationObserver stub

[Gabriel Ivăncescu (＠insn)]

Gabriel Ivăncescu (@insn) commented about dlls/mshtml/tests/documentmode.js:
> -
>    ok(e.name == "TypeMismatchError", "MutationObserver with non-function arg threw exception " + e.name);
> - }
> -
> - try {
> -
>    new window.MutationObserver(function() {});
> - } catch(e) {
> -
>    ok(false, "MutationObserver threw exception " + e.number);
> - }
> -
> - try {
> -
>    new window.MutationObserver(function() {}, 1);
> - } catch(e) {
> -
>    ok(false, "MutationObserver with extra args threw exception " + e.number);
> - }
>
> +})
> -

Small nitpick, but you don't need try {} catch {} blocks for *successful* tests, just put the succesful test without any try/catch. If it fails it will generate an exception and the test will fail (but obviously, it shouldn't, so that's fine).