Allow showing pending URL for new tab navigations, but only if safe.

content/browser/android/content_view_core_impl.cc

Index: content/browser/android/content_view_core_impl.cc
diff --git a/content/browser/android/content_view_core_impl.cc b/content/browser/android/content_view_core_impl.cc
index 96916f497ef71bd99dab85db03744f9153892a0d..a65a490a5a201c91734a8fa963009d7b0fc64767 100644
--- a/content/browser/android/content_view_core_impl.cc
+++ b/content/browser/android/content_view_core_impl.cc
@@ -748,7 +748,10 @@ void ContentViewCoreImpl::SetAllUserAgentOverridesInHistory(
 
 ScopedJavaLocalRef<jstring> ContentViewCoreImpl::GetURL(
     JNIEnv* env, jobject) const {
-  return ConvertUTF8ToJavaString(env, GetWebContents()->GetURL().spec());
+  content::NavigationEntry* entry =

[joth, 2013/03/22 17:12:52]

I think this should comment why it's not using GetURL() (even if just saying the
change from crrev.com/12541018 was incompatibile with the existing users of the
Java API)

[Charlie Reis, 2013/03/28 19:23:19]

Done.

+      web_contents_->GetController().GetActiveEntry();
+  GURL url = entry ? entry->GetVirtualURL() : GURL::EmptyGURL();
+  return ConvertUTF8ToJavaString(env, url.spec());
 }
 
 ScopedJavaLocalRef<jstring> ContentViewCoreImpl::GetTitle(