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

Unified Diff: LayoutTests/fast/dom/event-target-arguments.html

Issue 1233483002: Add counters for add/removeEventListener() called with one argument (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: No [Conditional] support Created 5 years, 5 months 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | LayoutTests/fast/dom/event-target-arguments-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 a91793dd24b5db4410fed9b780536874261aaadc..4ed6589d4d20b708b8d3795565c1d5b118f76761 100644
--- a/LayoutTests/fast/dom/event-target-arguments.html
+++ b/LayoutTests/fast/dom/event-target-arguments.html
@@ -13,8 +13,14 @@ debug('void addEventListener(DOMString type, EventListener listener, optional bo
// FIXME: should throw on missing arguments: http://crbug.com/353484
// shouldThrow('document.addEventListener()');
// shouldThrow('document.addEventListener("foo")');
+var AddEventListenerNoArguments = 656;
+shouldBeFalse('internals.isUseCounted(document, AddEventListenerNoArguments)');
shouldBe('document.addEventListener()', 'undefined');
+shouldBeTrue('internals.isUseCounted(document, AddEventListenerNoArguments)');
+var AddEventListenerOneArgument = 657;
+shouldBeFalse('internals.isUseCounted(document, AddEventListenerOneArgument)');
shouldBe('document.addEventListener("foo")', 'undefined');
+shouldBeTrue('internals.isUseCounted(document, AddEventListenerOneArgument)');
shouldNotThrow('document.addEventListener("foo", listener)');
shouldNotThrow('document.addEventListener("", listener)');
shouldNotThrow('document.addEventListener("", function(){})');
@@ -44,8 +50,14 @@ debug('void removeEventListener(DOMString type, EventListener listener, optional
// FIXME: should throw on missing arguments: http://crbug.com/353484
// shouldThrow('document.removeEventListener()');
// shouldThrow('document.removeEventListener("foo")');
+var RemoveEventListenerNoArguments = 658;
+shouldBeFalse('internals.isUseCounted(document, RemoveEventListenerNoArguments)');
shouldBe('document.removeEventListener()', 'undefined');
+shouldBeTrue('internals.isUseCounted(document, RemoveEventListenerNoArguments)');
+var RemoveEventListenerOneArgument = 659;
+shouldBeFalse('internals.isUseCounted(document, RemoveEventListenerOneArgument)');
shouldBe('document.removeEventListener("foo")', 'undefined');
+shouldBeTrue('internals.isUseCounted(document, RemoveEventListenerOneArgument)');
shouldNotThrow('document.removeEventListener("foo", listener)');
shouldNotThrow('document.removeEventListener("foo", listener, true)');
shouldNotThrow('document.removeEventListener("bar", listener, false)');
« no previous file with comments | « no previous file | LayoutTests/fast/dom/event-target-arguments-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698