Deprecate TimeToGetLinkToText histogram
Bug: 1214879
Change-Id: I0956622b469aed98cf79dce68fcfeba1e924665f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3072781
Reviewed-by: Guillaume Jenkins <gujen@google.com>
Reviewed-by: Kyle Milka <kmilka@chromium.org>
Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Commit-Queue: Gayane Petrosyan <gayane@chromium.org>
Cr-Commit-Position: refs/heads/master@{#909902}
diff --git a/chrome/browser/share/android/java/src/org/chromium/chrome/browser/share/link_to_text/LinkToTextCoordinator.java b/chrome/browser/share/android/java/src/org/chromium/chrome/browser/share/link_to_text/LinkToTextCoordinator.java
index cc35d33b..5f6feb4 100644
--- a/chrome/browser/share/android/java/src/org/chromium/chrome/browser/share/link_to_text/LinkToTextCoordinator.java
+++ b/chrome/browser/share/android/java/src/org/chromium/chrome/browser/share/link_to_text/LinkToTextCoordinator.java
@@ -11,7 +11,6 @@
import androidx.annotation.VisibleForTesting;
import org.chromium.base.Callback;
-import org.chromium.base.metrics.RecordHistogram;
import org.chromium.base.task.PostTask;
import org.chromium.blink.mojom.TextFragmentReceiver;
import org.chromium.chrome.R;
@@ -59,7 +58,6 @@
private final Tab mTab;
private final ChromeShareExtras mChromeShareExtras;
private final long mShareStartTime;
- private final long mRequestSelectorStartTime;
private String mShareUrl;
private TextFragmentReceiver mProducer;
@@ -102,7 +100,6 @@
mTab.addObserver(this);
mCancelRequest = false;
- mRequestSelectorStartTime = System.currentTimeMillis();
}
public ShareParams getShareParams(@LinkToggleState int linkToggleState) {
@@ -122,9 +119,6 @@
@VisibleForTesting
void onSelectorReady(String selector) {
- RecordHistogram.recordTimesHistogram(
- "Sharing.SharingHubAndroid.SharedHighlights.TimeToGetLinkToText",
- System.currentTimeMillis() - mRequestSelectorStartTime);
if (mCancelRequest) return;
if (ChromeFeatureList.isEnabled(ChromeFeatureList.PREEMPTIVE_LINK_TO_TEXT_GENERATION)) {
diff --git a/tools/metrics/histograms/metadata/sharing/histograms.xml b/tools/metrics/histograms/metadata/sharing/histograms.xml
index b159855..06ec046 100644
--- a/tools/metrics/histograms/metadata/sharing/histograms.xml
+++ b/tools/metrics/histograms/metadata/sharing/histograms.xml
@@ -567,6 +567,9 @@
<histogram
name="Sharing.SharingHubAndroid.SharedHighlights.TimeToGetLinkToText"
units="ms" expires_after="2021-07-11">
+ <obsolete>
+ Removed in M94.
+ </obsolete>
<owner>gayane@chromium.org</owner>
<owner>chrome-shared-highlighting@google.com</owner>
<summary>
