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

Issue 852550 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 14
Cc:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Record better UMA about early omnibox use

Project Member Reported by mariakho...@chromium.org, Jun 13 2018

Issue description

Chrome spends a lot of time doing early layout inflation and also doing asynchronous startup so that the user is able to interact with the omnibox early.

However, we don't know how useful this is to the users in general and low-end device users in particular.

We currently have a histogram called MobileStartup.ToolbarFirstFocusTime.<ActivityName>, but it doesn't have sufficient resolution at the low-end and also doesn't tell us whether the user waited for the last page to get restored before clicking omnibox (e.g. on high end even if they clicked in < 1s bucket, the page restore could have been already completed).

The proposal is instead to record the following histogram:
# of users who click on omnibox before native finished initialization
# of users who click on omnibox after native initialization but before first meaningful paint occurred
# of users who click while first page is loading
# of users who clicked within several seconds of the first page load finishing

This would allow us to determine what fraction of users today is waiting for page load to occur before navigating somewhere else and how often do we need to support users early omnibox interaction. Perhaps we can get rid of async initialization altogether.
 
Cc: mheikal@chromium.org
Owner: mheikal@chromium.org
Status: Assigned (was: Available)
Are we deprecating the time based histogram and removing it from the code or should that remain as it is?
One of the reasons for why there was a bump at 0th bucket was because we were/are double reporting this histogram if the first focus happened before the deferred startup handler is run. The fix for the double reporting bug is simple but I am unsure if it would require a rename for the older histograms or not.

Due to the same bug "MobileStartup.ToolbarFirstDrawTime" histogram is also being double reported.
I think it's worth a rename since it's a known shift to this histogram distribution. 
Project Member

Comment 5 by bugdroid1@chromium.org, Jul 4

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

commit 7a5a4d531fe80bfb904c736fc737ace196621e1d
Author: Mohamed Heikal <mheikal@google.com>
Date: Wed Jul 04 21:57:49 2018

[UMA] Adds UMA histogram to track startup state on first Url bar focus

This CL adds a histogram
"MobileStartup.ToolbarFirstFocusStartupState.<activity name>" to track
the state of chrome startup at the time of the first focus event on the
URL bar. The current histogram "MobileStartup.ToolbarFirstDrawTime.<activity
name>" only tracks the time since startup but the state of chrome
startup at that time would differ based on the capabilities of the
device. The new histogram, tracks the more useful startup state instead.

Bug:  852550 
Change-Id: I7c99c53e27dba5843eaaeab083c95be777513713
Reviewed-on: https://chromium-review.googlesource.com/1107308
Commit-Queue: Mohamed Heikal <mheikal@chromium.org>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Reviewed-by: Brian White <bcwhite@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572662}
[modify] https://crrev.com/7a5a4d531fe80bfb904c736fc737ace196621e1d/chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java
[add] https://crrev.com/7a5a4d531fe80bfb904c736fc737ace196621e1d/chrome/android/java/src/org/chromium/chrome/browser/metrics/OmniboxStartupMetrics.java
[modify] https://crrev.com/7a5a4d531fe80bfb904c736fc737ace196621e1d/chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBar.java
[modify] https://crrev.com/7a5a4d531fe80bfb904c736fc737ace196621e1d/chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java
[modify] https://crrev.com/7a5a4d531fe80bfb904c736fc737ace196621e1d/chrome/android/java/src/org/chromium/chrome/browser/omnibox/UrlBar.java
[modify] https://crrev.com/7a5a4d531fe80bfb904c736fc737ace196621e1d/chrome/android/java/src/org/chromium/chrome/browser/toolbar/CustomTabToolbar.java
[modify] https://crrev.com/7a5a4d531fe80bfb904c736fc737ace196621e1d/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarManager.java
[modify] https://crrev.com/7a5a4d531fe80bfb904c736fc737ace196621e1d/chrome/android/java_sources.gni
[modify] https://crrev.com/7a5a4d531fe80bfb904c736fc737ace196621e1d/tools/metrics/histograms/enums.xml
[modify] https://crrev.com/7a5a4d531fe80bfb904c736fc737ace196621e1d/tools/metrics/histograms/histograms.xml

Status: Fixed (was: Assigned)

Sign in to add a comment