Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(474)

Side by Side Diff: third_party/WebKit/LayoutTests/fast/dom/node-legacy-event-listener.html

Issue 1461993002: Make addEventListener/removeEventListener arguments non-optional (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: drop the use counters Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 <!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML//EN"> 1 <!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML//EN">
2 <html> 2 <html>
3 <head> 3 <head>
4 <script src="../../resources/js-test.js"></script> 4 <script src="../../resources/js-test.js"></script>
5 </head> 5 </head>
6 <body> 6 <body>
7 <script> 7 <script>
8 description("Test addEventListener() and removeEventListener() fail silently if arguments are missing."); 8 description("Test addEventListener() and removeEventListener() fail silently if arguments are missing.");
9 9
10 shouldBe('document.addEventListener("foo")', 'undefined'); 10 shouldThrow('document.addEventListener("foo")');
11 shouldBe('document.removeEventListener("bar")', 'undefined'); 11 shouldThrow('document.removeEventListener("bar")');
12 12
13 shouldBe('document.addEventListener()', 'undefined'); 13 shouldThrow('document.addEventListener()');
14 shouldBe('document.removeEventListener()', 'undefined'); 14 shouldThrow('document.removeEventListener()');
15 </script> 15 </script>
16 </body> 16 </body>
17 </html> 17 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698