Commit af8eb3f1 authored by Donn Denman's avatar Donn Denman Committed by Commit Bot

[TTS] Add CTR histograms that apply to all users.

This CL adds two new ResultsSeen histograms that apply to all users.

One records whether results were seen for the tap gesture, and the other
for any gesture that triggers the Contextual Search UI.

All the previous histograms slice the data one way or another, usually
based on whether the user opted-in to our privacy policy.

These new histograms will make it easy to see the overall impact of
experiments that affect CTR.

BUG=821893

Change-Id: I0a20ca56668d2086a213fd946b0704e4db601714
Reviewed-on: https://chromium-review.googlesource.com/963291Reviewed-by: default avatarTheresa <twellington@chromium.org>
Reviewed-by: default avatarRobert Kaplow <rkaplow@chromium.org>
Commit-Queue: Donn Denman <donnd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543715}
parent 8299aa51
......@@ -144,7 +144,9 @@ public class ContextualSearchPanelMetrics {
mWasSearchContentViewSeen, wasAnySuppressionHeuristicSatisfied);
ContextualSearchUma.logSelectionLengthResultsSeen(
mWasSearchContentViewSeen, mSelectionLength);
ContextualSearchUma.logTapResultsSeen(mWasSearchContentViewSeen);
}
ContextualSearchUma.logAllResultsSeen(mWasSearchContentViewSeen);
// Notifications to Feature Engagement.
ContextualSearchIPH.doSearchFinishedNotifications(profile, mWasSearchContentViewSeen,
......
......@@ -694,6 +694,24 @@ public class ContextualSearchUma {
logHistogramByGesture(wasPanelSeen, wasTap, "Search.ContextualSearchResultsSeenByGesture");
}
/**
* Logs whether search results were seen for a Tap gesture. Recorded for all users.
* @param wasPanelSeen Whether the panel was seen.
*/
public static void logTapResultsSeen(boolean wasPanelSeen) {
RecordHistogram.recordBooleanHistogram(
"Search.ContextualSearch.Tap.ResultsSeen", wasPanelSeen);
}
/**
* Logs whether search results were seen for all gestures. Recorded for all users.
* @param wasPanelSeen Whether the panel was seen.
*/
public static void logAllResultsSeen(boolean wasPanelSeen) {
RecordHistogram.recordBooleanHistogram(
"Search.ContextualSearch.All.ResultsSeen", wasPanelSeen);
}
/**
* Logs the whether the panel was seen and the type of the trigger and if Bar nearly overlapped.
* If the panel was seen, logs the duration of the panel view into a BarOverlap or BarNoOverlap
......
......@@ -79352,6 +79352,15 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
<histogram name="Search.ContextualSearch.All.ResultsSeen" enum="Boolean">
<owner>donnd@chromium.org</owner>
<owner>twellington@chromium.org</owner>
<summary>
Records whether the user opened the panel when it was shown. Recorded for
all gestures for all users when the UX is hidden. Implemented for Android.
</summary>
</histogram>
<histogram name="Search.ContextualSearch.Ranker.FeaturesAvailable"
enum="Boolean">
<owner>donnd@chromium.org</owner>
......@@ -79463,6 +79472,16 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
<histogram name="Search.ContextualSearch.Tap.ResultsSeen" enum="Boolean">
<owner>donnd@chromium.org</owner>
<owner>twellington@chromium.org</owner>
<summary>
Records whether the user opened the panel when it was shown in response to a
tap gesture, for all users. Recorded when the UX is hidden. Implemented for
Android.
</summary>
</histogram>
<histogram name="Search.ContextualSearchAllCapsResultsSeen"
enum="ContextualSearchResultsSeen">
<obsolete>
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment