New issue
Advanced search Search tips

Issue 859116 link

Starred by 2 users
Status: Fixed
Owner:
cthomp@chromium.org
Closed: Jul 16
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac , Fuchsia
Pri: 2
Type: Feature
Team-Security-UX



Sign in to add a comment

Add metrics for how long PageInfo is open

Project Member Reported by cthomp@chromium.org, Jun 29 2018 
Per a suggestion of martinshelton@, it would be nice to have metrics for how long the PageInfo bubble was open, and whether an action was taken while it was open.

I have an initial CL out for review now: https://crrev.com/c/1096336

Ideally, we'd like to get this into M68 so we can have some initial metrics before more significant UI changes become default in M69.
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 11 

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

commit 0c4e06c2d21cc9a69f0a2a871a873b30c12b8fee
Author: Christopher Thompson <cthomp@chromium.org>
Date: Wed Jul 11 22:29:59 2018

Add metrics tracking how long Page Info is open

Per a suggestion of martinshelton@, this adds a new set of histograms to
track how long users have the Page Info bubble open. This will help us
better understand users who open Page Info but take no actions in it
(e.g., are these opens likely to be more accidental or incidental).

This metric is split by the security level of the page to allow us to
see differences between different security indicators. Two sub-histograms
("Action" and "NoAction") further split up the metric by whether the user
performed an action in the Page Info bubble before closing it (including
actions that cause the bubble to close).

Bug:  859116 
Change-Id: I630a1f37ecff13bc7ee44b3e8e5a5c34337c9267
Reviewed-on: https://chromium-review.googlesource.com/1096336
Commit-Queue: Christopher Thompson <cthomp@chromium.org>
Reviewed-by: Steven Holte <holte@chromium.org>
Reviewed-by: Emily Stark <estark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#574381}
[modify] https://crrev.com/0c4e06c2d21cc9a69f0a2a871a873b30c12b8fee/chrome/browser/ui/page_info/page_info.cc
[modify] https://crrev.com/0c4e06c2d21cc9a69f0a2a871a873b30c12b8fee/chrome/browser/ui/page_info/page_info.h
[modify] https://crrev.com/0c4e06c2d21cc9a69f0a2a871a873b30c12b8fee/chrome/browser/ui/page_info/page_info_unittest.cc
[modify] https://crrev.com/0c4e06c2d21cc9a69f0a2a871a873b30c12b8fee/tools/metrics/histograms/histograms.xml

Comment 2 by cthomp@chromium.org, Jul 16

Status: Fixed (was: Started)
The metrics are now showing up in the UMA dashboard, so marking this as fixed. 

 Issue 858912  which was preventing these metrics from being correctly recorded on Android is now fixed. It does not seem worth trying to merge both back into M68, so the current plan will be to look at these metrics starting in M69.
►

Sign in to add a comment