check EventTarget params (#2018)

denoland · Mar 30, 2019 · ad3cbc5 · ad3cbc5
1 parent 51c6f33
commit ad3cbc5
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/js/event_target.ts b/js/event_target.ts
@@ -1,5 +1,6 @@
 // Copyright 2018-2019 the Deno authors. All rights reserved. MIT license.
 import * as domTypes from "./dom_types";
+import { requiredArguments } from "./util";
 
 /* TODO: This is an incomplete implementation to provide functionality
  * for Event. A proper spec is still required for a proper Web API.
@@ -14,6 +15,7 @@ export class EventTarget implements domTypes.EventTarget {
  listener: domTypes.EventListenerOrEventListenerObject | null,
  _options?: boolean | domTypes.AddEventListenerOptions
  ): void {
+ requiredArguments("EventTarget.addEventListener", arguments.length, 2);
  if (!this.listeners.hasOwnProperty(type)) {
  this.listeners[type] = [];
  }
@@ -27,6 +29,7 @@ export class EventTarget implements domTypes.EventTarget {
  callback: domTypes.EventListenerOrEventListenerObject | null,
  _options?: domTypes.EventListenerOptions | boolean
  ): void {
+ requiredArguments("EventTarget.removeEventListener", arguments.length, 2);
  if (this.listeners.hasOwnProperty(type) && callback !== null) {
  this.listeners[type] = this.listeners[type].filter(
  listener => listener !== callback
@@ -35,6 +38,7 @@ export class EventTarget implements domTypes.EventTarget {
  }
 
  public dispatchEvent(event: domTypes.Event): boolean {
+ requiredArguments("EventTarget.dispatchEvent", arguments.length, 1);
  if (!this.listeners.hasOwnProperty(event.type)) {
  return true;
  }