[Presentation API] fire onconnectionavailable and onconnect event asynchronously

third_party/WebKit/Source/modules/presentation/PresentationConnectionCallbacks.cpp

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "modules/presentation/PresentationConnectionCallbacks.h"
 
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "core/dom/DOMException.h"
+#include "core/dom/ExecutionContextTask.h"
 #include "modules/presentation/PresentationConnection.h"
+#include "modules/presentation/PresentationConnectionAvailableEvent.h"
 #include "modules/presentation/PresentationError.h"
 #include "modules/presentation/PresentationRequest.h"
 #include "public/platform/modules/presentation/WebPresentationConnectionClient.h"
 #include "public/platform/modules/presentation/WebPresentationError.h"
 #include "wtf/PtrUtil.h"
 #include <memory>
 
 namespace blink {
 
 PresentationConnectionCallbacks::PresentationConnectionCallbacks(
     ScriptPromiseResolver* resolver,
     PresentationRequest* request)
     : m_resolver(resolver), m_request(request) {
   ASSERT(m_resolver);
   ASSERT(m_request);
 }
 
 void PresentationConnectionCallbacks::onSuccess(
     std::unique_ptr<WebPresentationConnectionClient>
         PresentationConnectionClient) {
   std::unique_ptr<WebPresentationConnectionClient> result(
       wrapUnique(PresentationConnectionClient.release()));
 
   if (!m_resolver->getExecutionContext() ||
       m_resolver->getExecutionContext()->activeDOMObjectsAreStopped())
     return;
-  m_resolver->resolve(PresentationConnection::take(
-      m_resolver.get(), std::move(result), m_request));
+
+  auto* connection = PresentationConnection::take(m_resolver.get(),
+                                                  std::move(result), m_request);
+  m_resolver->resolve(connection);
+
+  m_resolver->getExecutionContext()->postTask(
+      BLINK_FROM_HERE,
+      createSameThreadTask(
+          &PresentationRequest::dispatchConnectionAvailableEvent, m_request,
+          wrapPersistent(connection)));

[mlamouri (slow - plz ping), 2016/12/05 13:39:10]

Any reason I'm missing why this is not in the ::take() from
PresentationConnection?

 }
 
 void PresentationConnectionCallbacks::onError(
     const WebPresentationError& error) {
   if (!m_resolver->getExecutionContext() ||
       m_resolver->getExecutionContext()->activeDOMObjectsAreStopped())
     return;
   m_resolver->reject(PresentationError::take(m_resolver.get(), error));
 }
 
 }  // namespace blink