New issue
Advanced search Search tips

Issue 638773 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner: ----
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Fix Precache.DownloadedPrecacheMotivated UMA

Project Member Reported by rajendrant@chromium.org, Aug 17 2016

Issue description

Precache.DownloadedPrecacheMotivated should account for the number of new bytes added to cache in every run.

However RecordStatsForFetchInternal() assumes that all requests are due to precache when precache task is running. This is not true. Frequently sync requests are sent when precache task starts, and this can lead to overcount the Precache.DownloadedPrecacheMotivated UMA.

https://cs.chromium.org/chromium/src/components/precache/content/precache_manager.cc?rcl=0&l=265

 
Cc: wifiprefetch-reviews@google.com

Comment 2 by bengr@chromium.org, Oct 28 2016

rajendrant: please double check that this is fixed and close.
Components: Internals>Preload
Components: -Internals>Network>DataUse
Owner: ----
Status: Available (was: Assigned)

Comment 6 by bengr@chromium.org, Jun 30 2017

Status: WontFix (was: Available)

Sign in to add a comment