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

Unified Diff: chrome/browser/android/vr_shell/vr_shell_gl.cc

Issue 2865463003: Tracks GVR version crossed with headset type using UMA. (Closed)
Patch Set: Fixed comments Created 3 years, 6 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/android/vr_shell/vr_shell_delegate.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/android/vr_shell/vr_shell_gl.cc
diff --git a/chrome/browser/android/vr_shell/vr_shell_gl.cc b/chrome/browser/android/vr_shell/vr_shell_gl.cc
index deb62e5d0a953ba8eae7388de54ab9f7686036b7..2dbc09862e1f5dff01234d4ebd44094a4e41972f 100644
--- a/chrome/browser/android/vr_shell/vr_shell_gl.cc
+++ b/chrome/browser/android/vr_shell/vr_shell_gl.cc
@@ -22,8 +22,10 @@
#include "chrome/browser/android/vr_shell/vr_browser_interface.h"
#include "chrome/browser/android/vr_shell/vr_controller.h"
#include "chrome/browser/android/vr_shell/vr_gl_util.h"
+#include "chrome/browser/android/vr_shell/vr_metrics_util.h"
#include "chrome/browser/android/vr_shell/vr_shell.h"
#include "chrome/browser/android/vr_shell/vr_shell_renderer.h"
+#include "chrome/browser/android/vr_shell/vr_usage_monitor.h"
#include "device/vr/android/gvr/gvr_delegate.h"
#include "device/vr/android/gvr/gvr_device.h"
#include "device/vr/android/gvr/gvr_gamepad_data_provider.h"
@@ -171,12 +173,6 @@ std::unique_ptr<blink::WebMouseEvent> MakeMouseEvent(
return mouse_event;
}
-enum class ViewerType {
- UNKNOWN_TYPE = 0,
- CARDBOARD = 1,
- DAYDREAM = 2,
- VIEWER_TYPE_MAX,
-};
void MatfToGvrMat(const vr::Mat4f& in, gvr::Mat4f* out) {
// If our std::array implementation doesn't have any non-data members, we can
@@ -435,23 +431,10 @@ void VrShellGl::GvrInit(gvr_context* gvr_api) {
gvr_api_ = gvr::GvrApi::WrapNonOwned(gvr_api);
controller_.reset(new VrController(gvr_api));
- ViewerType viewerType;
- switch (gvr_api_->GetViewerType()) {
- case gvr::ViewerType::GVR_VIEWER_TYPE_DAYDREAM:
- viewerType = ViewerType::DAYDREAM;
- break;
- case gvr::ViewerType::GVR_VIEWER_TYPE_CARDBOARD:
- viewerType = ViewerType::CARDBOARD;
- break;
- default:
- NOTREACHED();
- viewerType = ViewerType::UNKNOWN_TYPE;
- break;
- }
- UMA_HISTOGRAM_ENUMERATION("VRViewerType", static_cast<int>(viewerType),
- static_cast<int>(ViewerType::VIEWER_TYPE_MAX));
+ VrMetricsUtil::LogVrViewerType(gvr_api);
- cardboard_ = (viewerType == ViewerType::CARDBOARD);
+ cardboard_ =
+ (gvr_api_->GetViewerType() == gvr::ViewerType::GVR_VIEWER_TYPE_CARDBOARD);
if (cardboard_ && web_vr_mode_) {
browser_->ToggleCardboardGamepad(true);
}
« no previous file with comments | « chrome/browser/android/vr_shell/vr_shell_delegate.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698