[SH-Clank] Remove existing fragment from url

This CL also adds fragment identifier to the url that was ommited
before.

Bug: 1102382
Change-Id: I175c409cef5a90a3e8e2958c66a90e779dbad0fe
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363347Reviewed-by: Gayane Petrosyan <gayane@chromium.org>
Reviewed-by: Tommy Martino <tmartino@chromium.org>
Commit-Queue: Gayane Petrosyan <gayane@chromium.org>
Cr-Commit-Position: refs/heads/master@{#799313}

chrome/browser/share/android/java/src/org/chromium/chrome/browser/share/link_to_text/LinkToTextCoordinator.java

@@ -5,6 +5,7 @@
 package org.chromium.chrome.browser.share.link_to_text;
 
 import android.content.Context;
+import android.net.Uri;
 
 import org.chromium.chrome.R;
 import org.chromium.chrome.browser.share.ChromeShareExtras;
@@ -17,7 +18,7 @@ import org.chromium.ui.base.WindowAndroid;
  */
 public class LinkToTextCoordinator {
     private static final String SHARE_TEXT_TEMPLATE = "\"%s\"\n%s";
-    private static final String URL_TEMPLATE = "%s:~:text=%s";
+    private static final String TEXT_FRAGMENT_PREFIX = ":~:text=";
     private final Context mContext;
     private final WindowAndroid mWindow;
     private final ChromeOptionShareCallback mChromeOptionShareCallback;
@@ -58,11 +59,12 @@ public class LinkToTextCoordinator {
     }
 
     public String getTextToShare(String selector) {
-        // TODO(1102382): visbileUrl might need a cleanup. For example, if already has text fragment
-        // selector.
         String url = mVisibleUrl;
-        if (!selector.isEmpty()) url = String.format(URL_TEMPLATE, mVisibleUrl, selector);
-
+        if (!selector.isEmpty()) {
+            // Set the fragment which will also remove existing fragment, including text fragments.
+            Uri uri = Uri.parse(url);
+            url = uri.buildUpon().encodedFragment(TEXT_FRAGMENT_PREFIX + selector).toString();
+        }
         return String.format(SHARE_TEXT_TEMPLATE, mSelectedText, url);
     }
 }
\ No newline at end of file

chrome/browser/share/android/javatests/src/org/chromium/chrome/browser/share/link_to_text/LinkToTextCoordinatorTest.java

@@ -53,12 +53,22 @@ public class LinkToTextCoordinatorTest {
     @SmallTest
     public void getTextToShareTest() {
         String selector = "selector";
-        String expectedTextToShare = "\"selection\"\nwww.example.com:~:text=selector";
+        String expectedTextToShare = "\"selection\"\nwww.example.com#:~:text=selector";
         LinkToTextCoordinator coordinator = new LinkToTextCoordinator(
                 mAcivity, mWindow, mShareCallback, VISIBLE_URL, SELECTED_TEXT);
         Assert.assertEquals(expectedTextToShare, coordinator.getTextToShare(selector));
     }
 
+    @Test
+    @SmallTest
+    public void getTextToShareTest_URLWithFragment() {
+        String selector = "selector";
+        String expectedTextToShare = "\"selection\"\nwww.example.com#:~:text=selector";
+        LinkToTextCoordinator coordinator = new LinkToTextCoordinator(
+                mAcivity, mWindow, mShareCallback, VISIBLE_URL + "#elementid", SELECTED_TEXT);
+        Assert.assertEquals(expectedTextToShare, coordinator.getTextToShare(selector));
+    }
+
     @Test
     @SmallTest
     public void getTextToShareTest_EmptySelector() {