[PresentationAPI] Plumbing |onstatechange| event for the PresentationSession from Blink to platform/

Source/modules/presentation/PresentationController.cpp

Index: Source/modules/presentation/PresentationController.cpp
diff --git a/Source/modules/presentation/PresentationController.cpp b/Source/modules/presentation/PresentationController.cpp
index e16165776cef9fd356f81d69ab16fe8a0e5a526e..7f134cb196984e7157773556c1091b474c9b9c9f 100644
--- a/Source/modules/presentation/PresentationController.cpp
+++ b/Source/modules/presentation/PresentationController.cpp
@@ -86,6 +86,17 @@ void PresentationController::didStartDefaultSession(WebPresentationSessionClient
     m_presentation->didStartDefaultSession(session);
 }
 
+void PresentationController::didChangeSessionState(WebPresentationSessionClient* sessionClient, WebPresentationSessionClient::SessionState state)
+{
+    if (m_presentation) {
+        PresentationSession* session = m_presentation->findSession(sessionClient);

[Peter Beverloo, 2015/03/20 19:07:40]

Do you expect more events following this paradigm? Otherwise you could avoid
exposing something like findSession() in favor of following the rest of your
plumbing, e.g. isAvailableChangeWatched().


if (m_presentation)
    m_presentation->didChangeSessionState(sessionClient, state);

[whywhat, 2015/03/23 21:12:16]

Yes, there will be at least an event about receiving a message. I can move
findSession to the private section of Presentation class though and follow the
pattern of delegating to Presentation.

+        if (session)
+            session->didChangeState(state);
+    }
+
+    PresentationSession::dispose(sessionClient);
+}
+
 void PresentationController::startSession(const String& presentationUrl, const String& presentationId, WebPresentationSessionClientCallbacks* callbacks)
 {
     if (!m_client) {