metrics/base: log whether drives have seek penalties.

chrome/browser/metrics/drive_metrics_provider.cc

Index: chrome/browser/metrics/drive_metrics_provider.cc
diff --git a/chrome/browser/metrics/drive_metrics_provider.cc b/chrome/browser/metrics/drive_metrics_provider.cc
new file mode 100644
index 0000000000000000000000000000000000000000..3b63f72fa336bd68bbd8c993cd39c13336c3b0b0
--- /dev/null
+++ b/chrome/browser/metrics/drive_metrics_provider.cc
@@ -0,0 +1,77 @@
+// 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/path_service.h"
+#include "base/sys_info.h"
+#include "chrome/common/chrome_paths.h"
+#include "content/public/browser/browser_thread.h"
+
+DriveMetricsProvider::DriveMetricsProvider()
+    : got_metrics_(false), weak_ptr_factory_(this) {}
+
+DriveMetricsProvider::~DriveMetricsProvider() {}
+
+void DriveMetricsProvider::ProvideSystemProfileMetrics(
+    metrics::SystemProfileProto* system_profile_proto) {
+  if (!got_metrics_) {
+    content::BrowserThread::PostTaskAndReplyWithResult(
+      content::BrowserThread::FILE, FROM_HERE,
+      base::Bind(&DriveMetricsProvider::GetDriveMetrics,
+                 weak_ptr_factory_.GetWeakPtr()),

[Dan Beam, 2015/03/16 19:20:24]

this didn't work

+      base::Bind(&DriveMetricsProvider::GotDriveMetrics,
+                 weak_ptr_factory_.GetWeakPtr()));
+    return;
+  }
+
+  auto* hardware = system_profile_proto->mutable_hardware();
+  FillDriveMetrics(metrics_.app_drive, hardware->mutable_app_drive());
+  FillDriveMetrics(metrics_.user_data_drive,
+                   hardware->mutable_user_data_drive());
+}
+
+DriveMetricsProvider::SeekPenaltyResponse::SeekPenaltyResponse()
+    : success(false) {}
+
+DriveMetricsProvider::DriveMetrics DriveMetricsProvider::GetDriveMetrics() {
+  DCHECK_CURRENTLY_ON(content::BrowserThread::FILE);
+
+  DriveMetricsProvider::DriveMetrics metrics;
+  QuerySeekPenalty(base::FILE_EXE, &metrics.app_drive);
+  QuerySeekPenalty(chrome::FILE_LOCAL_STATE, &metrics.user_data_drive);
+  return metrics;
+}
+
+void DriveMetricsProvider::QuerySeekPenalty(
+    int path_service_key,
+    DriveMetricsProvider::SeekPenaltyResponse* response) {
+  DCHECK(response);
+
+  base::FilePath path;
+  if (!PathService::Get(path_service_key, &path))
+    return;
+
+  response->success =
+      base::SysInfo::HasSeekPenalty(path, &response->has_seek_penalty);
+}
+
+void DriveMetricsProvider::GotDriveMetrics(
+    const DriveMetricsProvider::DriveMetrics& metrics) {
+  DCHECK(thread_checker_.CalledOnValidThread());
+  got_metrics_ = true;
+  metrics_ = metrics;
+}
+
+void DriveMetricsProvider::FillDriveMetrics(
+    const DriveMetricsProvider::SeekPenaltyResponse& response,
+    metrics::SystemProfileProto::Hardware::Drive* drive) {
+  if (response.success)
+    drive->set_has_seek_penalty(response.has_seek_penalty);
+}