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

Unified Diff: content/browser/devtools/devtools_system_info_handler.cc

Issue 21682002: Expose GPU information to Telemetry. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Refactored more code into WebSocketBrowserConnection on nduca's request. Created 7 years, 4 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 | « content/browser/devtools/devtools_system_info_handler.h ('k') | content/content_browser.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/devtools/devtools_system_info_handler.cc
diff --git a/content/browser/devtools/devtools_system_info_handler.cc b/content/browser/devtools/devtools_system_info_handler.cc
new file mode 100644
index 0000000000000000000000000000000000000000..bab40480ef10722cd5fe923ccc02bb7923bfa740
--- /dev/null
+++ b/content/browser/devtools/devtools_system_info_handler.cc
@@ -0,0 +1,123 @@
+// 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 "content/browser/devtools/devtools_system_info_handler.h"
+
+#include "base/bind.h"
+#include "base/callback.h"
+#include "base/values.h"
+#include "content/browser/devtools/devtools_protocol_constants.h"
+#include "content/browser/gpu/gpu_data_manager_impl.h"
+#include "gpu/config/gpu_info.h"
+
+namespace content {
+
+namespace {
+
+const char kAuxAttributes[] = "auxAttributes";
+const char kDeviceId[] = "deviceId";
+const char kDeviceString[] = "deviceString";
+const char kDevices[] = "devices";
+const char kGPU[] = "gpu";
+const char kModelName[] = "modelName";
+const char kVendorId[] = "vendorId";
+const char kVendorString[] = "vendorString";
+
+class AuxGPUInfoEnumerator : public gpu::GPUInfo::Enumerator {
+ public:
+ AuxGPUInfoEnumerator(base::DictionaryValue* dictionary)
+ : dictionary_(dictionary),
+ in_aux_attributes_(false) { }
+
+ virtual void AddInt64(const char* name, int64 value) OVERRIDE {
+ if (in_aux_attributes_)
+ dictionary_->SetDouble(name, value);
+ }
+
+ virtual void AddInt(const char* name, int value) OVERRIDE {
+ if (in_aux_attributes_)
+ dictionary_->SetInteger(name, value);
+ }
+
+ virtual void AddString(const char* name, const std::string& value) OVERRIDE {
+ if (in_aux_attributes_)
+ dictionary_->SetString(name, value);
+ }
+
+ virtual void AddBool(const char* name, bool value) OVERRIDE {
+ if (in_aux_attributes_)
+ dictionary_->SetBoolean(name, value);
+ }
+
+ virtual void AddTimeDeltaInSecondsF(const char* name,
+ const base::TimeDelta& value) OVERRIDE {
+ if (in_aux_attributes_)
+ dictionary_->SetDouble(name, value.InSecondsF());
+ }
+
+ virtual void BeginGPUDevice() OVERRIDE {
+ }
+
+ virtual void EndGPUDevice() OVERRIDE {
+ }
+
+ virtual void BeginAuxAttributes() OVERRIDE {
+ in_aux_attributes_ = true;
+ }
+
+ virtual void EndAuxAttributes() OVERRIDE {
+ in_aux_attributes_ = false;
+ }
+
+ private:
+ base::DictionaryValue* dictionary_;
+ bool in_aux_attributes_;
+};
+
+base::DictionaryValue* GPUDeviceToDictionary(
+ const gpu::GPUInfo::GPUDevice& device) {
+ base::DictionaryValue* result = new base::DictionaryValue;
+ result->SetInteger(kVendorId, device.vendor_id);
+ result->SetInteger(kDeviceId, device.device_id);
+ result->SetString(kVendorString, device.vendor_string);
+ result->SetString(kDeviceString, device.device_string);
+ return result;
+}
+
+} // namespace
+
+DevToolsSystemInfoHandler::DevToolsSystemInfoHandler() {
+ RegisterCommandHandler(devtools::SystemInfo::getInfo::kName,
+ base::Bind(&DevToolsSystemInfoHandler::OnGetInfo,
+ base::Unretained(this)));
+}
+
+DevToolsSystemInfoHandler::~DevToolsSystemInfoHandler() {
+}
+
+scoped_refptr<DevToolsProtocol::Response>
+DevToolsSystemInfoHandler::OnGetInfo(
+ scoped_refptr<DevToolsProtocol::Command> command) {
+ gpu::GPUInfo gpu_info = GpuDataManagerImpl::GetInstance()->GetGPUInfo();
+ base::DictionaryValue* gpu_dict = new base::DictionaryValue;
+
+ base::ListValue* devices = new base::ListValue;
+ devices->Append(GPUDeviceToDictionary(gpu_info.gpu));
+ for (size_t ii = 0; ii < gpu_info.secondary_gpus.size(); ++ii) {
pfeldman 2013/08/15 05:49:47 nit: prefer iterator while iterating stl collectio
+ devices->Append(GPUDeviceToDictionary(gpu_info.secondary_gpus[ii]));
+ }
+ gpu_dict->Set(kDevices, devices);
+
+ base::DictionaryValue* aux_attributes = new base::DictionaryValue;
+ AuxGPUInfoEnumerator enumerator(aux_attributes);
+ gpu_info.EnumerateFields(&enumerator);
+ gpu_dict->Set(kAuxAttributes, aux_attributes);
+
+ base::DictionaryValue* system_dict = new base::DictionaryValue;
+ system_dict->SetString(kModelName, gpu_info.machine_model);
+ system_dict->Set(kGPU, gpu_dict);
+ return command->SuccessResponse(system_dict);
+}
+
+} // namespace content
« no previous file with comments | « content/browser/devtools/devtools_system_info_handler.h ('k') | content/content_browser.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698