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

public/platform/modules/presentation/WebPresentationSessionClient.h

 // 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.
 
 #ifndef WebPresentationSessionClient_h
 #define WebPresentationSessionClient_h
 
 #include "public/platform/WebString.h"
 
 namespace blink {
 
 // The implementation the embedder has to provide for the Presentation API to work.
 class WebPresentationSessionClient {
 public:
+    enum SessionState {

[mlamouri (slow - plz ping), 2015/03/23 17:54:16]

Could you make this an enum class?

It will require you to rename SessionStafeValue as Value but it will allow
forward declaration.

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

Done.

+        SessionStateConnected = 0,
+        SessionStateDisconnected,
+        SessionStateUnknown,

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

When would we use "Unknown"?

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

When WFT happens?

+        SessionStateLast = SessionStateUnknown

[mlamouri (slow - plz ping), 2015/03/23 17:54:16]

Do you use SessionStateLast? It is commonly used for IPC but I assume you will
be using Mojo, right?

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

No, I have it for consistency only.

+    };
+
     virtual ~WebPresentationSessionClient() { }
 
     virtual WebString getId() = 0;
     virtual WebString getUrl() = 0;
 };
 
 } // namespace blink
 
 #endif // WebPresentationSessionClient_h