HTML MessagePort as mojo::MessagePipeHandle

content/browser/web_contents/web_contents_android.cc

 // Copyright 2013 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/browser/web_contents/web_contents_android.h"
 
 #include <stdint.h>
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_array.h"
 #include "base/android/jni_string.h"
 #include "base/command_line.h"
 #include "base/containers/hash_tables.h"
 #include "base/json/json_writer.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
 #include "content/browser/accessibility/browser_accessibility_android.h"
 #include "content/browser/accessibility/browser_accessibility_manager_android.h"
+#include "content/browser/android/app_web_message_port.h"
 #include "content/browser/android/content_view_core_impl.h"
 #include "content/browser/android/interstitial_page_delegate_android.h"
 #include "content/browser/frame_host/interstitial_page_impl.h"
 #include "content/browser/media/android/browser_media_player_manager.h"
 #include "content/browser/media/android/media_web_contents_observer_android.h"
 #include "content/browser/renderer_host/render_view_host_impl.h"
 #include "content/browser/web_contents/web_contents_impl.h"
 #include "content/browser/web_contents/web_contents_view_android.h"
 #include "content/common/devtools_messages.h"
 #include "content/common/frame_messages.h"
@@ skipping 527 matching lines @@
       static_cast<ConsoleMessageLevel>(level),
       ConvertJavaStringToUTF8(env, message));
 }
 
 void WebContentsAndroid::PostMessageToFrame(
     JNIEnv* env,
     const JavaParamRef<jobject>& obj,
     const JavaParamRef<jstring>& jframe_name,
     const JavaParamRef<jstring>& jmessage,
     const JavaParamRef<jstring>& jtarget_origin,
-    const JavaParamRef<jintArray>& jsent_ports) {
+    const JavaParamRef<jobjectArray>& jports) {
   base::string16 source_origin;
   base::string16 target_origin(ConvertJavaStringToUTF16(env, jtarget_origin));
   base::string16 message(ConvertJavaStringToUTF16(env, jmessage));
-  std::vector<int> ports;
+  std::vector<MessagePort> ports(
+      AppWebMessagePort::UnwrapJavaArray(env, jports));
 
-  if (!jsent_ports.is_null())
-    base::android::JavaIntArrayToIntVector(env, jsent_ports, &ports);
   content::MessagePortProvider::PostMessageToFrame(

[sgurun-gerrit only, 2017/01/27 01:16:37]

can we move the code in messageportprovider::postmessagetoframe to here? then we
can get rid of it entirely. after the refactorings for CCT and this, it is not
meaningful anymore IMO.

       web_contents_, source_origin, target_origin, message, ports);
 }
 
 void WebContentsAndroid::CreateMessageChannel(
     JNIEnv* env,
     const JavaParamRef<jobject>& obj,
     const JavaParamRef<jobjectArray>& ports) {
   content::MessagePortProvider::GetAppWebMessagePortService()
       ->CreateMessageChannel(env, ports, web_contents_);

[sgurun-gerrit only, 2017/01/27 01:16:37]

this will not need to get appwebmessageportservice once the implementation moves
here.

 }
 
 jboolean WebContentsAndroid::HasAccessedInitialDocument(
     JNIEnv* env,
     const JavaParamRef<jobject>& jobj) {
   return static_cast<WebContentsImpl*>(web_contents_)->
       HasAccessedInitialDocument();
 }
 
 jint WebContentsAndroid::GetThemeColor(JNIEnv* env,
@@ skipping 138 matching lines @@
       env, obj, callback, id, http_status_code, jurl, jbitmaps, jsizes);
 }
 
 void WebContentsAndroid::SetMediaSession(
     const ScopedJavaLocalRef<jobject>& j_media_session) {
   JNIEnv* env = base::android::AttachCurrentThread();
   Java_WebContentsImpl_setMediaSession(env, obj_, j_media_session);
 }
 
 }  // namespace content