New issue
Advanced search Search tips

Issue 638827 link

Starred by 2 users
Status: Fixed
Owner:
horo@chromium.org
Closed: Mar 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 616502


Sign in to add a comment

Remove ServiceWorker.NavigationHintLatency UMAs when "speculatively start Service Workers" experimentation finished.

Project Member Reported by horo@chromium.org, Aug 18 2016 
https://codereview.chromium.org/2211783003/ will add NavigationHintLatency UMAs.
- Event.Latency.TouchToFirstScrollUpdateSwapBegin.IsRunningNavigationHintTask
- Event.Latency.TouchToScrollUpdateSwapBegin.IsRunningNavigationHintTask

These UMAs are only for the experimentation.
So we will remove them after the experimentation finished.

Comment 1 by horo@chromium.org, Aug 18 2016

Labels: M-56
https://codereview.chromium.org/2211783003/ will land in M54.

I will remove the UMAs when M54 goes stable.
So setting M56 for this issue.

Comment 2 by horo@chromium.org, Aug 18 2016

Blockedon: 616502
Project Member

Comment 3 by bugdroid1@chromium.org, Aug 19 2016 

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

commit f5d309f062b41c29a2adbcc469a967b95f61dca3
Author: horo <horo@chromium.org>
Date: Fri Aug 19 00:47:07 2016

Track input latency while starting a ServiceWorker for a navigation hint.

This CL introduces these new UMAs:
- Event.Latency.TouchToFirstScrollUpdateSwapBegin.IsRunningNavigationHintTask
- Event.Latency.TouchToScrollUpdateSwapBegin.IsRunningNavigationHintTask

BUG= 616502 ,  638827 

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

[modify] https://crrev.com/f5d309f062b41c29a2adbcc469a967b95f61dca3/content/browser/renderer_host/input/render_widget_host_latency_tracker.cc
[modify] https://crrev.com/f5d309f062b41c29a2adbcc469a967b95f61dca3/content/browser/renderer_host/input/render_widget_host_latency_tracker.h
[modify] https://crrev.com/f5d309f062b41c29a2adbcc469a967b95f61dca3/content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc
[modify] https://crrev.com/f5d309f062b41c29a2adbcc469a967b95f61dca3/content/browser/renderer_host/render_widget_host_impl.cc
[modify] https://crrev.com/f5d309f062b41c29a2adbcc469a967b95f61dca3/content/browser/service_worker/service_worker_context_wrapper.cc
[modify] https://crrev.com/f5d309f062b41c29a2adbcc469a967b95f61dca3/content/browser/service_worker/service_worker_context_wrapper.h
[modify] https://crrev.com/f5d309f062b41c29a2adbcc469a967b95f61dca3/tools/metrics/histograms/histograms.xml
Project Member

Comment 4 by bugdroid1@chromium.org, Mar 29 2017 

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

commit a9acfbde04faff434f737608a7f6632d6e4b69e8
Author: horo <horo@chromium.org>
Date: Wed Mar 29 06:22:11 2017

Revert "Track input latency while starting a ServiceWorker for a navigation hint."

The NavigationHint experiment finished.

This CL reverts commit
 https://crrev.com/f5d309f062b41c29a2adbcc469a967b95f61dca3
 https://codereview.chromium.org/2211783003

BUG= 616502 ,  638827 

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

[modify] https://crrev.com/a9acfbde04faff434f737608a7f6632d6e4b69e8/content/browser/renderer_host/input/render_widget_host_latency_tracker.cc
[modify] https://crrev.com/a9acfbde04faff434f737608a7f6632d6e4b69e8/content/browser/renderer_host/input/render_widget_host_latency_tracker.h
[modify] https://crrev.com/a9acfbde04faff434f737608a7f6632d6e4b69e8/content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc
[modify] https://crrev.com/a9acfbde04faff434f737608a7f6632d6e4b69e8/content/browser/renderer_host/render_widget_host_impl.cc
[modify] https://crrev.com/a9acfbde04faff434f737608a7f6632d6e4b69e8/content/browser/service_worker/service_worker_context_wrapper.cc
[modify] https://crrev.com/a9acfbde04faff434f737608a7f6632d6e4b69e8/content/browser/service_worker/service_worker_context_wrapper.h
[modify] https://crrev.com/a9acfbde04faff434f737608a7f6632d6e4b69e8/tools/metrics/histograms/histograms.xml

Comment 5 by horo@chromium.org, Mar 30 2017

Status: Fixed (was: Assigned)
►

Sign in to add a comment