[Contextual Search] Add metrics for Resolve and View timing

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

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.chrome.browser.compositor.bottombar.contextualsearch;
 
 import android.app.Activity;
 import android.content.Context;
 import android.os.Handler;
 
@@ skipping 365 matching lines @@
     /**
      * Called when the SERP finishes loading, this records the duration of loading the SERP from
      * the time the panel was opened until the present.
      * @param wasPrefetch Whether the request was prefetch-enabled.
      */
     public void onSearchResultsLoaded(boolean wasPrefetch) {
         mPanelMetrics.onSearchResultsLoaded(wasPrefetch);
     }
 
     /**
+     * Notifies the panel that navigation to prefetched Search Results are now viewable.

[Theresa, 2016/02/29 19:42:44]

I was confused about what "navigation" meant until reading the docstrings in
ContextualSearchUma.

Initially I was thinking this should read "Notifies the panel that prefetched
Search Results are now viewable", but it looks like this method isn't notifying
the panel, it's notifying panel metrics. Maybe something like "Called after the
panel has navigated to prefetched Search Results" or something like that.

In the same vein, I think the method name is confusing.
onNavigateToPrefetchedSearch() makes more sense to me or
onPanelNavigatedToPrefetchedSearch()

[Donn Denman, 2016/03/01 21:18:07]

OK, I adopted your suggestions for "onPanelNavigatedToPrefetchedSearch" as the
method name and updated the description as you suggested.

+     * If the user has the panel open then they will see the prefetched result starting to load.
+     * @param didResolve Whether the search required Search Term Resolution.
+     */
+    public void onPrefetchedSearchNavigated(boolean didResolve) {
+        mPanelMetrics.onPrefetchedSearchNavigated(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.
      */
     public void maximizePanelThenPromoteToTab(StateChangeReason reason) {
         mShouldPromoteToTabAfterMaximizing = true;
         maximizePanel(reason);
     }
 
     /**
      * Maximizes the Contextual Search Panel, then promotes it to a regular Tab.
@@ skipping 67 matching lines @@
     public void displaySearchContext(String selection, String end) {
         cancelSearchTermResolutionAnimation();
         getSearchBarControl().setSearchContext(selection, end);
         resetSearchBarContextOpacity();
     }
 
     /**
      * 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();
     }
 
     /**
      * Sets that the contextual search involved the promo.
      */
     public void setDidSearchInvolvePromo() {
         mPanelMetrics.setDidSearchInvolvePromo();
     }
@@ skipping 250 matching lines @@
 
     /**
      * Destroy the current content in the panel.
      * NOTE(mdjones): This should not be exposed. The only use is in ContextualSearchManager for a
      * bug related to loading new panel content.
      */
     public void destroyContent() {
         super.destroyOverlayPanelContent();
     }
 }