Chromium Code Reviews Chromium Code Reviews
Issue 22687002:
Treat non-callable input as null for EventHandler attributes (Closed)
Base URL: svn://svn.chromium.org/blink/trunk| Index: LayoutTests/fast/html/eventhandler-attribute-non-callable.html |
| diff --git a/LayoutTests/fast/html/eventhandler-attribute-non-callable.html b/LayoutTests/fast/html/eventhandler-attribute-non-callable.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..67fcbd8f8106e614811d34d5d976f4ed68e6c0ce |
| --- /dev/null |
| +++ b/LayoutTests/fast/html/eventhandler-attribute-non-callable.html |
| @@ -0,0 +1,48 @@ |
| +<html> |
| +<head> |
| +<link rel="help" href="http://www.whatwg.org/specs/web-apps/current-work/multipage/webappapis.html#eventhandler"> |
| +<script src="../js/resources/js-test-pre.js"></script> |
| +</head> |
| +<body> |
| +<div id="div"></div> |
| +<script> |
| +description("This test checks that EventHandler attributes only accept JS functions as input."); |
| + |
| +var div = document.getElementById("div"); |
| +var callbackCount = 0; |
| + |
| +function callback() { |
| + ++callbackCount; |
| +} |
| + |
| +function dispatchKeyEvent() |
| +{ |
| + var event = new KeyboardEvent('keydown', { bubbles: true, cancelable: true, view: null, keyIdentifier: 'Enter' }); |
| + div.dispatchEvent(event); |
| +} |
| + |
| +// Input is a JS function, this should work. |
| +shouldBeNull('div.onkeydown'); |
| +shouldNotThrow('div.onkeydown = callback'); |
| +shouldBe('div.onkeydown', 'callback'); |
| +shouldBe('callbackCount', '0'); |
| +dispatchKeyEvent(); |
| +shouldBe('callbackCount', '1'); |
| + |
| +// Non callable input should be treated as null. |
| +var o = { 'handleEvent': callback }; |
|
arv (Not doing code reviews)
2013/08/08 19:45:11
No need for quotes if the property name is a valid
do-not-use
2013/08/08 19:53:28
Done.
|
| +shouldNotBe('div.onkeydown', 'null'); |
| +shouldNotThrow('div.onkeydown = o'); |
| +shouldBeNull('div.onkeydown'); |
| +dispatchKeyEvent(); |
| +shouldBe('callbackCount', '1'); |
| + |
| +// Test null assignment. |
| +shouldNotThrow('div.onkeydown = callback'); |
| +shouldBe('div.onkeydown', 'callback'); |
| +shouldNotThrow('div.onkeydown = null'); |
| +shouldBeNull('div.onkeydown'); |
| +</script> |
| +<script src="../js/resources/js-test-post.js"></script> |
| +</body> |
| +</html> |
