Marks session as connected upon StartSession success

Source/modules/presentation/PresentationSessionClientCallbacks.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 "config.h"
 #include "modules/presentation/PresentationSessionClientCallbacks.h"
 
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "core/dom/DOMException.h"
 #include "core/dom/ExceptionCode.h"
@@ skipping 21 matching lines @@
 }
 
 void PresentationSessionClientCallbacks::onSuccess(WebPresentationSessionClient* client)
 {
     ASSERT(client);
     if (!m_resolver->executionContext() || m_resolver->executionContext()->activeDOMObjectsAreStopped()) {
         PresentationSession::dispose(client);
         return;
     }
 
-    m_resolver->resolve(PresentationSession::take(client, m_presentation));
+    PresentationSession* session = PresentationSession::take(client, m_presentation);
+    session->didChangeState(WebPresentationSessionState::Connected);

[whywhat, 2015/06/25 17:33:45]

Seems wasteful to dispatch an event to just created and not yet returned to the
page session.
Maybe it's better to pass the initial state with WebPresentationSessionClient?
Or you could just set the session's state to Connected in the constructor.

[haibinlu, 2015/06/25 18:45:36]

Make sense. Pass in the initial state.

+    m_resolver->resolve(session);
 }
 
 void PresentationSessionClientCallbacks::onError(WebPresentationError* error)
 {
     ASSERT(error);
     if (!m_resolver->executionContext() || m_resolver->executionContext()->activeDOMObjectsAreStopped()) {
         PresentationError::dispose(error);
         return;
     }
 
     m_resolver->reject(PresentationError::take(error));
 }
 
 } // namespace blink