Changeset 89781 in webkit


Ignore:
Timestamp:
Jun 26, 2011 9:45:43 PM (13 years ago)
Author:
abarth@webkit.org
Message:

2011-06-26 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Add [Optional] attributes where appropriate for addEventListener and removeEventListener
​https://bugs.webkit.org/show_bug.cgi?id=63417

This patch doesn't actually change behavior because these interfaces
have the LegacyDefaultOptionalArguments attribute. This patch paves
the way to removing LegacyDefaultOptionalArguments from some of these
interfaces as well as making the project safer for copy-and-pasting
these methods into new IDL files that don't have the
LegacyDefaultOptionalArguments attribute.

  • bindings/scripts/test/TestObj.idl:
  • dom/EventTarget.idl:
  • dom/MediaStream.idl:
  • dom/MessagePort.idl:
  • dom/Node.idl:
  • loader/appcache/DOMApplicationCache.idl:
  • notifications/Notification.idl:
  • page/DOMWindow.idl:
  • page/EventSource.idl:
  • storage/IDBRequest.idl:
  • storage/IDBTransaction.idl:
  • svg/SVGElementInstance.idl:
  • websockets/WebSocket.idl:
  • workers/AbstractWorker.idl:
  • workers/WorkerContext.idl:
  • xml/XMLHttpRequest.idl:
  • xml/XMLHttpRequestUpload.idl:
Location:
trunk/Source/WebCore
Files:
18 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/WebCore/ChangeLog

    r89780 r89781  
     12011-06-26  Adam Barth  <abarth@webkit.org>
     2
     3        Reviewed by Eric Seidel.
     4
     5        Add [Optional] attributes where appropriate for addEventListener and removeEventListener
     6        https://bugs.webkit.org/show_bug.cgi?id=63417
     7
     8        This patch doesn't actually change behavior because these interfaces
     9        have the LegacyDefaultOptionalArguments attribute.  This patch paves
     10        the way to removing LegacyDefaultOptionalArguments from some of these
     11        interfaces as well as making the project safer for copy-and-pasting
     12        these methods into new IDL files that don't have the
     13        LegacyDefaultOptionalArguments attribute.
     14
     15        * bindings/scripts/test/TestObj.idl:
     16        * dom/EventTarget.idl:
     17        * dom/MediaStream.idl:
     18        * dom/MessagePort.idl:
     19        * dom/Node.idl:
     20        * loader/appcache/DOMApplicationCache.idl:
     21        * notifications/Notification.idl:
     22        * page/DOMWindow.idl:
     23        * page/EventSource.idl:
     24        * storage/IDBRequest.idl:
     25        * storage/IDBTransaction.idl:
     26        * svg/SVGElementInstance.idl:
     27        * websockets/WebSocket.idl:
     28        * workers/AbstractWorker.idl:
     29        * workers/WorkerContext.idl:
     30        * xml/XMLHttpRequest.idl:
     31        * xml/XMLHttpRequestUpload.idl:
     32
    1332011-06-26  Adam Barth  <abarth@webkit.org>
    234

  • trunk/Source/WebCore/bindings/scripts/test/TestObj.idl

    r89148 r89781  
    9797        void addEventListener(in DOMString type,
    9898                              in EventListener listener,
    99                               in boolean useCapture);
     99                              in [Optional] boolean useCapture);
    100100        void removeEventListener(in DOMString type,
    101101                                 in EventListener listener,
    102                                  in boolean useCapture);
     102                                 in [Optional] boolean useCapture);
    103103
    104104        // 'CallWith' extended attribute

  • trunk/Source/WebCore/dom/EventTarget.idl

    r89148 r89781  
    3030        [OldStyleObjC] void addEventListener(in DOMString type,
    3131                                             in EventListener listener,
    32                                              in boolean useCapture);
     32                                             in [Optional] boolean useCapture);
    3333        [OldStyleObjC] void removeEventListener(in DOMString type,
    3434                                                in EventListener listener,
    35                                                 in boolean useCapture);
     35                                                in [Optional] boolean useCapture);
    3636        boolean dispatchEvent(in Event event)
    3737            raises(EventException);

  • trunk/Source/WebCore/dom/MediaStream.idl

    r89567 r89781  
    4040        void addEventListener(in DOMString type,
    4141                              in EventListener listener,
    42                               in boolean useCapture);
     42                              in [Optional] boolean useCapture);
    4343        void removeEventListener(in DOMString type,
    4444                                 in EventListener listener,
    45                                  in boolean useCapture);
     45                                 in [Optional] boolean useCapture);
    4646        boolean dispatchEvent(in Event event)
    4747            raises(EventException);

  • trunk/Source/WebCore/dom/MessagePort.idl

    r89148 r89781  
    4949        void addEventListener(in DOMString type,
    5050                              in EventListener listener,
    51                               in boolean useCapture);
     51                              in [Optional] boolean useCapture);
    5252        void removeEventListener(in DOMString type,
    5353                                 in EventListener listener,
    54                                  in boolean useCapture);
     54                                 in [Optional] boolean useCapture);
    5555        boolean dispatchEvent(in Event evt)
    5656            raises(EventException);

  • trunk/Source/WebCore/dom/Node.idl

    r89148 r89781  
    141141        void addEventListener(in DOMString type,
    142142                              in EventListener listener,
    143                               in boolean useCapture);
     143                              in [Optional] boolean useCapture);
    144144        void removeEventListener(in DOMString type,
    145145                                 in EventListener listener,
    146                                  in boolean useCapture);
     146                                 in [Optional] boolean useCapture);
    147147        boolean dispatchEvent(in Event event)
    148148            raises(EventException);

  • trunk/Source/WebCore/loader/appcache/DOMApplicationCache.idl

    r89148 r89781  
    6161        void addEventListener(in DOMString type,
    6262                              in EventListener listener,
    63                               in boolean useCapture);
     63                              in [Optional] boolean useCapture);
    6464        void removeEventListener(in DOMString type,
    6565                                 in EventListener listener,
    66                                  in boolean useCapture);
     66                                 in [Optional] boolean useCapture);
    6767        boolean dispatchEvent(in Event evt)
    6868            raises(EventException);

  • trunk/Source/WebCore/notifications/Notification.idl

    r89148 r89781  
    5353        void addEventListener(in DOMString type,
    5454                              in EventListener listener,
    55                               in boolean useCapture);
     55                              in [Optional] boolean useCapture);
    5656        void removeEventListener(in DOMString type,
    5757                                 in EventListener listener,
    58                                  in boolean useCapture);
     58                                 in [Optional] boolean useCapture);
    5959        boolean dispatchEvent(in Event evt)
    6060            raises(EventException);

  • trunk/Source/WebCore/page/DOMWindow.idl

    r89732 r89781  
    341341        [Custom] void addEventListener(in DOMString type,
    342342                                      in EventListener listener,
    343                                       in boolean useCapture);
     343                                      in [Optional] boolean useCapture);
    344344        [Custom] void removeEventListener(in DOMString type,
    345345                                          in EventListener listener,
    346                                           in boolean useCapture);
     346                                          in [Optional] boolean useCapture);
    347347        boolean dispatchEvent(in Event evt)
    348348            raises(EventException);

  • trunk/Source/WebCore/page/EventSource.idl

    r89148 r89781  
    6161        void addEventListener(in DOMString type,
    6262                              in EventListener listener,
    63                               in boolean useCapture);
     63                              in [Optional] boolean useCapture);
    6464        void removeEventListener(in DOMString type,
    6565                                 in EventListener listener,
    66                                  in boolean useCapture);
     66                                 in [Optional] boolean useCapture);
    6767        boolean dispatchEvent(in Event evt)
    6868            raises(EventException);

  • trunk/Source/WebCore/storage/IDBRequest.idl

    r89148 r89781  
    5757        void addEventListener(in DOMString type,
    5858                              in EventListener listener,
    59                               in boolean useCapture);
     59                              in [Optional] boolean useCapture);
    6060        void removeEventListener(in DOMString type,
    6161                                 in EventListener listener,
    62                                  in boolean useCapture);
     62                                 in [Optional] boolean useCapture);
    6363        boolean dispatchEvent(in Event evt)
    6464            raises(EventException);

  • trunk/Source/WebCore/storage/IDBTransaction.idl

    r89148 r89781  
    5252        void addEventListener(in DOMString type,
    5353                              in EventListener listener,
    54                               in boolean useCapture);
     54                              in [Optional] boolean useCapture);
    5555        void removeEventListener(in DOMString type,
    5656                                 in EventListener listener,
    57                                  in boolean useCapture);
     57                                 in [Optional] boolean useCapture);
    5858        boolean dispatchEvent(in Event evt)
    5959            raises(EventException);

  • trunk/Source/WebCore/svg/SVGElementInstance.idl

    r89148 r89781  
    9191        void addEventListener(in DOMString type,
    9292                              in EventListener listener,
    93                               in boolean useCapture);
     93                              in [Optional] boolean useCapture);
    9494        void removeEventListener(in DOMString type,
    9595                                 in EventListener listener,
    96                                  in boolean useCapture);
     96                                 in [Optional] boolean useCapture);
    9797        boolean dispatchEvent(in Event event)
    9898            raises(EventException);

  • trunk/Source/WebCore/websockets/WebSocket.idl

    r89148 r89781  
    6666        void addEventListener(in DOMString type,
    6767                              in EventListener listener,
    68                               in boolean useCapture);
     68                              in [Optional] boolean useCapture);
    6969        void removeEventListener(in DOMString type,
    7070                                 in EventListener listener,
    71                                  in boolean useCapture);
     71                                 in [Optional] boolean useCapture);
    7272        boolean dispatchEvent(in Event evt)
    7373            raises(EventException);

  • trunk/Source/WebCore/workers/AbstractWorker.idl

    r89148 r89781  
    4444        void addEventListener(in DOMString type,
    4545                              in EventListener listener,
    46                               in boolean useCapture);
     46                              in [Optional] boolean useCapture);
    4747        void removeEventListener(in DOMString type,
    4848                                 in EventListener listener,
    49                                  in boolean useCapture);
     49                                 in [Optional] boolean useCapture);
    5050        boolean dispatchEvent(in Event evt)
    5151            raises(EventException);

  • trunk/Source/WebCore/workers/WorkerContext.idl

    r89148 r89781  
    7070        void addEventListener(in DOMString type,
    7171                              in EventListener listener,
    72                               in boolean useCapture);
     72                              in [Optional] boolean useCapture);
    7373        void removeEventListener(in DOMString type,
    7474                                 in EventListener listener,
    75                                  in boolean useCapture);
     75                                 in [Optional] boolean useCapture);
    7676        boolean dispatchEvent(in Event evt)
    7777            raises(EventException);

  • trunk/Source/WebCore/xml/XMLHttpRequest.idl

    r89148 r89781  
    115115        void addEventListener(in DOMString type,
    116116                              in EventListener listener,
    117                               in boolean useCapture);
     117                              in [Optional] boolean useCapture);
    118118        void removeEventListener(in DOMString type,
    119119                                 in EventListener listener,
    120                                  in boolean useCapture);
     120                                 in [Optional] boolean useCapture);
    121121        boolean dispatchEvent(in Event evt)
    122122            raises(EventException);

  • trunk/Source/WebCore/xml/XMLHttpRequestUpload.idl

    r89148 r89781  
    4646        void addEventListener(in DOMString type,
    4747                              in EventListener listener,
    48                               in boolean useCapture);
     48                              in [Optional] boolean useCapture);
    4949        void removeEventListener(in DOMString type,
    5050                                 in EventListener listener,
    51                                  in boolean useCapture);
     51                                 in [Optional] boolean useCapture);
    5252        boolean dispatchEvent(in Event evt)
    5353            raises(EventException);
Note: See TracChangeset for help on using the changeset viewer.