Navigation transitions (web to native app): Pass data after starting provisional load (Chrome side)

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 "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/command_line.h"
 #include "base/json/json_writer.h"
@@ skipping 306 matching lines @@
 
   std::vector<GURL>::const_iterator iter = entering_stylesheets.begin();
   for (; iter != entering_stylesheets.end(); ++iter) {
     ScopedJavaLocalRef<jstring> jstring_url(
         ConvertUTF8ToJavaString(env, iter->spec()));
     Java_WebContentsImpl_addEnteringStylesheetToTransition(
         env, obj_.obj(), jstring_url.obj());
   }
 }
 
-void WebContentsAndroid::DidStartNavigationTransitionForFrame(int64 frame_id) {
+void WebContentsAndroid::DidStartNavigationTransitionForFrame(
+    int64 frame_id,
+    const TransitionLayerData& transition_data) {
   JNIEnv* env = AttachCurrentThread();
+
+  ScopedJavaLocalRef<jstring> jstring_css_selector(
+      ConvertUTF8ToJavaString(env, transition_data.css_selector));
+
   Java_WebContentsImpl_didStartNavigationTransitionForFrame(
-      env, obj_.obj(), frame_id);
+      env, obj_.obj(), frame_id, jstring_css_selector.obj());
+
+  std::vector<std::string>::const_iterator it = transition_data.names.begin();
+  std::vector<gfx::Rect>::const_iterator jt = transition_data.rects.begin();
+  for (; it != transition_data.names.end() && jt != transition_data.rects.end();

[oystein (OOO til 10th of July), 2014/10/27 18:45:12]

Could the names+rects be stored together in a vector of structs, rather than two
vectors? It should be cheaper and more readable.

[Zhen Wang, 2014/10/27 23:47:19]

Done.

But I didn't find whether customized struct is available over jni from C++ side
to Java side (jobject seems to be used from Java side to C++ side only).

+      ++it, ++jt) {
+    ScopedJavaLocalRef<jstring> jstring_name(ConvertUTF8ToJavaString(env, *it));
+    Java_WebContentsImpl_addNavigationTransitionElements(
+        env, obj_.obj(), jstring_name.obj(),
+        jt->x(), jt->y(), jt->width(), jt->height());
+  }
 }
 
 void WebContentsAndroid::EvaluateJavaScript(JNIEnv* env,
                                             jobject obj,
                                             jstring script,
                                             jobject callback) {
   RenderViewHost* rvh = web_contents_->GetRenderViewHost();
   DCHECK(rvh);
 
   if (!rvh->IsRenderViewLive()) {
@@ skipping 16 matching lines @@
   ScopedJavaGlobalRef<jobject> j_callback;
   j_callback.Reset(env, callback);
   content::RenderFrameHost::JavaScriptResultCallback js_callback =
       base::Bind(&JavaScriptResultCallback, j_callback);
 
   web_contents_->GetMainFrame()->ExecuteJavaScript(
       ConvertJavaStringToUTF16(env, script), js_callback);
 }
 
 }  // namespace content