IDL: make optional arguments (without default) explicit sometimes

Source/modules/websockets/DOMWebSocket.h

 /*
  * Copyright (C) 2011 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 13 matching lines @@
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #ifndef DOMWebSocket_h
 #define DOMWebSocket_h
 
+#include "bindings/core/v8/Optional.h"
 #include "core/dom/ActiveDOMObject.h"
 #include "core/events/EventListener.h"
 #include "modules/EventTargetModules.h"
 #include "modules/websockets/WebSocketChannel.h"
 #include "modules/websockets/WebSocketChannelClient.h"
 #include "platform/Timer.h"
 #include "platform/heap/Handle.h"
 #include "platform/weborigin/KURL.h"
 #include "wtf/Deque.h"
 #include "wtf/Forward.h"
 #include "wtf/text/AtomicStringHash.h"
 
 namespace blink {
 
 class Blob;
 class ExceptionState;
 
 class DOMWebSocket : public RefCountedGarbageCollectedWillBeGarbageCollectedFinalized<DOMWebSocket>, public EventTargetWithInlineData, public ActiveDOMObject, public WebSocketChannelClient {
     DEFINE_EVENT_TARGET_REFCOUNTING_WILL_BE_REMOVED(RefCountedGarbageCollected<DOMWebSocket>);
     USING_GARBAGE_COLLECTED_MIXIN(DOMWebSocket);
 public:
     static const char* subprotocolSeperator();
     // DOMWebSocket instances must be used with a wrapper since this class's
     // lifetime management is designed assuming the V8 holds a ref on it while
     // hasPendingActivity() returns true.
-    static DOMWebSocket* create(ExecutionContext*, const String& url, ExceptionState&);
     static DOMWebSocket* create(ExecutionContext*, const String& url, const String& protocol, ExceptionState&);
     static DOMWebSocket* create(ExecutionContext*, const String& url, const Vector<String>& protocols, ExceptionState&);
     virtual ~DOMWebSocket();
 
     enum State {
         CONNECTING = 0,
         OPEN = 1,
         CLOSING = 2,
         CLOSED = 3
     };
 
     void connect(const String& url, const Vector<String>& protocols, ExceptionState&);
 
     void send(const String& message, ExceptionState&);
     void send(ArrayBuffer*, ExceptionState&);
     void send(ArrayBufferView*, ExceptionState&);
     void send(Blob*, ExceptionState&);
 
-    // To distinguish close method call with the code parameter from one
-    // without, we have these three signatures. Use of
-    // Optional=DefaultIsUndefined in the IDL file doesn't help for now since
-    // it's bound to a value of 0 which is indistinguishable from the case 0
-    // is passed as code parameter.
-    void close(unsigned short code, const String& reason, ExceptionState&);
-    void close(ExceptionState&);
-    void close(unsigned short code, ExceptionState&);
+    void close(Optional<unsigned short> code, const String& reason, ExceptionState&);
 
     const KURL& url() const;
     State readyState() const;
     unsigned long bufferedAmount() const;
 
     String protocol() const;
     String extensions() const;
 
     String binaryType() const;
     void setBinaryType(const String&);
@@ skipping 126 matching lines @@
     String m_subprotocol;
     String m_extensions;
 
     Member<EventQueue> m_eventQueue;
     Timer<DOMWebSocket> m_bufferedAmountConsumeTimer;
 };
 
 } // namespace blink
 
 #endif // DOMWebSocket_h