Make sure to show the correct URL from CVC.

content/browser/android/content_view_core_impl.cc

 // Copyright 2012 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/android/content_view_core_impl.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_array.h"
 #include "base/android/jni_string.h"
 #include "base/android/scoped_java_ref.h"
@@ skipping 805 matching lines @@
 }
 
 jint ContentViewCoreImpl::GetCurrentRenderProcessId(JNIEnv* env, jobject obj) {
   return GetRenderProcessIdFromRenderViewHost(
       web_contents_->GetRenderViewHost());
 }
 
 ScopedJavaLocalRef<jstring> ContentViewCoreImpl::GetURL(
     JNIEnv* env, jobject) const {
   // The current users of the Java API expect to use the active entry
   // rather than the visible entry, which is exposed by WebContents::GetURL.

[Charlie Reis, 2013/10/04 20:31:48]

Let's remove the comment as well.  I just remembered that Nasko was removing
this as well, but he had to revert his larger CL for unrelated reasons.

Context:
https://codereview.chromium.org/25654005/diff/1/content/browser/android/conte...

[David Trainor- moved to gerrit, 2013/10/04 20:50:07]

Gah sorry was going too fast.  Missed this.  Thanks! :)
On 2013/10/04 20:31:48, creis wrote:

   content::NavigationEntry* entry =
-      web_contents_->GetController().GetActiveEntry();
+      web_contents_->GetController().GetVisibleEntry();
   GURL url = entry ? entry->GetVirtualURL() : GURL::EmptyGURL();
   return ConvertUTF8ToJavaString(env, url.spec());
 }
 
 ScopedJavaLocalRef<jstring> ContentViewCoreImpl::GetTitle(
     JNIEnv* env, jobject obj) const {
   return ConvertUTF16ToJavaString(env, GetWebContents()->GetTitle());
 }
 
 jboolean ContentViewCoreImpl::IsIncognito(JNIEnv* env, jobject obj) {
@@ skipping 765 matching lines @@
       reinterpret_cast<ui::ViewAndroid*>(view_android),
       reinterpret_cast<ui::WindowAndroid*>(window_android));
   return reinterpret_cast<jint>(view);
 }
 
 bool RegisterContentViewCore(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace content