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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanelMetrics.java

Issue 1736203002: [Contextual Search] Add metrics for Resolve and View timing (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 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/compositor/bottombar/contextualsearch/ContextualSearchPanelMetrics.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanelMetrics.java b/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanelMetrics.java
index 7eb80bc2f6c00033246c14935597f2bf0fb9650e..3c7a748ac356e1e1614d6d72e524cc327ffedf54 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanelMetrics.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanelMetrics.java
@@ -187,6 +187,16 @@ public class ContextualSearchPanelMetrics {
mIsSearchPanelFullyPreloaded = true;
}
+ public void onSearchTermResolved() {
pedro (no code reviews) 2016/02/26 22:30:09 Nit: JavaDoc
Donn Denman 2016/02/27 00:18:39 Done.
+ long durationMs = (System.nanoTime() - mSearchViewStartTimeNs) / 1000000;
+ ContextualSearchUma.logSearchTermResolution(durationMs);
+ }
+
+ public void onSearchResultsViewable(boolean didResolve) {
pedro (no code reviews) 2016/02/26 22:30:09 Nit: JavaDoc
Donn Denman 2016/02/27 00:18:39 Done.
+ long durationMs = (System.nanoTime() - mSearchViewStartTimeNs) / 1000000;
+ ContextualSearchUma.logSearchResultsViewable(durationMs, didResolve);
+ }
+
/**
* Records timing information when the search panel has been viewed for the first time.
*/

Powered by Google App Engine
This is Rietveld 408576698