Marks session as connected upon StartSession success

Source/modules/presentation/PresentationSession.h

 // Copyright 2014 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 PresentationSession_h
 #define PresentationSession_h
 
 #include "core/events/EventTarget.h"
 #include "core/fileapi/Blob.h"
 #include "core/fileapi/FileError.h"
@@ skipping 13 matching lines @@
 class Presentation;
 class PresentationController;
 
 class PresentationSession final
     : public RefCountedGarbageCollectedEventTargetWithInlineData<PresentationSession>
     , public DOMWindowProperty {
     REFCOUNTED_GARBAGE_COLLECTED_EVENT_TARGET(PresentationSession);
     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();
 
     // EventTarget implementation.
     virtual const AtomicString& interfaceName() const override;
     virtual ExecutionContext* executionContext() const override;
 
     DECLARE_VIRTUAL_TRACE();
 
     const String id() const { return m_id; }
@@ skipping 38 matching lines @@
         Message(PassRefPtr<BlobDataHandle> blobDataHandle)
             : type(MessageTypeBlob)
             , blobDataHandle(blobDataHandle) { }
 
         MessageType type;
         String text;
         RefPtr<DOMArrayBuffer> arrayBuffer;
         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
     // detached from the document.
     PresentationController* presentationController();
 
     bool canSendMessage(ExceptionState&);
     void handleMessageQueue();
 
     // Callbacks invoked from BlobLoader.
     void didFinishLoadingBlob(PassRefPtr<DOMArrayBuffer>);
     void didFailLoadingBlob(FileError::ErrorCode);
 
     String m_id;
     String m_url;
     WebPresentationSessionState m_state;
 
     // For Blob data handling.
     Member<BlobLoader> m_blobLoader;
     Deque<OwnPtr<Message>> m_messages;
 };
 
 } // namespace blink
 
 #endif // PresentationSession_h