New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 788768 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Feature
Proj-XR

Blocked on:
issue 791747

Blocking:
issue 715118



Sign in to add a comment

Assets Component: Collect relevant initial UMA

Project Member Reported by tiborg@chromium.org, Nov 27 2017

Issue description

We would like to track the following UMA for our initial VR assets component:

- Duration from entering VR browsing/WebVR until component is ready: tells us how long the user has to wait until assets are available in a particular VR mode.
- Status of the component when entering VR browsing/WebVR: gives us the ratio of users that can enter VR without friction and also other error situations.
- Duration from launching Chrome until component is ready: tells us generally how long it takes to have the component available.
- Update status: generally tells us whether Chrome works with the component.
- Data mode (WiFi, cellular) when entering VR mode: the assets will have a considerable size. This gives an indication how many users can download the component over WiFi.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 29 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d5dc7c7e6de4e31e0cedc7f202550987e0772588

commit d5dc7c7e6de4e31e0cedc7f202550987e0772588
Author: Tibor Goldschwendt <tiborg@chromium.org>
Date: Wed Nov 29 22:43:17 2017

[vr] Track assets component's status when entering VR and ready latency

Bug:  788768 
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: I08799d797537598d38d4d5b8f2862f4021942910
Reviewed-on: https://chromium-review.googlesource.com/788251
Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Reviewed-by: Bill Orr <billorr@chromium.org>
Reviewed-by: Biao She <bshe@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520280}
[modify] https://crrev.com/d5dc7c7e6de4e31e0cedc7f202550987e0772588/chrome/browser/android/vr_shell/vr_shell.cc
[modify] https://crrev.com/d5dc7c7e6de4e31e0cedc7f202550987e0772588/chrome/browser/android/vr_shell/vr_shell.h
[modify] https://crrev.com/d5dc7c7e6de4e31e0cedc7f202550987e0772588/chrome/browser/android/vr_shell/vr_usage_monitor.cc
[modify] https://crrev.com/d5dc7c7e6de4e31e0cedc7f202550987e0772588/chrome/browser/android/vr_shell/vr_usage_monitor.h
[modify] https://crrev.com/d5dc7c7e6de4e31e0cedc7f202550987e0772588/chrome/browser/vr/BUILD.gn
[modify] https://crrev.com/d5dc7c7e6de4e31e0cedc7f202550987e0772588/chrome/browser/vr/assets.cc
[modify] https://crrev.com/d5dc7c7e6de4e31e0cedc7f202550987e0772588/chrome/browser/vr/assets.h
[add] https://crrev.com/d5dc7c7e6de4e31e0cedc7f202550987e0772588/chrome/browser/vr/metrics_helper.cc
[add] https://crrev.com/d5dc7c7e6de4e31e0cedc7f202550987e0772588/chrome/browser/vr/metrics_helper.h
[add] https://crrev.com/d5dc7c7e6de4e31e0cedc7f202550987e0772588/chrome/browser/vr/mode.h
[modify] https://crrev.com/d5dc7c7e6de4e31e0cedc7f202550987e0772588/tools/metrics/histograms/enums.xml
[modify] https://crrev.com/d5dc7c7e6de4e31e0cedc7f202550987e0772588/tools/metrics/histograms/histograms.xml

Project Member

Comment 2 by bugdroid1@chromium.org, Nov 30 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/55d4236e77f3354a36f7ba38bfc5f971eb74c442

commit 55d4236e77f3354a36f7ba38bfc5f971eb74c442
Author: Tibor Goldschwendt <tiborg@chromium.org>
Date: Thu Nov 30 00:12:29 2017

[vr] Track connection type when registering component, entering VR

Bug:  788768 
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: I3dfde1ba03fd7a0d7aa4fc6ea2a62ca03a87582b
Reviewed-on: https://chromium-review.googlesource.com/791332
Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: Joshua Pawlicki <waffles@chromium.org>
Reviewed-by: Biao She <bshe@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520319}
[modify] https://crrev.com/55d4236e77f3354a36f7ba38bfc5f971eb74c442/chrome/browser/component_updater/vr_assets_component_installer.cc
[modify] https://crrev.com/55d4236e77f3354a36f7ba38bfc5f971eb74c442/chrome/browser/vr/BUILD.gn
[modify] https://crrev.com/55d4236e77f3354a36f7ba38bfc5f971eb74c442/chrome/browser/vr/metrics_helper.cc
[modify] https://crrev.com/55d4236e77f3354a36f7ba38bfc5f971eb74c442/chrome/browser/vr/metrics_helper.h
[modify] https://crrev.com/55d4236e77f3354a36f7ba38bfc5f971eb74c442/tools/metrics/histograms/histograms.xml

Project Member

Comment 3 by bugdroid1@chromium.org, Nov 30 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/58eb5351ab2d330efc7045222f1aa9006dcb9e68

commit 58eb5351ab2d330efc7045222f1aa9006dcb9e68
Author: Tibor Goldschwendt <tiborg@chromium.org>
Date: Thu Nov 30 22:25:24 2017

[vr] Track assets component update status and assets load status

Bug:  788768 
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: Ifd74ce6a4384a4585631e15effc09b29d2244949
Reviewed-on: https://chromium-review.googlesource.com/794041
Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Biao She <bshe@chromium.org>
Reviewed-by: Joshua Pawlicki <waffles@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520718}
[modify] https://crrev.com/58eb5351ab2d330efc7045222f1aa9006dcb9e68/chrome/browser/android/vr_shell/gl_browser_interface.h
[modify] https://crrev.com/58eb5351ab2d330efc7045222f1aa9006dcb9e68/chrome/browser/android/vr_shell/vr_gl_thread.cc
[modify] https://crrev.com/58eb5351ab2d330efc7045222f1aa9006dcb9e68/chrome/browser/android/vr_shell/vr_gl_thread.h
[modify] https://crrev.com/58eb5351ab2d330efc7045222f1aa9006dcb9e68/chrome/browser/android/vr_shell/vr_shell.cc
[modify] https://crrev.com/58eb5351ab2d330efc7045222f1aa9006dcb9e68/chrome/browser/android/vr_shell/vr_shell.h
[modify] https://crrev.com/58eb5351ab2d330efc7045222f1aa9006dcb9e68/chrome/browser/android/vr_shell/vr_shell_gl.cc
[modify] https://crrev.com/58eb5351ab2d330efc7045222f1aa9006dcb9e68/chrome/browser/android/vr_shell/vr_shell_gl.h
[modify] https://crrev.com/58eb5351ab2d330efc7045222f1aa9006dcb9e68/chrome/browser/component_updater/vr_assets_component_installer.cc
[modify] https://crrev.com/58eb5351ab2d330efc7045222f1aa9006dcb9e68/chrome/browser/vr/BUILD.gn
[modify] https://crrev.com/58eb5351ab2d330efc7045222f1aa9006dcb9e68/chrome/browser/vr/assets.cc
[modify] https://crrev.com/58eb5351ab2d330efc7045222f1aa9006dcb9e68/chrome/browser/vr/assets.h
[add] https://crrev.com/58eb5351ab2d330efc7045222f1aa9006dcb9e68/chrome/browser/vr/assets_component_update_status.h
[add] https://crrev.com/58eb5351ab2d330efc7045222f1aa9006dcb9e68/chrome/browser/vr/assets_load_status.h
[modify] https://crrev.com/58eb5351ab2d330efc7045222f1aa9006dcb9e68/chrome/browser/vr/metrics_helper.cc
[modify] https://crrev.com/58eb5351ab2d330efc7045222f1aa9006dcb9e68/chrome/browser/vr/metrics_helper.h
[modify] https://crrev.com/58eb5351ab2d330efc7045222f1aa9006dcb9e68/tools/metrics/histograms/enums.xml
[modify] https://crrev.com/58eb5351ab2d330efc7045222f1aa9006dcb9e68/tools/metrics/histograms/histograms.xml

Project Member

Comment 4 by bugdroid1@chromium.org, Dec 1 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/57703df8f1c596c7d51095a91d70095f729c071a

commit 57703df8f1c596c7d51095a91d70095f729c071a
Author: Tibor Goldschwendt <tiborg@chromium.org>
Date: Fri Dec 01 22:05:34 2017

[vr] Add UMA to track duration until assets component is ready

Tracks the duration from Chrome launch until the assets component is
ready to use.

Bug:  788768 
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: I5e500bc2ea3eb8a508aa3453dd62caad496e9e4a
Reviewed-on: https://chromium-review.googlesource.com/790971
Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: David Dorwin <ddorwin@chromium.org>
Reviewed-by: Biao She <bshe@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521081}
[modify] https://crrev.com/57703df8f1c596c7d51095a91d70095f729c071a/chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrShellDelegate.java
[modify] https://crrev.com/57703df8f1c596c7d51095a91d70095f729c071a/chrome/browser/android/vr_shell/vr_shell_delegate.cc
[modify] https://crrev.com/57703df8f1c596c7d51095a91d70095f729c071a/chrome/browser/vr/metrics_helper.cc
[modify] https://crrev.com/57703df8f1c596c7d51095a91d70095f729c071a/chrome/browser/vr/metrics_helper.h
[modify] https://crrev.com/57703df8f1c596c7d51095a91d70095f729c071a/tools/metrics/histograms/histograms.xml

Project Member

Comment 5 by bugdroid1@chromium.org, Dec 1 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/35eb246ef089371ad11b6bd24c384004eea274e3

commit 35eb246ef089371ad11b6bd24c384004eea274e3
Author: David Dorwin <ddorwin@chromium.org>
Date: Fri Dec 01 22:31:10 2017

Restructure VR asset component histogram names

These histograms were just added so this should not affect data.

Bug:  788768 
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: Ibac06800a1f7e491cd53067e4c6f0f7c2e7edbfb
Reviewed-on: https://chromium-review.googlesource.com/798733
Commit-Queue: David Dorwin <ddorwin@chromium.org>
Reviewed-by: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521092}
[modify] https://crrev.com/35eb246ef089371ad11b6bd24c384004eea274e3/chrome/browser/vr/metrics_helper.cc
[modify] https://crrev.com/35eb246ef089371ad11b6bd24c384004eea274e3/tools/metrics/histograms/histograms.xml

Blockedon: 791747
Project Member

Comment 7 by bugdroid1@chromium.org, Dec 6 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b9ddd92d20ab2e23e464d50bad318f409e6db1d9

commit b9ddd92d20ab2e23e464d50bad318f409e6db1d9
Author: David Dorwin <ddorwin@chromium.org>
Date: Wed Dec 06 03:53:09 2017

Rename VRAssetsComponentEnterStatus to VRComponentStatus

Bug:  788768 
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: I205ec4c543c26f42e41d124981a9a258faa7c594
Reviewed-on: https://chromium-review.googlesource.com/798632
Commit-Queue: David Dorwin <ddorwin@chromium.org>
Reviewed-by: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521993}
[modify] https://crrev.com/b9ddd92d20ab2e23e464d50bad318f409e6db1d9/chrome/browser/vr/metrics_helper.cc
[modify] https://crrev.com/b9ddd92d20ab2e23e464d50bad318f409e6db1d9/tools/metrics/histograms/enums.xml
[modify] https://crrev.com/b9ddd92d20ab2e23e464d50bad318f409e6db1d9/tools/metrics/histograms/histograms.xml

Labels: VR-metrics
Project Member

Comment 9 by bugdroid1@chromium.org, Dec 11 2017

Labels: merge-merged-3282
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3d6a4471d9469307a0c03868a11ece2c64d5aa10

commit 3d6a4471d9469307a0c03868a11ece2c64d5aa10
Author: Tibor Goldschwendt <tiborg@chromium.org>
Date: Mon Dec 11 18:50:59 2017

[vr] Add UMA to track duration until assets component is ready

Tracks the duration from Chrome launch until the assets component is
ready to use.

Bug:  788768 
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: I5e500bc2ea3eb8a508aa3453dd62caad496e9e4a
Reviewed-on: https://chromium-review.googlesource.com/790971
Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: David Dorwin <ddorwin@chromium.org>
Reviewed-by: Biao She <bshe@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#521081}(cherry picked from commit 57703df8f1c596c7d51095a91d70095f729c071a)
Reviewed-on: https://chromium-review.googlesource.com/819492
Reviewed-by: Tibor Goldschwendt <tiborg@chromium.org>
Cr-Commit-Position: refs/branch-heads/3282@{#136}
Cr-Branched-From: 5fdc0fab22ce7efd32532ee989b223fa12f8171e-refs/heads/master@{#520840}
[modify] https://crrev.com/3d6a4471d9469307a0c03868a11ece2c64d5aa10/chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrShellDelegate.java
[modify] https://crrev.com/3d6a4471d9469307a0c03868a11ece2c64d5aa10/chrome/browser/android/vr_shell/vr_shell_delegate.cc
[modify] https://crrev.com/3d6a4471d9469307a0c03868a11ece2c64d5aa10/chrome/browser/vr/metrics_helper.cc
[modify] https://crrev.com/3d6a4471d9469307a0c03868a11ece2c64d5aa10/chrome/browser/vr/metrics_helper.h
[modify] https://crrev.com/3d6a4471d9469307a0c03868a11ece2c64d5aa10/tools/metrics/histograms/histograms.xml

Project Member

Comment 10 by bugdroid1@chromium.org, Dec 11 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d7f44a3f1c2ce4e99f635879db686b106a391f41

commit d7f44a3f1c2ce4e99f635879db686b106a391f41
Author: David Dorwin <ddorwin@chromium.org>
Date: Mon Dec 11 19:36:27 2017

Restructure VR asset component histogram names

These histograms were just added so this should not affect data.

Bug:  788768 
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: Ibac06800a1f7e491cd53067e4c6f0f7c2e7edbfb
Reviewed-on: https://chromium-review.googlesource.com/798733
Commit-Queue: David Dorwin <ddorwin@chromium.org>
Reviewed-by: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#521092}(cherry picked from commit 35eb246ef089371ad11b6bd24c384004eea274e3)
Reviewed-on: https://chromium-review.googlesource.com/820372
Reviewed-by: David Dorwin <ddorwin@chromium.org>
Cr-Commit-Position: refs/branch-heads/3282@{#140}
Cr-Branched-From: 5fdc0fab22ce7efd32532ee989b223fa12f8171e-refs/heads/master@{#520840}
[modify] https://crrev.com/d7f44a3f1c2ce4e99f635879db686b106a391f41/chrome/browser/vr/metrics_helper.cc
[modify] https://crrev.com/d7f44a3f1c2ce4e99f635879db686b106a391f41/tools/metrics/histograms/histograms.xml

Status: Fixed (was: Started)
Components: Internals>XR

Sign in to add a comment