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

Issue 813258 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Add UMA to measure Data Saver savings

Project Member Reported by bengr@chromium.org, Feb 17 2018

Issue description

We should record the percentage data savings at the start of each browsing session. This should be straightforward to write, modulo two considerations:

First, we should only record savings for users who have browsed a reasonably amount. For example, we could limit this to users that have fetched at least 10MB of page data, or have navigated to at least 10 pages, or who have had Data Saver enabled for at least 10 days. I think I have a slight preference for the constraints in that order.

Second, we should do something special to record negative savings. That could be a second histogram, and the first histogram would report 0% for any session that had negative savings. Note crbug.com/812910 is related.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Feb 28 2018

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

commit 488b299a00c8417d0741d92e01f084fd8cc8bdba
Author: rajendrant <rajendrant@chromium.org>
Date: Wed Feb 28 18:05:29 2018

Record data savings percent at startup

Records the data savings percentage at Chrome startup for data saver enabled users
who have browsed at least 10MB. Positive and negative savings are recorded in
separate histograms.

Bug:  813258 
Change-Id: If764b59e72583aa495fcb604c771540aa1785287
Reviewed-on: https://chromium-review.googlesource.com/935665
Commit-Queue: rajendrant <rajendrant@chromium.org>
Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539880}
[modify] https://crrev.com/488b299a00c8417d0741d92e01f084fd8cc8bdba/components/data_reduction_proxy/core/browser/data_reduction_proxy_settings.cc
[modify] https://crrev.com/488b299a00c8417d0741d92e01f084fd8cc8bdba/components/data_reduction_proxy/core/browser/data_reduction_proxy_settings.h
[modify] https://crrev.com/488b299a00c8417d0741d92e01f084fd8cc8bdba/tools/metrics/histograms/histograms.xml

Status: Fixed (was: Assigned)

Sign in to add a comment