[PresentationAPI] on-session-message handler for ArrayBuffer and Blob messages.

content/renderer/presentation/presentation_dispatcher.cc

 // Copyright 2015 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 "content/renderer/presentation/presentation_dispatcher.h"
 
 #include "base/logging.h"
 #include "content/common/presentation/presentation_service.mojom.h"
 #include "content/public/common/presentation_constants.h"
 #include "content/public/common/service_registry.h"
@@ skipping 351 matching lines @@
       GetWebPresentationSessionStateFromMojo(session_state));
 }
 
 void PresentationDispatcher::OnSessionMessagesReceived(
     mojo::Array<presentation::SessionMessagePtr> messages) {
   // When messages is null, there is an error at presentation service side.
   if (!controller_ || messages.is_null())
     return;
 
   for (size_t i = 0; i < messages.size(); ++i) {
-    if (messages[i]->type ==
-        presentation::PresentationMessageType::PRESENTATION_MESSAGE_TYPE_TEXT) {
-      controller_->didReceiveSessionTextMessage(
-          new PresentationSessionClient(messages[i]->presentation_url,
-                                        messages[i]->presentation_id),
-          blink::WebString::fromUTF8(messages[i]->message));
-    } else {
-      // TODO(haibinlu): handle binary message
+    switch (messages[i]->type) {
+      case presentation::PresentationMessageType::
+          PRESENTATION_MESSAGE_TYPE_TEXT: {
+        controller_->didReceiveSessionTextMessage(
+            new PresentationSessionClient(messages[i]->presentation_url,

[Habib Virji, 2015/06/23 13:04:23]

This can be handled prior to the switch statement as it is getting replicated in
each case. 

[USE s.singapati at gmail.com, 2015/06/23 15:06:35]

Done.

+                                          messages[i]->presentation_id),
+            blink::WebString::fromUTF8(messages[i]->message));
+        break;
+      }
+      case presentation::PresentationMessageType::
+          PRESENTATION_MESSAGE_TYPE_ARRAY_BUFFER: {
+        controller_->didReceiveSessionArrayBufferMessage(
+            new PresentationSessionClient(messages[i]->presentation_url,
+                                          messages[i]->presentation_id),
+            &(messages[i]->data.front()), messages[i]->data.size());
+        break;
+      }
+      case presentation::PresentationMessageType::
+          PRESENTATION_MESSAGE_TYPE_BLOB: {
+        controller_->didReceiveSessionBlobMessage(
+            new PresentationSessionClient(messages[i]->presentation_url,
+                                          messages[i]->presentation_id),
+            &(messages[i]->data.front()), messages[i]->data.size());
+        break;
+      }
+      default: {
+        NOTREACHED();
+        break;
+      }
     }
   }
 
   presentation_service_->ListenForSessionMessages(
       base::Bind(&PresentationDispatcher::OnSessionMessagesReceived,
                  base::Unretained(this)));
 }
 
 void PresentationDispatcher::ConnectToPresentationServiceIfNeeded() {
   if (presentation_service_.get())
@@ skipping 14 matching lines @@
   presentation_service_->ListenForSessionStateChange(base::Bind(
       &PresentationDispatcher::OnSessionStateChange,
       base::Unretained(this)));
   presentation_service_->ListenForSessionMessages(
       base::Bind(&PresentationDispatcher::OnSessionMessagesReceived,
       base::Unretained(this)));
   */
 }
 
 }  // namespace content