Revert "Migrate webview Webcontents#getVisibleUrl to GURL"

This reverts commit a25a688e.

Reason for revert: https://bugs.chromium.org/p/chromium/issues/detail?id=1049536

Original change's description:
> Migrate webview Webcontents#getVisibleUrl to GURL
> 
> Bug: 783819
> Change-Id: Ib1970a41bb45a31907ea91358fd27bee6c6b20a7
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2039301
> Reviewed-by: Bo <boliu@chromium.org>
> Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#738694}

TBR=mthiesse@chromium.org,boliu@chromium.org

Change-Id: Ib9a94a412c9ac7625b37dbc6075c61f2d7bcc9fb
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 783819
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2042087Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
Cr-Commit-Position: refs/heads/master@{#738967}

android_webview/BUILD.gn

@@ -471,7 +471,6 @@ android_library("browser_java") {
     "//third_party/android_deps:androidx_annotation_annotation_java",
     "//third_party/blink/public:blink_headers_java",
     "//ui/android:ui_java",
-    "//url:gurl_java",
   ]
   annotation_processor_deps = [ "//base/android/jni_generator:jni_processor" ]
 

android_webview/glue/BUILD.gn

@@ -24,7 +24,6 @@ android_library("glue") {
     "//content/public/android:content_java",
     "//net/android:net_java",
     "//ui/android:ui_java",
-    "//url:gurl_java",
   ]
   annotation_processor_deps = [ "//base/android/jni_generator:jni_processor" ]
 

android_webview/glue/java/src/com/android/webview/chromium/WebViewChromium.java

@@ -1096,7 +1096,7 @@ class WebViewChromium implements WebViewProvider, WebViewProvider.ScrollDelegate
             return ret;
         }
         sWebViewApiCallSample.record(ApiCall.GET_URL);
-        return mAwContents.getUrl().getSpec();
+        return mAwContents.getUrl();
     }
 
     @Override

android_webview/java/src/org/chromium/android_webview/AwContents.java

@@ -110,7 +110,6 @@ import org.chromium.ui.base.PageTransition;
 import org.chromium.ui.base.ViewAndroidDelegate;
 import org.chromium.ui.base.WindowAndroid;
 import org.chromium.ui.display.DisplayAndroid.DisplayAndroidObserver;
-import org.chromium.url.GURL;
 
 import java.io.File;
 import java.lang.annotation.Annotation;
@@ -1969,10 +1968,10 @@ public class AwContents implements SmartClipProvider {
      *
      * @return The URL of the current page or null if it's empty.
      */
-    public GURL getUrl() {
+    public String getUrl() {
         if (isDestroyed(WARN)) return null;
-        GURL url = mWebContents.getVisibleUrl();
-        if (url == null || url.getSpec().trim().isEmpty()) return null;
+        String url = mWebContents.getVisibleUrlString();
+        if (url == null || url.trim().isEmpty()) return null;
         return url;
     }
 

android_webview/javatests/src/org/chromium/android_webview/test/AwContentsClientFullScreenTest.java

@@ -28,7 +28,6 @@ import org.chromium.content_public.browser.test.util.CriteriaHelper;
 import org.chromium.content_public.browser.test.util.DOMUtils;
 import org.chromium.content_public.browser.test.util.TestThreadUtils;
 import org.chromium.content_public.browser.test.util.TouchCommon;
-import org.chromium.url.GURL;
 
 import java.util.concurrent.TimeoutException;
 
@@ -327,9 +326,8 @@ public class AwContentsClientFullScreenTest {
     }
 
     private boolean shouldPlayOnFullScreenView() throws Exception {
-        GURL testUrl = mTestContainerView.getAwContents().getUrl();
-        return new GURL(VIDEO_TEST_URL).equals(testUrl)
-                && DOMUtils.isFullscreen(getWebContentsOnUiThread());
+        String testUrl = mTestContainerView.getAwContents().getUrl();
+        return VIDEO_TEST_URL.equals(testUrl) && DOMUtils.isFullscreen(getWebContentsOnUiThread());
     }
 
     private void playVideo() throws Exception {

android_webview/javatests/src/org/chromium/android_webview/test/AwContentsClientShouldOverrideUrlLoadingTest.java

@@ -36,7 +36,6 @@ import org.chromium.content_public.browser.test.util.TestCallbackHelperContainer
 import org.chromium.content_public.browser.test.util.TestCallbackHelperContainer.OnReceivedErrorHelper;
 import org.chromium.content_public.common.ContentUrlConstants;
 import org.chromium.net.test.util.TestWebServer;
-import org.chromium.url.GURL;
 
 import java.util.ArrayList;
 import java.util.List;
@@ -857,7 +856,7 @@ public class AwContentsClientShouldOverrideUrlLoadingTest {
         clickOnLinkUsingJs();
         onPageFinishedHelper.waitForCallback(pageFinishedCount);
 
-        Assert.assertEquals(new GURL(linkUrl), mAwContents.getUrl());
+        Assert.assertEquals(linkUrl, mAwContents.getUrl());
         Assert.assertTrue("Should have a navigation history", mAwContents.canGoBack());
         NavigationHistory navHistory = mAwContents.getNavigationHistory();
         Assert.assertEquals(2, navHistory.getEntryCount());
@@ -869,7 +868,7 @@ public class AwContentsClientShouldOverrideUrlLoadingTest {
         onPageFinishedHelper.waitForCallback(pageFinishedCount);
 
         Assert.assertFalse("Should not be able to navigate backward", mAwContents.canGoBack());
-        Assert.assertEquals(new GURL(firstUrl), mAwContents.getUrl());
+        Assert.assertEquals(firstUrl, mAwContents.getUrl());
         navHistory = mAwContents.getNavigationHistory();
         Assert.assertEquals(2, navHistory.getEntryCount());
         Assert.assertEquals(0, navHistory.getCurrentEntryIndex());
@@ -1002,8 +1001,7 @@ public class AwContentsClientShouldOverrideUrlLoadingTest {
             mContentsClient.getOnPageFinishedHelper().waitForCallback(
                     currentCallCount, 1, WAIT_TIMEOUT_MS, TimeUnit.MILLISECONDS);
 
-            Assert.assertEquals(
-                    new GURL(ContentUrlConstants.ABOUT_BLANK_DISPLAY_URL), mAwContents.getUrl());
+            Assert.assertEquals(ContentUrlConstants.ABOUT_BLANK_DISPLAY_URL, mAwContents.getUrl());
         } finally {
             mActivityTestRule.getActivity().setIgnoreStartActivity(false);
         }

android_webview/javatests/src/org/chromium/android_webview/test/SafeBrowsingTest.java

@@ -59,7 +59,6 @@ import org.chromium.content_public.browser.test.util.CriteriaHelper;
 import org.chromium.content_public.browser.test.util.TestThreadUtils;
 import org.chromium.content_public.common.ContentUrlConstants;
 import org.chromium.net.test.EmbeddedTestServer;
-import org.chromium.url.GURL;
 
 import java.util.ArrayList;
 import java.util.Arrays;
@@ -993,10 +992,9 @@ public class SafeBrowsingTest {
         // We'll successfully load IFRAME_HTML_PATH, and will soon call onSafeBrowsingHit
         mContentsClient.getOnPageFinishedHelper().waitForCallback(pageFinishedCount);
 
-        final GURL aboutBlank = new GURL(ContentUrlConstants.ABOUT_BLANK_DISPLAY_URL);
-
         // Wait for the onSafeBrowsingHit to call BACK_TO_SAFETY and navigate back
-        mActivityTestRule.pollUiThread(() -> aboutBlank.equals(mAwContents.getUrl()));
+        mActivityTestRule.pollUiThread(
+                () -> ContentUrlConstants.ABOUT_BLANK_DISPLAY_URL.equals(mAwContents.getUrl()));
 
         // Check onSafeBrowsingHit arguments
         Assert.assertFalse(mContentsClient.getLastRequest().isMainFrame);
@@ -1273,7 +1271,7 @@ public class SafeBrowsingTest {
             // Make sure the URL was seen for a main frame navigation.
             Assert.assertTrue(requestsForUrl.isMainFrame);
         } else {
-            Assert.assertEquals(new GURL(linkUrl), mAwContents.getUrl());
+            Assert.assertEquals(linkUrl, mAwContents.getUrl());
         }
     }
 

android_webview/test/BUILD.gn

@@ -71,7 +71,6 @@ android_apk("webview_instrumentation_apk") {
     "//third_party/android_support_test_runner:runner_java",
     "//third_party/junit",
     "//ui/android:ui_java",
-    "//url:gurl_java",
   ]
 
   apk_name = "WebViewInstrumentation"
@@ -190,7 +189,6 @@ instrumentation_test_apk("webview_instrumentation_test_apk") {
     "//third_party/junit",
     "//third_party/metrics_proto:metrics_proto_java",
     "//ui/android:ui_java",
-    "//url:gurl_java",
   ]
   sources = [
     "../javatests/src/org/chromium/android_webview/test/AcceptLanguageTest.java",

android_webview/test/shell/src/org/chromium/android_webview/shell/AwShellActivity.java

@@ -270,7 +270,7 @@ public class AwShellActivity extends Activity {
             mNextButton.setVisibility(hasFocus ? View.GONE : View.VISIBLE);
             mPrevButton.setVisibility(hasFocus ? View.GONE : View.VISIBLE);
             if (!hasFocus) {
-                mUrlTextView.setText(mWebContents.getVisibleUrl().getSpec());
+                mUrlTextView.setText(mWebContents.getVisibleUrlString());
             }
         });
     }