| Index: LayoutTests/fast/dom/event-target-arguments.html
|
| diff --git a/LayoutTests/fast/dom/event-target-arguments.html b/LayoutTests/fast/dom/event-target-arguments.html
|
| index d828867340455a247558db9909e01deac21be9e5..a91793dd24b5db4410fed9b780536874261aaadc 100644
|
| --- a/LayoutTests/fast/dom/event-target-arguments.html
|
| +++ b/LayoutTests/fast/dom/event-target-arguments.html
|
| @@ -21,15 +21,21 @@ shouldNotThrow('document.addEventListener("", function(){})');
|
| shouldNotThrow('document.addEventListener("bar", listener, false)');
|
| shouldNotThrow('document.addEventListener("bar", listener, true)');
|
| // null
|
| +shouldBe('document.addEventListener(null)', 'undefined');
|
| shouldNotThrow('document.addEventListener(null, listener)'); // converted to "null"
|
| // FIXME: throw on |null|: http://crbug.com/249598
|
| // shouldThrow('document.addEventListener("foo", null)');
|
| +shouldBe('document.addEventListener("foo", null)', 'undefined');
|
| shouldNotThrow('document.addEventListener("zork", listener, null)');
|
| // undefined
|
| -// FIXME: behavior of undefined here is unclear, but probably should throw
|
| +// FIXME: behavior of undefined for mandatory arguments is unclear, but
|
| +// probably should throw
|
| // https://www.w3.org/Bugs/Public/show_bug.cgi?id=23532
|
| +shouldBe('document.addEventListener(undefined)', 'undefined');
|
| // shouldThrow('document.addEventListener(undefined, listener)');
|
| // shouldThrow('document.addEventListener("foo", undefined)');
|
| +shouldBe('document.addEventListener(undefined, listener)', 'undefined');
|
| +shouldBe('document.addEventListener("foo", undefined)', 'undefined');
|
| shouldNotThrow('document.addEventListener("zork", listener, undefined)');
|
|
|
| debug('');
|
| @@ -44,16 +50,22 @@ shouldNotThrow('document.removeEventListener("foo", listener)');
|
| shouldNotThrow('document.removeEventListener("foo", listener, true)');
|
| shouldNotThrow('document.removeEventListener("bar", listener, false)');
|
| shouldNotThrow('document.removeEventListener("bar", listener, false)');
|
| -shouldNotThrow('document.addEventListener("bar", listener, true)');
|
| +shouldNotThrow('document.removeEventListener("bar", listener, true)');
|
| // null
|
| +shouldBe('document.removeEventListener(null)', 'undefined');
|
| shouldNotThrow('document.removeEventListener(null, listener)'); // converted to "null"
|
| // FIXME: throw on |null|: http://crbug.com/249598
|
| // shouldThrow('document.removeEventListener("foo", null)');
|
| +shouldBe('document.removeEventListener("foo", null)', 'undefined');
|
| shouldNotThrow('document.removeEventListener("zork", listener, null)');
|
| // undefined
|
| -// FIXME: behavior of undefined here is unclear, but probably should throw
|
| +// FIXME: behavior of undefined for mandatory arguments is unclear, but
|
| +// probably should throw
|
| // https://www.w3.org/Bugs/Public/show_bug.cgi?id=23532
|
| -// shouldThrow('document.removeEventListener("foo", undefined)');
|
| -// shouldThrow('document.removeEventListener(undefined, listener)');
|
| +shouldBe('document.removeEventListener(undefined)', 'undefined');
|
| +// shouldthrow('document.removeeventlistener("foo", undefined)');
|
| +// shouldthrow('document.removeeventlistener(undefined, listener)');
|
| +shouldBe('document.removeEventListener(undefined, listener)', 'undefined');
|
| +shouldBe('document.removeEventListener("foo", undefined)', 'undefined');
|
| shouldNotThrow('document.removeEventListener("zork", listener, undefined)');
|
| </script>
|
|
|