mac: Add metrics to record Bluetooth availability and capabilities.

chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.cc

 // Copyright (c) 2013 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/chrome_browser_main_extra_parts_metrics.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/cpu.h"
 #include "base/metrics/histogram.h"
 #include "base/metrics/sparse_histogram.h"
 #include "base/sys_info.h"
 #include "base/threading/sequenced_worker_pool.h"
 #include "base/time/time.h"
 #include "chrome/browser/about_flags.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chrome_browser_main.h"
 #include "chrome/browser/chrome_browser_metrics_service_observer.h"
+#include "chrome/browser/mac/hardware_utility.h"
 #include "chrome/browser/pref_service_flags_storage.h"
 #include "chrome/browser/shell_integration.h"
 #include "content/public/browser/browser_thread.h"
 #include "ui/base/touch/touch_device.h"
 #include "ui/base/ui_base_switches.h"
 #include "ui/events/event_switches.h"
 
 #if defined(OS_LINUX) && !defined(OS_CHROMEOS)
 #include <gnu/libc-version.h>
 
@@ skipping 56 matching lines @@
   UMA_HISTOGRAM_SPARSE_SLOWLY("Platform.LogicalCpuCount",
                               base::SysInfo::NumberOfProcessors());
 }
 
 // Called on the blocking pool some time after startup to avoid slowing down
 // startup with metrics that aren't trivial to compute.
 void RecordStartupMetricsOnBlockingPool() {
 #if defined(OS_WIN)
   GoogleUpdateSettings::RecordChromeUpdatePolicyHistograms();
 #endif  // defined(OS_WIN)
+
+#if defined(OS_MACOSX) && !defined(OS_IOS)
+  hardware_utility::BluetoothAvailability availability =
+      hardware_utility::GetBluetoothAvailability();
+  UMA_HISTOGRAM_ENUMERATION("OSX.BluetoothAvailability",
+                            availability,
+                            hardware_utility::BLUETOOTH_AVAILABILITY_COUNT);

[Ilya Sherman, 2014/07/09 21:19:33]

I might be mistaken, but I don't think that the bluetooth system is threadsafe. 
Are you sure that it's ok to query bluetooth state from the blocking pool's
thread?

[Ilya Sherman, 2014/07/09 21:19:33]

By the way, note that there is a Bluetooth section in the system_profile.proto
message definition.  I think that's currently only filled out on ChromeOS, and
the histogram is probably more convenient to work with, but still... FYI.

[erikchen, 2014/07/11 17:25:44]

We're not touching the bluetooth system, we're touching IOKit, which should be
fine.

[erikchen, 2014/07/11 17:25:44]

As per offline discussion, I am keeping this a histogram.

+#endif  // defined(OS_MACOSX) && !defined(OS_IOS)
 }
 
 void RecordLinuxGlibcVersion() {
 #if defined(OS_LINUX) && !defined(OS_CHROMEOS)
   Version version(gnu_get_libc_version());
 
   UMALinuxGlibcVersion glibc_version_result = UMA_LINUX_GLIBC_NOT_PARSEABLE;
   if (version.IsValid() && version.components().size() == 2) {
     glibc_version_result = UMA_LINUX_GLIBC_UNKNOWN;
     int glibc_major_version = version.components()[0];
@@ skipping 124 matching lines @@
 #endif
 }
 
 namespace chrome {
 
 void AddMetricsExtraParts(ChromeBrowserMainParts* main_parts) {
   main_parts->AddParts(new ChromeBrowserMainExtraPartsMetrics());
 }
 
 }  // namespace chrome