Add blink-side binding code and tests for ServiceWorker -> Document postMessage (3/3)

Source/web/ServiceWorkerGlobalScopeClientImpl.cpp

 /*
  * Copyright (C) 2014 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 13 matching lines @@
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 #include "web/ServiceWorkerGlobalScopeClientImpl.h"
 
+#include "bindings/v8/SerializedScriptValue.h"
 #include "modules/serviceworkers/Response.h"
 #include "platform/NotImplemented.h"
 #include "public/platform/WebServiceWorkerResponse.h"
 #include "public/platform/WebURL.h"
 #include "public/web/WebServiceWorkerContextClient.h"
 #include "wtf/PassOwnPtr.h"
 
 using namespace WebCore;
 
 namespace blink {
@@ skipping 37 matching lines @@
     WebServiceWorkerResponse webResponse;
     response->populateWebServiceWorkerResponse(webResponse);
     m_client.didHandleFetchEvent(fetchEventID, webResponse);
 }
 
 void ServiceWorkerGlobalScopeClientImpl::didHandleSyncEvent(int syncEventID)
 {
     m_client.didHandleSyncEvent(syncEventID);
 }
 
+void ServiceWorkerGlobalScopeClientImpl::postMessageToClient(int clientID, PassRefPtr<SerializedScriptValue> message, PassOwnPtr<MessagePortChannelArray> channels)
+{
+    blink::WebString messageString = message->toWireString();

[jsbell, 2014/05/07 23:46:33]

Just curious - is there a reason to defer the toWireString and
toWebMessagePortChannelArray from Client.cpp to here?

It's done directly in ServiceWorker::postMessage() so I'm wondering if the
difference is intentional and we should change it there too.

[kinuko, 2014/05/08 02:28:23]

No strong reason, did so just to avoid some header churn. I moved them to
Client.cpp as it'd be more readable & to make them look more alike.

+    OwnPtr<blink::WebMessagePortChannelArray> webChannels = MessagePort::toWebMessagePortChannelArray(channels);
+    m_client.postMessageToClient(clientID, messageString, webChannels.leakPtr());
+}
+
 ServiceWorkerGlobalScopeClientImpl::ServiceWorkerGlobalScopeClientImpl(WebServiceWorkerContextClient& client)
     : m_client(client)
 {
 }
 
 } // namespace blink