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

Issue 741924 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Task



Sign in to add a comment

Add UMA for priority keyed HttpJob completion times.

Project Member Reported by devdeepray@chromium.org, Jul 12 2017

Issue description

Net.HttpJob.TotalTimeSuccess is not split by request priorities. Doing this would make it easier to understand the effect of changing the behavior of the resource scheduler on requests of different priorities.
 

Comment 1 by benhenry@google.com, Jul 13 2017

Components: Speed>Metrics
Labels: -Performance Performance-Network
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 14 2017

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

commit 0c6966dfc39175050e55d5ffb90a885650bdb45d
Author: Devdeep Ray <devdeepray@chromium.org>
Date: Fri Jul 14 00:58:22 2017

Add UMA to record priority keyed completion times of requests

Bug:  741924 
Change-Id: Ifd3355657a30a0e40d73517fbf79dca41d992512
Reviewed-on: https://chromium-review.googlesource.com/568716
Commit-Queue: Devdeep Ray <devdeepray@chromium.org>
Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#486563}
[modify] https://crrev.com/0c6966dfc39175050e55d5ffb90a885650bdb45d/net/url_request/url_request_http_job.cc
[modify] https://crrev.com/0c6966dfc39175050e55d5ffb90a885650bdb45d/net/url_request/url_request_http_job_unittest.cc
[modify] https://crrev.com/0c6966dfc39175050e55d5ffb90a885650bdb45d/tools/metrics/histograms/histograms.xml

Status: Fixed (was: Started)

Sign in to add a comment