metrics: log whether drives have seek penalties (and whether this was determined successfully).

chrome/browser/metrics/drive_metrics_provider.cc

 // 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.
 
 #include "chrome/browser/metrics/drive_metrics_provider.h"
 
 #include "base/base_paths.h"
 #include "base/bind.h"
 #include "base/files/file_path.h"
 #include "base/location.h"
 #include "base/logging.h"
+#include "base/metrics/histogram_macros.h"
 #include "base/path_service.h"
 #include "base/sys_info.h"
+#include "base/time/time.h"
 #include "chrome/common/chrome_paths.h"
 #include "content/public/browser/browser_thread.h"
 
 DriveMetricsProvider::DriveMetricsProvider() : weak_ptr_factory_(this) {}
 
 DriveMetricsProvider::~DriveMetricsProvider() {}
 
 void DriveMetricsProvider::ProvideSystemProfileMetrics(
     metrics::SystemProfileProto* system_profile_proto) {
   auto* hardware = system_profile_proto->mutable_hardware();
@@ skipping 29 matching lines @@
 // static
 void DriveMetricsProvider::QuerySeekPenalty(
     int path_service_key,
     DriveMetricsProvider::SeekPenaltyResponse* response) {
   DCHECK(response);
 
   base::FilePath path;
   if (!PathService::Get(path_service_key, &path))
     return;
 
+  base::TimeTicks start = base::TimeTicks::Now();
+
   response->success =
       base::SysInfo::HasSeekPenalty(path, &response->has_seek_penalty);
+
+  UMA_HISTOGRAM_TIMES("Hardware.Drive.HasSeekPenalty_Time",
+                      base::TimeTicks::Now() - start);
+  UMA_HISTOGRAM_BOOLEAN("Hardware.Drive.HasSeekPenalty_Success",
+                        response->success);
+  if (response->success) {
+    UMA_HISTOGRAM_BOOLEAN("Hardware.Drive.HasSeekPenalty",
+                          response->has_seek_penalty);
+  }
 }
 
 void DriveMetricsProvider::GotDriveMetrics(
     const DriveMetricsProvider::DriveMetrics& metrics) {
   DCHECK(thread_checker_.CalledOnValidThread());
   metrics_ = metrics;
   got_metrics_callback_.Run();
 }
 
 void DriveMetricsProvider::FillDriveMetrics(
     const DriveMetricsProvider::SeekPenaltyResponse& response,
     metrics::SystemProfileProto::Hardware::Drive* drive) {
   if (response.success)
     drive->set_has_seek_penalty(response.has_seek_penalty);
 }