MediaStream API: Changing the device enumeration to be async

Source/modules/mediastream/MediaStreamTrackSourcesRequestImpl.h

 /*
  * Copyright (C) 2013 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:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
  *
  * THIS SOFTWARE IS PROVIDED BY GOOGLE INC. ``AS IS'' AND ANY
  * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
  * PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL GOOGLE INC. OR
  * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
  * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT 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 RTCDTMFToneChangeEvent_h
-#define RTCDTMFToneChangeEvent_h
+#ifndef MediaStreamTrackSourcesRequestImpl_h
+#define MediaStreamTrackSourcesRequestImpl_h
 
-#include "core/dom/Event.h"
-#include "wtf/text/AtomicString.h"
+#include "core/dom/ActiveDOMObject.h"
+#include "core/platform/mediastream/MediaStreamTrackSourcesRequest.h"
 
 namespace WebCore {
 
-struct RTCDTMFToneChangeEventInit : public EventInit {
-    String tone;
-};
+class MediaStreamTrackSourcesCallback;
 
-class RTCDTMFToneChangeEvent : public Event {
+class MediaStreamTrackSourcesRequestImpl : public MediaStreamTrackSourcesRequest, public ActiveDOMObject {

[abarth-chromium, 2013/06/11 23:50:44]

Why does this need to be an active DOM object?  The
MediaStreamTrackSourcesCallback object should already be notified when the
context shuts down...

[Tommy Widenflycht, 2013/06/12 12:29:34]

Double safety belts... Fixed.

 public:
-    virtual ~RTCDTMFToneChangeEvent();
+    static PassRefPtr<MediaStreamTrackSourcesRequestImpl> create(ScriptExecutionContext*, PassRefPtr<MediaStreamTrackSourcesCallback>);
+    virtual ~MediaStreamTrackSourcesRequestImpl();
 
-    static PassRefPtr<RTCDTMFToneChangeEvent> create();
-    static PassRefPtr<RTCDTMFToneChangeEvent> create(const String& tone);
-    static PassRefPtr<RTCDTMFToneChangeEvent> create(const AtomicString& type, const RTCDTMFToneChangeEventInit& initializer);
+    virtual String url() { return m_url; }
 
-    const String& tone() const;
+    virtual void requestSucceeded(const WebKit::WebVector<WebKit::WebSourceInfo>&) OVERRIDE;
 
-    virtual const AtomicString& interfaceName() const;
+    // ActiveDOMObject
+    virtual void stop() OVERRIDE;
 
 private:
-    RTCDTMFToneChangeEvent();
-    explicit RTCDTMFToneChangeEvent(const String& tone);
-    explicit RTCDTMFToneChangeEvent(const RTCDTMFToneChangeEventInit&);
+    MediaStreamTrackSourcesRequestImpl(ScriptExecutionContext*, PassRefPtr<MediaStreamTrackSourcesCallback>);
 
-    String m_tone;
+    void clear();
+
+    String m_url;
+    RefPtr<MediaStreamTrackSourcesCallback> m_callback;
 };
 
 } // namespace WebCore
 
-#endif // RTCDTMFToneChangeEvent_h
+#endif // MediaStreamTrackSourcesRequestImpl_h
+
+