Commit 5bd1bf91 authored by tby's avatar tby Committed by Commit Bot

[Hindsight] Record launcher usage.

Add a CrSOActionRecorder call for each usage of the launcher, with type
and query hash as conditions.

Bug: 1012936
Change-Id: Ie1e42487c67dc922407bb39f538b67384297f48a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1865911Reviewed-by: default avatarJia Meng <jiameng@chromium.org>
Reviewed-by: default avatarCharles . <charleszhao@chromium.org>
Commit-Queue: Tony Yeoman <tby@chromium.org>
Cr-Commit-Position: refs/heads/master@{#706772}
parent 8c144581
...@@ -15,6 +15,8 @@ ...@@ -15,6 +15,8 @@
#include "ash/public/cpp/tablet_mode.h" #include "ash/public/cpp/tablet_mode.h"
#include "base/bind.h" #include "base/bind.h"
#include "base/metrics/histogram_macros.h" #include "base/metrics/histogram_macros.h"
#include "base/metrics/metrics_hashes.h"
#include "base/strings/strcat.h"
#include "base/strings/string_util.h" #include "base/strings/string_util.h"
#include "base/strings/utf_string_conversions.h" #include "base/strings/utf_string_conversions.h"
#include "chrome/browser/chromeos/profiles/profile_helper.h" #include "chrome/browser/chromeos/profiles/profile_helper.h"
...@@ -23,6 +25,7 @@ ...@@ -23,6 +25,7 @@
#include "chrome/browser/ui/app_list/app_list_controller_delegate.h" #include "chrome/browser/ui/app_list/app_list_controller_delegate.h"
#include "chrome/browser/ui/app_list/app_list_model_updater.h" #include "chrome/browser/ui/app_list/app_list_model_updater.h"
#include "chrome/browser/ui/app_list/search/chrome_search_result.h" #include "chrome/browser/ui/app_list/search/chrome_search_result.h"
#include "chrome/browser/ui/app_list/search/cros_action_history/cros_action_recorder.h"
#include "chrome/browser/ui/app_list/search/search_provider.h" #include "chrome/browser/ui/app_list/search/search_provider.h"
#include "chrome/browser/ui/app_list/search/search_result_ranker/app_list_launch_recorder.h" #include "chrome/browser/ui/app_list/search/search_result_ranker/app_list_launch_recorder.h"
#include "chrome/browser/ui/app_list/search/search_result_ranker/histogram_util.h" #include "chrome/browser/ui/app_list/search/search_result_ranker/histogram_util.h"
...@@ -258,6 +261,14 @@ void SearchController::Train(AppLaunchData&& app_launch_data) { ...@@ -258,6 +261,14 @@ void SearchController::Train(AppLaunchData&& app_launch_data) {
} }
} }
// CrOS action recorder.
CrOSActionRecorder::GetCrosActionRecorder()->RecordAction(
{base::StrCat(
{"SearchResultLaunched-", NormalizeId(app_launch_data.id)})},
{{"ResultType", static_cast<int>(app_launch_data.ranking_item_type)},
{"Query", static_cast<int>(
base::HashMetricName(base::UTF16ToUTF8(last_query_)))}});
for (const auto& provider : providers_) for (const auto& provider : providers_)
provider->Train(app_launch_data.id, app_launch_data.ranking_item_type); provider->Train(app_launch_data.id, app_launch_data.ranking_item_type);
app_launch_data.query = base::UTF16ToUTF8(last_query_); app_launch_data.query = base::UTF16ToUTF8(last_query_);
......
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