[PresentationAPI] Plumbing send(Blob) from PresentationSession IDL to platform/.

Source/modules/presentation/PresentationSession.cpp

 // 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.
 
 #include "config.h"
 #include "modules/presentation/PresentationSession.h"
 
 #include "core/dom/DOMArrayBuffer.h"
 #include "core/dom/DOMArrayBufferView.h"
 #include "core/dom/Document.h"
 #include "core/dom/ExceptionCode.h"
 #include "core/events/Event.h"
 #include "core/events/MessageEvent.h"
+#include "core/fileapi/FileReaderLoader.h"
+#include "core/fileapi/FileReaderLoaderClient.h"
 #include "core/frame/LocalFrame.h"
 #include "modules/EventTargetModules.h"
 #include "modules/presentation/Presentation.h"
 #include "modules/presentation/PresentationController.h"
 #include "public/platform/modules/presentation/WebPresentationSessionClient.h"
 #include "wtf/Assertions.h"
 #include "wtf/OwnPtr.h"
 #include "wtf/text/AtomicString.h"
 
 namespace blink {
@@ skipping 16 matching lines @@
     return disconnectedValue;
 }
 
 void throwPresentationDisconnectedError(ExceptionState& exceptionState)
 {
     exceptionState.throwDOMException(InvalidStateError, "Presentation session is disconnected.");
 }
 
 } // namespace
 
+class PresentationSession::BlobLoader final : public GarbageCollectedFinalized<PresentationSession::BlobLoader>, public FileReaderLoaderClient {
+public:
+    BlobLoader(PassRefPtr<BlobDataHandle> blobDataHandle, PresentationSession* presentationSession)
+        : m_presentationSession(presentationSession)
+        , m_loader(FileReaderLoader::ReadAsArrayBuffer, this)
+    {
+        m_loader.start(m_presentationSession->executionContext(), blobDataHandle);
+    }
+    virtual ~BlobLoader() { }

[mlamouri (slow - plz ping), 2015/05/19 13:08:55]

override maybe?

[USE s.singapati at gmail.com, 2015/05/21 19:49:54]

Done.

+
+    // FileReaderLoaderClient functions.
+    virtual void didStartLoading() override { }
+    virtual void didReceiveData() override { }
+    virtual void didFinishLoading() override

[mlamouri (slow - plz ping), 2015/05/19 13:08:54]

No need for virtual if you already have override.

[USE s.singapati at gmail.com, 2015/05/21 19:49:54]

Done.

+    {
+        m_presentationSession->didFinishLoadingBlob(m_loader.arrayBufferResult());
+    }
+    virtual void didFail(FileError::ErrorCode errorCode) override
+    {
+        m_presentationSession->didFailLoadingBlob(errorCode);
+    }
+
+    void cancel()
+    {
+        m_loader.cancel();
+    }
+
+    DEFINE_INLINE_TRACE()
+    {
+        visitor->trace(m_presentationSession);
+    }
+
+private:
+    Member<PresentationSession> m_presentationSession;
+    FileReaderLoader m_loader;
+};
+
 PresentationSession::PresentationSession(LocalFrame* frame, const String& id, const String& url)
     : DOMWindowProperty(frame)
     , m_id(id)
     , m_url(url)
     , m_state(WebPresentationSessionState::Disconnected)
 {
 }
 
 PresentationSession::~PresentationSession()
 {
+    ASSERT(!m_blobLoader);
 }
 
 // static
 PresentationSession* PresentationSession::take(WebPresentationSessionClient* clientRaw, Presentation* presentation)
 {
     ASSERT(clientRaw);
     ASSERT(presentation);
     OwnPtr<WebPresentationSessionClient> client = adoptPtr(clientRaw);
 
     PresentationSession* session = new PresentationSession(presentation->frame(), client->getId(), client->getUrl());
@@ skipping 13 matching lines @@
 }
 
 ExecutionContext* PresentationSession::executionContext() const
 {
     if (!frame())
         return nullptr;
     return frame()->document();}
 
 DEFINE_TRACE(PresentationSession)
 {
+    visitor->trace(m_blobLoader);
     RefCountedGarbageCollectedEventTargetWithInlineData<PresentationSession>::trace(visitor);
     DOMWindowProperty::trace(visitor);
 }
 
 const AtomicString& PresentationSession::state() const
 {
     return SessionStateToString(m_state);
 }
 
 void PresentationSession::send(const String& message, ExceptionState& exceptionState)
@@ skipping 26 matching lines @@
     if (m_state == WebPresentationSessionState::Disconnected) {
         throwPresentationDisconnectedError(exceptionState);
         return;
     }
 
     PresentationController* controller = presentationController();
     if (controller)
         controller->send(m_url, m_id, data, size);
 }
 
+void PresentationSession::send(Blob* data, ExceptionState& exceptionState)
+{
+    ASSERT(data);
+    if (m_state == WebPresentationSessionState::Disconnected) {
+        throwPresentationDisconnectedError(exceptionState);
+        return;
+    }
+
+    m_messages.append(adoptPtr(new Message(data->blobDataHandle())));
+    handleMessageQueue();
+}
+
+void PresentationSession::handleMessageQueue()
+{
+    if (!m_messages.isEmpty() && !m_blobLoader) {

[mlamouri (slow - plz ping), 2015/05/19 13:08:54]

if (m_messages.isEmpty() || m_blobLoader)
  return;

[USE s.singapati at gmail.com, 2015/05/21 19:49:54]

Done.

+        Message* message = m_messages.first().get();
+        ASSERT(!m_blobLoader);
+        m_blobLoader = new BlobLoader(message->blobDataHandle, this);
+    }
+}
+
 void PresentationSession::didReceiveTextMessage(const String& message)
 {
     dispatchEvent(MessageEvent::create(message));
 }
 
 void PresentationSession::close()
 {
     if (m_state != WebPresentationSessionState::Connected)
         return;
     PresentationController* controller = presentationController();
     if (controller)
         controller->closeSession(m_url, m_id);
+
+    // Cancel current Blob loading if any.
+    if (m_blobLoader) {
+        m_blobLoader->cancel();
+        m_blobLoader.clear();
+    }
+
+    // Clear Blob message queue.
+    Deque<OwnPtr<Message>> empty;
+    m_messages.swap(empty);
 }
 
 bool PresentationSession::matches(WebPresentationSessionClient* client) const
 {
     return client && m_url == static_cast<String>(client->getUrl()) && m_id == static_cast<String>(client->getId());
 }
 
 void PresentationSession::didChangeState(WebPresentationSessionState state)
 {
     if (m_state == state)
         return;
 
     m_state = state;
     dispatchEvent(Event::create(EventTypeNames::statechange));
 }
 
 PresentationController* PresentationSession::presentationController()
 {
     if (!frame())
         return nullptr;
     return PresentationController::from(*frame());
 }
 
+void PresentationSession::didFinishLoadingBlob(PassRefPtr<DOMArrayBuffer> buffer)
+{
+    m_blobLoader.clear();
+    ASSERT(buffer && buffer->buffer());
+
+    PresentationController* controller = presentationController();
+    if (controller)
+        controller->sendBlobData(m_url, m_id, static_cast<const uint8_t*>(buffer->data()), buffer->byteLength());
+
+    ASSERT(m_messages.size() > 0);

[mlamouri (slow - plz ping), 2015/05/19 13:08:55]

You probably just want to check that it is not empty. You might also want to
move the check at the beginning of this method?

[USE s.singapati at gmail.com, 2015/05/21 19:49:54]

Done.

+    m_messages.removeFirst();
+    handleMessageQueue();
+}
+
+void PresentationSession::didFailLoadingBlob(FileError::ErrorCode errorCode)
+{
+    m_blobLoader.clear();
+    // FIXME: generate error message?

[mlamouri (slow - plz ping), 2015/05/19 13:08:54]

Yes, probably. Could .send() return a Promise that you could reject?

[USE s.singapati at gmail.com, 2015/05/21 19:49:54]

Seems to be API change.
Mark: Do you have any comments?

+    // Ignore the current failed blob item and continue with next items.
+    ASSERT(m_messages.size() > 0);
+    m_messages.removeFirst();
+    handleMessageQueue();
+}
+
 } // namespace blink