Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1941)

Unified Diff: chrome/browser/metrics/metrics_log_chromeos.cc

Issue 146913005: Factor ChromeOS specific code out of MetricsLog. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address rtenneti nits. Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/metrics/metrics_log_chromeos.h ('k') | chrome/browser/metrics/metrics_log_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/metrics/metrics_log_chromeos.cc
diff --git a/chrome/browser/metrics/metrics_log_chromeos.cc b/chrome/browser/metrics/metrics_log_chromeos.cc
new file mode 100644
index 0000000000000000000000000000000000000000..cb427c469477c76681c3f51678381d1ecab543b3
--- /dev/null
+++ b/chrome/browser/metrics/metrics_log_chromeos.cc
@@ -0,0 +1,168 @@
+// Copyright 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/metrics_log_chromeos.h"
+
+#include "base/prefs/pref_service.h"
+#include "base/strings/string_number_conversions.h"
+#include "base/strings/string_util.h"
+#include "base/strings/utf_string_conversions.h"
+#include "chrome/browser/chromeos/login/user_manager.h"
+#include "chrome/common/metrics/proto/chrome_user_metrics_extension.pb.h"
+#include "chrome/common/pref_names.h"
+#include "device/bluetooth/bluetooth_adapter.h"
+#include "device/bluetooth/bluetooth_adapter_factory.h"
+#include "device/bluetooth/bluetooth_device.h"
+
+using metrics::ChromeUserMetricsExtension;
+using metrics::PerfDataProto;
+using metrics::SystemProfileProto;
+typedef SystemProfileProto::Hardware::Bluetooth::PairedDevice PairedDevice;
+
+PairedDevice::Type AsBluetoothDeviceType(
+ device::BluetoothDevice::DeviceType device_type) {
+ switch (device_type) {
+ case device::BluetoothDevice::DEVICE_UNKNOWN:
+ return PairedDevice::DEVICE_UNKNOWN;
+ case device::BluetoothDevice::DEVICE_COMPUTER:
+ return PairedDevice::DEVICE_COMPUTER;
+ case device::BluetoothDevice::DEVICE_PHONE:
+ return PairedDevice::DEVICE_PHONE;
+ case device::BluetoothDevice::DEVICE_MODEM:
+ return PairedDevice::DEVICE_MODEM;
+ case device::BluetoothDevice::DEVICE_AUDIO:
+ return PairedDevice::DEVICE_AUDIO;
+ case device::BluetoothDevice::DEVICE_CAR_AUDIO:
+ return PairedDevice::DEVICE_CAR_AUDIO;
+ case device::BluetoothDevice::DEVICE_VIDEO:
+ return PairedDevice::DEVICE_VIDEO;
+ case device::BluetoothDevice::DEVICE_PERIPHERAL:
+ return PairedDevice::DEVICE_PERIPHERAL;
+ case device::BluetoothDevice::DEVICE_JOYSTICK:
+ return PairedDevice::DEVICE_JOYSTICK;
+ case device::BluetoothDevice::DEVICE_GAMEPAD:
+ return PairedDevice::DEVICE_GAMEPAD;
+ case device::BluetoothDevice::DEVICE_KEYBOARD:
+ return PairedDevice::DEVICE_KEYBOARD;
+ case device::BluetoothDevice::DEVICE_MOUSE:
+ return PairedDevice::DEVICE_MOUSE;
+ case device::BluetoothDevice::DEVICE_TABLET:
+ return PairedDevice::DEVICE_TABLET;
+ case device::BluetoothDevice::DEVICE_KEYBOARD_MOUSE_COMBO:
+ return PairedDevice::DEVICE_KEYBOARD_MOUSE_COMBO;
+ }
+
+ NOTREACHED();
+ return PairedDevice::DEVICE_UNKNOWN;
+}
+
+MetricsLogChromeOS::~MetricsLogChromeOS() {
+}
+
+MetricsLogChromeOS::MetricsLogChromeOS(ChromeUserMetricsExtension* uma_proto)
+ : uma_proto_(uma_proto) {
+ UpdateMultiProfileUserCount();
+}
+
+void MetricsLogChromeOS::LogChromeOSMetrics() {
+ PerfDataProto perf_data_proto;
+ if (perf_provider_.GetPerfData(&perf_data_proto))
+ uma_proto_->add_perf_data()->Swap(&perf_data_proto);
+
+ WriteBluetoothProto();
+ UpdateMultiProfileUserCount();
+}
+
+void MetricsLogChromeOS::WriteRealtimeStabilityAttributes(PrefService* pref) {
+ SystemProfileProto::Stability* stability =
+ uma_proto_->mutable_system_profile()->mutable_stability();
+
+ int count = pref->GetInteger(prefs::kStabilityOtherUserCrashCount);
+ if (count) {
+ stability->set_other_user_crash_count(count);
+ pref->SetInteger(prefs::kStabilityOtherUserCrashCount, 0);
+ }
+
+ count = pref->GetInteger(prefs::kStabilityKernelCrashCount);
+ if (count) {
+ stability->set_kernel_crash_count(count);
+ pref->SetInteger(prefs::kStabilityKernelCrashCount, 0);
+ }
+
+ count = pref->GetInteger(prefs::kStabilitySystemUncleanShutdownCount);
+ if (count) {
+ stability->set_unclean_system_shutdown_count(count);
+ pref->SetInteger(prefs::kStabilitySystemUncleanShutdownCount, 0);
+ }
+}
+
+void MetricsLogChromeOS::WriteBluetoothProto() {
+ SystemProfileProto::Hardware* hardware =
+ uma_proto_->mutable_system_profile()->mutable_hardware();
+
+ // BluetoothAdapterFactory::GetAdapter is synchronous on Chrome OS; if that
+ // changes this will fail at the DCHECK().
+ device::BluetoothAdapterFactory::GetAdapter(
+ base::Bind(&MetricsLogChromeOS::SetBluetoothAdapter,
+ base::Unretained(this)));
+ DCHECK(adapter_.get());
+
+ SystemProfileProto::Hardware::Bluetooth* bluetooth =
+ hardware->mutable_bluetooth();
+
+ bluetooth->set_is_present(adapter_->IsPresent());
+ bluetooth->set_is_enabled(adapter_->IsPowered());
+
+ device::BluetoothAdapter::DeviceList devices = adapter_->GetDevices();
+ for (device::BluetoothAdapter::DeviceList::iterator iter =
+ devices.begin(); iter != devices.end(); ++iter) {
+ PairedDevice* paired_device = bluetooth->add_paired_device();
+
+ device::BluetoothDevice* device = *iter;
+ paired_device->set_bluetooth_class(device->GetBluetoothClass());
+ paired_device->set_type(AsBluetoothDeviceType(device->GetDeviceType()));
+
+ // |address| is xx:xx:xx:xx:xx:xx, extract the first three components and
+ // pack into a uint32.
+ std::string address = device->GetAddress();
+ if (address.size() > 9 &&
+ address[2] == ':' && address[5] == ':' && address[8] == ':') {
+ std::string vendor_prefix_str;
+ uint64 vendor_prefix;
+
+ base::RemoveChars(address.substr(0, 9), ":", &vendor_prefix_str);
+ DCHECK_EQ(6U, vendor_prefix_str.size());
+ base::HexStringToUInt64(vendor_prefix_str, &vendor_prefix);
+
+ paired_device->set_vendor_prefix(vendor_prefix);
+ }
+
+ paired_device->set_vendor_id(device->GetVendorID());
+ paired_device->set_product_id(device->GetProductID());
+ paired_device->set_device_id(device->GetDeviceID());
+ }
+}
+
+void MetricsLogChromeOS::UpdateMultiProfileUserCount() {
+ metrics::SystemProfileProto* system_profile =
+ uma_proto_->mutable_system_profile();
+
+ if (chromeos::UserManager::IsInitialized() &&
+ chromeos::UserManager::Get()->IsMultipleProfilesAllowed()) {
+ size_t user_count = chromeos::UserManager::Get()->GetLoggedInUsers().size();
+
+ // We invalidate the user count if it changed while the log was open.
+ if (system_profile->has_multi_profile_user_count() &&
+ user_count != system_profile->multi_profile_user_count()) {
+ user_count = 0;
+ }
+
+ system_profile->set_multi_profile_user_count(user_count);
+ }
+}
+
+void MetricsLogChromeOS::SetBluetoothAdapter(
+ scoped_refptr<device::BluetoothAdapter> adapter) {
+ adapter_ = adapter;
+}
« no previous file with comments | « chrome/browser/metrics/metrics_log_chromeos.h ('k') | chrome/browser/metrics/metrics_log_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698