MediaStream API: Changing the device enumeration to be async

Source/modules/mediastream/MediaStreamTrackSourcesRequest.cpp

Index: Source/modules/mediastream/MediaStreamTrackSourcesRequest.cpp
diff --git a/Source/modules/mediastream/MediaStreamTrackSourcesRequest.cpp b/Source/modules/mediastream/MediaStreamTrackSourcesRequest.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..9bd8a756684e9a255104ccbe946c81531e2adb4a
--- /dev/null
+++ b/Source/modules/mediastream/MediaStreamTrackSourcesRequest.cpp
@@ -0,0 +1,71 @@
+/*
+ * 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.
+ */
+
+#include "config.h"
+
+#include "modules/mediastream/MediaStreamTrackSourcesRequest.h"
+
+#include "core/dom/ScriptExecutionContext.h"
+#include "modules/mediastream/MediaStreamTrackSourcesCallback.h"
+#include "public/platform/WebSourceInfo.h"
+#include "weborigin/SecurityOrigin.h"
+
+namespace WebCore {
+
+PassRefPtr<MediaStreamTrackSourcesRequest> MediaStreamTrackSourcesRequest::create(ScriptExecutionContext* context, PassRefPtr<MediaStreamTrackSourcesCallback> callback)
+{
+    return adoptRef(new MediaStreamTrackSourcesRequest(context, callback));
+}
+
+MediaStreamTrackSourcesRequest::MediaStreamTrackSourcesRequest(ScriptExecutionContext* context, PassRefPtr<MediaStreamTrackSourcesCallback> callback)
+    : m_callback(callback)
+    , m_scheduledEventTimer(this, &MediaStreamTrackSourcesRequest::scheduledEventTimerFired)
+{
+    m_origin = context->securityOrigin()->toString();
+}
+
+MediaStreamTrackSourcesRequest::~MediaStreamTrackSourcesRequest()
+{
+}
+
+void MediaStreamTrackSourcesRequest::requestSucceeded(const WebKit::WebVector<WebKit::WebSourceInfo>& webSourceInfos)
+{
+    if (m_callback && !m_scheduledEventTimer.isActive()) {

[jochen (gone - plz use gerrit), 2013/06/19 13:09:33]

is it possible that requestSucceeded() is called multiple times? It seems like
the source infos are ignored then?

[Tommy Widenflycht, 2013/06/19 13:11:59]

No, it isn't allowed to call this more than once.

+        for (size_t i = 0; i < webSourceInfos.size(); ++i)
+            m_sourceInfos.append(SourceInfo::create(webSourceInfos[i]));
+
+        m_protect = this;
+        m_scheduledEventTimer.startOneShot(0);
+    }
+}
+
+void MediaStreamTrackSourcesRequest::scheduledEventTimerFired(Timer<MediaStreamTrackSourcesRequest>*)
+{
+    m_callback->handleEvent(m_sourceInfos);
+    m_callback.clear();
+    m_protect.release();
+}
+
+} // namespace WebCore