Handle an edge case to prevent a possible crash while logging.

UMA logging checks that enum values are strictly less than the specified
max value.

Bug: 1159285
Change-Id: I94b12b272c51d3c73285446807ab5aa57b07d9b6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2596729
Commit-Queue: Rachel Wong <wrong@chromium.org>
Commit-Queue: Tony Yeoman <tby@chromium.org>
Reviewed-by: Tony Yeoman <tby@chromium.org>
Cr-Commit-Position: refs/heads/master@{#837935}

chrome/browser/ui/app_list/search/search_metrics_observer.cc

@@ -60,6 +60,13 @@ void LogTypeAction(const std::string& histogram_prefix,
                    ash::AppListNotifier::Location location,
                    const base::string16& query,
                    const SearchMetricsObserver::Result& result) {
+  if (result.type == ash::SEARCH_RESULT_TYPE_BOUNDARY) {
+    // TODO(crbug.com/1159285): The boundary value is the default value for
+    // result types, but results should have a non-default type. In any case,
+    // return here to prevent a crash from logging the boundary value.
+    return;
+  }
+
   const std::string histogram_name = base::StrCat(
       {histogram_prefix, ".", GetHistogramSuffix(location, query)});
   base::UmaHistogramEnumeration(histogram_name, result.type,