Moar RELEASE_ASSERTs

Source/WebKit/chromium/src/WebNode.cpp

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 162 matching lines @@
     return m_private->isContentEditable();
 }
 
 bool WebNode::isElementNode() const
 {
     return m_private->isElementNode();
 }
 
 bool WebNode::hasEventListeners(const WebString& eventType) const
 {
+    // FIXME: "permissionrequest" seems like an implementation detail of the
+    //        the browser plug-in. Perhaps the browser plug-in should have
+    //        a more special-purpose mechanism?
+    //        See http://code.google.com/p/chromium/issues/detail?id=189561
+
+    // Please do not add more eventTypes to this list without an API review.
+    RELEASE_ASSERT(eventType == "permissionrequest");
     return m_private->hasEventListeners(eventType);
 }
 
 void WebNode::addEventListener(const WebString& eventType, WebDOMEventListener* listener, bool useCapture)
 {
-    EventListenerWrapper* listenerWrapper =
-        listener->createEventListenerWrapper(eventType, useCapture, m_private.get());
+    // Please do not add more eventTypes to this list without an API review.
+    RELEASE_ASSERT(eventType == "mousedown");
+
+    EventListenerWrapper* listenerWrapper = listener->createEventListenerWrapper(eventType, useCapture, m_private.get());
     // The listenerWrapper is only referenced by the actual Node.  Once it goes
     // away, the wrapper notifies the WebEventListener so it can clear its
     // pointer to it.
     m_private->addEventListener(eventType, adoptRef(listenerWrapper), useCapture);
 }
 
-void WebNode::removeEventListener(const WebString& eventType, WebDOMEventListener* listener, bool useCapture)
-{
-    EventListenerWrapper* listenerWrapper =
-        listener->getEventListenerWrapper(eventType, useCapture, m_private.get());
-    m_private->removeEventListener(eventType, listenerWrapper, useCapture);
-    // listenerWrapper is now deleted.
-}
-
 bool WebNode::dispatchEvent(const WebDOMEvent& event)
 {
     if (!event.isNull())
         return m_private->dispatchEvent(event);
     return false;
 }
 
 void WebNode::simulateClick()
 {
     m_private->dispatchSimulatedClick(0);
@@ skipping 66 matching lines @@
     m_private = node;
     return *this;
 }
 
 WebNode::operator PassRefPtr<Node>() const
 {
     return m_private.get();
 }
 
 } // namespace WebKit