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

public/platform/modules/presentation/WebPresentationController.h

Index: public/platform/modules/presentation/WebPresentationController.h
diff --git a/public/platform/modules/presentation/WebPresentationController.h b/public/platform/modules/presentation/WebPresentationController.h
index 8ed2da2f86fa5916a696978cba54dd8cd75a50ff..e0ff7e9e556878a46a401ce98c33a5a896e49498 100644
--- a/public/platform/modules/presentation/WebPresentationController.h
+++ b/public/platform/modules/presentation/WebPresentationController.h
@@ -5,9 +5,9 @@
 #ifndef WebPresentationController_h
 #define WebPresentationController_h
 
-namespace blink {
+#include "public/platform/modules/presentation/WebPresentationSessionClient.h"

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

You should make the enum an enum class so you can forward declare it.

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

Done.

 
-class WebPresentationSessionClient;
+namespace blink {
 
 // The delegate Blink provides to WebPresentationClient in order to get updates.
 class WebPresentationController {
@@ -24,6 +24,9 @@ public:
     // Called when the presentation session is started by the embedder using
     // the default presentation URL and id.
     virtual void didStartDefaultSession(WebPresentationSessionClient*) = 0;
+
+    // Called when the state of a session changes.
+    virtual void didChangeSessionState(WebPresentationSessionClient*, WebPresentationSessionClient::SessionState) = 0;

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

This means that the Chromium patch has to land first, otherwise you'll have
unresolved pure virtual methods in your class. The patch' descriptions don't
agree.

[Peter Beverloo, 2015/03/20 19:16:59]

Scrap that :-).

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

Acknowledged.

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

Acknowledged.

 };
 
 } // namespace blink