Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1303)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchUma.java

Issue 2863893002: [TTS] Remove two unused metrics. (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchUma.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchUma.java b/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchUma.java
index 2c6135f96b3fa9eac2f367ba09ee51cf9fd71fd8..0bae670e212bf72a46e62ceb5f45629fc8b34213 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchUma.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchUma.java
@@ -711,21 +711,6 @@ public class ContextualSearchUma {
}
/**
- * Logs whether search results were seen when the selection was part of a URL.
- * Unlike ContextualSearchResultsSeen, this histogram is logged for both decided and undecided
- * users.
- * @param wasPanelSeen Whether the panel was seen.
- * @param wasTap Whether the gesture that originally caused the panel to show was a Tap.
- */
- public static void logResultsSeenSelectionIsUrl(boolean wasPanelSeen, boolean wasTap) {
- int result = wasPanelSeen ? (wasTap ? RESULTS_SEEN_FROM_TAP : RESULTS_SEEN_FROM_LONG_PRESS)
- : (wasTap ? RESULTS_NOT_SEEN_FROM_TAP : RESULTS_NOT_SEEN_FROM_LONG_PRESS);
- RecordHistogram.recordEnumeratedHistogram(
- "Search.ContextualSearchResultsSeenSelectionWasUrl", result,
- RESULTS_BY_GESTURE_BOUNDARY);
- }
-
- /**
* Logs the whether the panel was seen and the type of the trigger and if Bar nearly overlapped.
* @param wasPanelSeen Whether the panel was seen.
* @param wasTap Whether the gesture was a Tap or not.
@@ -819,16 +804,6 @@ public class ContextualSearchUma {
}
/**
- * Logs whether results were seen when the selected text consisted of all capital letters.
- * @param wasSearchContentViewSeen If the panel was opened.
- */
- public static void logAllCapsResultsSeen(boolean wasSearchContentViewSeen) {
- RecordHistogram.recordEnumeratedHistogram("Search.ContextualSearchAllCapsResultsSeen",
- wasSearchContentViewSeen ? RESULTS_SEEN : RESULTS_NOT_SEEN,
- RESULTS_SEEN_BOUNDARY);
- }
-
- /**
* Logs whether results were seen when the selected text started with a capital letter but was
* not all capital letters.
* @param wasSearchContentViewSeen If the panel was opened.

Powered by Google App Engine
This is Rietveld 408576698