[Android] Expose GetPendingEntry from NavigationController

content/browser/frame_host/navigation_controller_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/frame_host/navigation_controller_android.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_array.h"
 #include "base/android/jni_string.h"
 #include "content/browser/frame_host/navigation_entry_impl.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/ssl_host_state_delegate.h"
 #include "jni/NavigationControllerImpl_jni.h"
 #include "ui/gfx/android/java_bitmap.h"
 
 using base::android::AttachCurrentThread;
 using base::android::ConvertJavaStringToUTF16;
 using base::android::ConvertJavaStringToUTF8;
 using base::android::ConvertUTF16ToJavaString;
 using base::android::ConvertUTF8ToJavaString;
 namespace {
 
 // static
-static void AddNavigationEntryToHistory(JNIEnv* env,
-                                        jobject obj,
-                                        jobject history,
-                                        content::NavigationEntry* entry,
-                                        int index) {
+static base::android::ScopedJavaLocalRef<jobject> CreateJavaNavigationEntry(
+    JNIEnv* env,
+    jobject obj,
+    content::NavigationEntry* entry,
+    int index) {
+  if (!entry)

[Ted C, 2014/09/18 20:29:06]

I don't think we should support NULL entries.  That should be handled in the
pending entry code.

[David Trainor- moved to gerrit, 2014/09/18 21:06:24]

Done.

+    return base::android::ScopedJavaLocalRef<jobject>();
+
   // Get the details of the current entry
   ScopedJavaLocalRef<jstring> j_url(
       ConvertUTF8ToJavaString(env, entry->GetURL().spec()));
   ScopedJavaLocalRef<jstring> j_virtual_url(
       ConvertUTF8ToJavaString(env, entry->GetVirtualURL().spec()));
   ScopedJavaLocalRef<jstring> j_original_url(
       ConvertUTF8ToJavaString(env, entry->GetOriginalRequestURL().spec()));
   ScopedJavaLocalRef<jstring> j_title(
       ConvertUTF16ToJavaString(env, entry->GetTitle()));
   ScopedJavaLocalRef<jobject> j_bitmap;
   const content::FaviconStatus& status = entry->GetFavicon();
   if (status.valid && status.image.ToSkBitmap()->getSize() > 0)
     j_bitmap = gfx::ConvertToJavaBitmap(status.image.ToSkBitmap());
 
-  // Add the item to the list
-  content::Java_NavigationControllerImpl_addToNavigationHistory(
+  return content::Java_NavigationControllerImpl_createNavigationEntry(
       env,
       obj,

[Ted C, 2014/09/18 20:29:06]

wha?  Why is this method not static?  Why was it not static before?

[David Trainor- moved to gerrit, 2014/09/18 21:06:24]

hah good question!

-      history,
       index,
       j_url.obj(),
       j_virtual_url.obj(),
       j_original_url.obj(),
       j_title.obj(),
       j_bitmap.obj());
 }
 
+static void AddNavigationEntryToHistory(JNIEnv* env,
+                                        jobject obj,
+                                        jobject history,
+                                        content::NavigationEntry* entry,
+                                        int index) {
+  base::android::ScopedJavaLocalRef<jobject> j_entry =
+      CreateJavaNavigationEntry(env, obj, entry, index);
+  if (j_entry.is_null())

[Ted C, 2014/09/18 20:29:06]

would be able to remove this check here based on above comment.

[David Trainor- moved to gerrit, 2014/09/18 21:06:24]

Done.

+    return;
+
+  content::Java_NavigationControllerImpl_addToNavigationHistory(
+      env,
+      obj,
+      history,
+      j_entry.obj());
+}
+
 }  // namespace
 
 namespace content {
 
 // static
 bool NavigationControllerAndroid::Register(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 NavigationControllerAndroid::NavigationControllerAndroid(
@@ skipping 218 matching lines @@
   entry->SetIsOverridingUserAgent(enabled);
 
   // Send the override to the renderer.
   if (reload_on_state_change) {
     // Reloading the page will send the override down as part of the
     // navigation IPC message.
     navigation_controller_->ReloadOriginalRequestURL(false);
   }
 }
 
+base::android::ScopedJavaLocalRef<jobject>
+NavigationControllerAndroid::GetPendingEntry(JNIEnv* env, jobject obj) {
+  return CreateJavaNavigationEntry(
+      env,
+      obj,
+      navigation_controller_->GetPendingEntry(),
+      navigation_controller_->GetPendingEntryIndex());
+}
+
 }  // namespace content