[Contextual Search] Add metrics for Resolve and View timing

chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanel.java

Index: chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanel.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanel.java b/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanel.java
index 50796ab6cae0f71c25fc96289dae8ded541f836a..fa02a3a0ff3f191df03eed37dc17614f98e54e25 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanel.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanel.java
@@ -383,6 +383,14 @@ public class ContextualSearchPanel extends OverlayPanel {
     }
 
     /**
+     * Notifies the panel that the Search Results are now viewable in the panel.
+     * @param didResolve Whether the search required Search Term Resolution.
+     */
+    public void onSearchResultsViewable(boolean didResolve) {

[pedro (no code reviews), 2016/02/26 22:30:09]

Does this mean the results were viewed, or that they simple could have been
viewed (because they were preteched)?

Assuming the latter, wouldn't it be easier to understand if this was called
onSearchResultsPrefetched()?

[Donn Denman, 2016/02/27 00:18:39]

Humm, onSearchResultsPrefetched seems confusing too because we when we just did
the prefetch loading.  How about onPrefetchedSearchNavigated?  This ties it in
with the navigation event and that it should only be called for prefetched
requests.

[pedro (no code reviews), 2016/02/27 00:39:04]

Good point. onPrefetchedSearchNavigated() seems good to me. Clearly more
intelligible than onSearchResultsViewable()

+        mPanelMetrics.onSearchResultsViewable(didResolve);
+    }
+
+    /**
      * Maximizes the Contextual Search Panel, then promotes it to a regular Tab.
      * @param reason The {@code StateChangeReason} behind the maximization and promotion to tab.
      */
@@ -470,7 +478,8 @@ public class ContextualSearchPanel extends OverlayPanel {
      * Handles showing the resolved search term in the SearchBar.
      * @param searchTerm The string that represents the search term.
      */
-    public void onSearchTermResolutionResponse(String searchTerm) {
+    public void onSearchTermResolved(String searchTerm) {
+        mPanelMetrics.onSearchTermResolved();
         getSearchBarControl().setSearchTerm(searchTerm);
         animateSearchTermResolution();
     }