Add counters for addEventListener/removeEventListener optional arguments

Add counters for addEventListener/removeEventListener optional arguments

Per the FIXME in EventTarget.h, making these non-optional per spec has
failed once, so add counters to determine if it is now safe or not.

BUG=353484
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=187434

[philipj_slow, 2014-12-17 19:27:25]

rebase

[philipj_slow, 2014-12-17 19:34:02]

philipj@opera.com changed reviewers:
+ haraken@chromium.org

[philipj_slow, 2014-12-17 19:34:02]

PTAL

[haraken, 2014-12-17 23:55:12]

LGTM

[philipj_slow, 2014-12-17 23:55:56]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2014-12-17 23:56:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/815503002/20001

[commit-bot: I haz the power, 2014-12-17 23:57:11]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-12-17 23:57:12]

Failed to apply patch for Source/core/frame/UseCounter.h:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Source/core/frame/UseCounter.h
  Hunk #1 FAILED at 570.
  1 out of 1 hunk FAILED -- saving rejects to file
Source/core/frame/UseCounter.h.rej

Patch:       Source/core/frame/UseCounter.h
Index: Source/core/frame/UseCounter.h
diff --git a/Source/core/frame/UseCounter.h b/Source/core/frame/UseCounter.h
index
4b02c83bfa474767da5839ae5ccbd95214b18875..fd9bf6df2f247d538abf6ed6ce4841ecde719cbe
100644
--- a/Source/core/frame/UseCounter.h
+++ b/Source/core/frame/UseCounter.h
@@ -570,6 +570,10 @@ public:
         CSSSelectorPseudoFullScreenAncestor = 628,
         CSSSelectorPseudoFullScreen = 629,
         WebKitCSSMatrix = 630,
+        AddEventListenerNoArguments = 631,
+        AddEventListenerOneArgument = 632,
+        RemoveEventListenerNoArguments = 633,
+        RemoveEventListenerOneArgument = 634,
 
         // Add new features immediately above this line. Don't change assigned
         // numbers of any item, and don't reuse removed slots.

[philipj_slow, 2014-12-18 00:08:04]

rebase

[philipj_slow, 2014-12-18 06:49:21]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2014-12-18 06:50:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/815503002/40001

[commit-bot: I haz the power, 2014-12-18 06:53:24]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=187434