Marks session as connected upon StartSession success

Source/modules/presentation/PresentationSession.h

Index: Source/modules/presentation/PresentationSession.h
diff --git a/Source/modules/presentation/PresentationSession.h b/Source/modules/presentation/PresentationSession.h
index 410fa047d9016fb8ce3394c5eaca0d3f918fc874..0d01ec9d2d4d5891187712b497d3d84e0bca7952 100644
--- a/Source/modules/presentation/PresentationSession.h
+++ b/Source/modules/presentation/PresentationSession.h
@@ -31,7 +31,7 @@ class PresentationSession final
     WILL_BE_USING_GARBAGE_COLLECTED_MIXIN(PresentationSession);
     DEFINE_WRAPPERTYPEINFO();
 public:
-    static PresentationSession* take(WebPresentationSessionClient*, Presentation*);
+    static PresentationSession* take(WebPresentationSessionClient*, Presentation*, WebPresentationSessionState);

[whywhat, 2015/06/25 19:57:53]

I'd prefer the state be passed as part of the WebPresentationSessionClient. Why
is it different from URL and id and is specified separately? I don't think the
Callback class is what should determine that the session is connected.
I understand that it's even more changes throughout though.

     static void dispose(WebPresentationSessionClient*);
     virtual ~PresentationSession();
 
@@ -90,7 +90,7 @@ private:
         RefPtr<BlobDataHandle> blobDataHandle;
     };
 
-    PresentationSession(LocalFrame*, const String& id, const String& url);
+    PresentationSession(LocalFrame*, const String& id, const String& url, WebPresentationSessionState);
 
     // Returns the |PresentationController| object associated with the frame
     // |Presentation| corresponds to. Can return |nullptr| if the frame is