[Presentation API] Plumbing of |onstatechange| from the Mojo service to the public/platform

content/common/presentation/presentation_service.mojom

 // 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.
 
 module presentation;
 
 struct PresentationSessionInfo {
     string url;
     string id;
 };
 
+enum PresentationSessionState {
+    CONNECTED,
+    DISCONNECTED
+};
+
 enum PresentationErrorType {
     NO_AVAILABLE_SCREENS,
     SESSION_REQUEST_CANCELLED,
     NO_PRESENTATION_FOUND,
     UNKNOWN,
 };
 
 struct PresentationError {
     PresentationErrorType error_type;
     string message;
@@ skipping 43 matching lines @@
 
     // Called when joinSession() is called by the frame. The result callback
     // works the same as for the method above. JoinSession will join a known
     // session (i.e. when the page navigates or the user opens another tab)
     // silently and without user action.
     JoinSession(string presentation_url, string? presentation_id)
         => (PresentationSessionInfo? sessionInfo, PresentationError? error);
 
     // Called when closeSession() is called by the frame.
     CloseSession(string presentation_url, string presentation_id);
+
+    // Called when the frame is ready to process the next state
+    // change. Returns the last session state if it’s
+    // changed since the last time the callback was called. Might

[Peter Beverloo, 2015/03/20 19:13:39]

nit: advocating for 70-character line length limits rather than 80-character
ones? :)

[whywhat, 2015/03/24 19:20:57]

Copy paste from the doc about the Mojo interface. Docs are narrower than code.

+    // cause the event fired with the initial state change.
+    ListenForSessionStateChange()

[mark a. foltz, 2015/03/20 21:22:43]

GetSessionState() for consistency with GetScreenAvailability()?

[whywhat, 2015/03/24 19:20:57]

But this is consistent with ListenForDefaultSessionStart. Probably should change
GetScreenAvailability to ListenForScreenAvailability in a separate CL?

[mark a. foltz, 2015/03/24 22:53:34]

SGTM

+        => (PresentationSessionInfo sessionInfo,
+            PresentationSessionState newState);
 };