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

Issue 684561 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Add histograms for time to parse start for service worker controlled pages.

Project Member Reported by bmcquade@chromium.org, Jan 24 2017

Issue description

Add histograms for time to parse start for service worker controlled pages.

Service worker controlled pages should drive time to parse start down to near
zero. We should track this latency.

Additionally, this will allow us to improve the service worker stats UMA, where
we currently compute the percentage of page loads controlled by SW using UMA
sourced from different parts of the code in numerator and denominator.
 
Cc: falken@chromium.org kenjibaheux@chromium.org

Comment 2 by falken@chromium.org, Jan 25 2017

Components: Blink>ServiceWorker
Status: Fixed (was: Started)
Project Member

Comment 4 by bugdroid1@chromium.org, Jan 27 2017

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

commit b9c7574d2f2d7baaf78d42c5db8068fed0081145
Author: bmcquade <bmcquade@chromium.org>
Date: Fri Jan 27 23:51:08 2017

Add histograms for time to parse start for service worker controlled pages.

Service worker controlled pages should drive time to parse start down to near
zero. This change allows us to track the actual latency.

Additionally, this will allow us to improve the service worker stats UMA, where
we currently compute the percentage of page loads controlled by SW using UMA
sourced from different parts of the code in numerator and denominator.

BUG= 684561 

Review-Url: https://codereview.chromium.org/2651883006
Cr-Commit-Position: refs/heads/master@{#446838}

[modify] https://crrev.com/b9c7574d2f2d7baaf78d42c5db8068fed0081145/chrome/browser/page_load_metrics/observers/service_worker_page_load_metrics_observer.cc
[modify] https://crrev.com/b9c7574d2f2d7baaf78d42c5db8068fed0081145/chrome/browser/page_load_metrics/observers/service_worker_page_load_metrics_observer.h
[modify] https://crrev.com/b9c7574d2f2d7baaf78d42c5db8068fed0081145/chrome/browser/page_load_metrics/observers/service_worker_page_load_metrics_observer_unittest.cc
[modify] https://crrev.com/b9c7574d2f2d7baaf78d42c5db8068fed0081145/tools/metrics/histograms/histograms.xml

Sign in to add a comment