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

Issue 709910 link

Starred by 1 user
Status: Fixed
Owner:
yhirano@chromium.org
Closed: Apr 2017
Cc:
kinuko@chromium.org
█ tyoshino@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Add a histograms for DetachableResourceHandler

Project Member Reported by yhirano@chromium.org, Apr 10 2017 
Add histograms in DetachableResourceHandler to see

 - the distribution of the time to load
 - the loading result
.


See https://docs.google.com/a/chromium.org/document/d/1iHJtFa3jOo5n9QXHb6Ok5nK8kavXSk2DrLoubPWi9ys/edit?disco=AAAAA-fwrVk.
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 18 2017 

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

commit 56866c20d1252bb4ad9a6a0ed51a14fa73b7632f
Author: yhirano <yhirano@chromium.org>
Date: Tue Apr 18 07:35:27 2017

Add a UMA histogram for measuring DetachableResourceHandler duration

BUG= 709910 

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

[modify] https://crrev.com/56866c20d1252bb4ad9a6a0ed51a14fa73b7632f/content/browser/loader/detachable_resource_handler.cc
[modify] https://crrev.com/56866c20d1252bb4ad9a6a0ed51a14fa73b7632f/tools/metrics/histograms/histograms.xml

Comment 2 by yhirano@chromium.org, Apr 18 2017

Status: Fixed (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Nov 13 2017 

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

commit 4b2e85648760081bed3b6df83f9f272b7007dcf0
Author: Yutaka Hirano <yhirano@chromium.org>
Date: Mon Nov 13 12:12:30 2017

Remove a UMA histogram for measuring DetachableResourceHandler duration

Bug:  709910 ,759934
Change-Id: I265d4c376a9fb68e577c1b1412564a614b48fc91
Reviewed-on: https://chromium-review.googlesource.com/757988
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Commit-Queue: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515930}
[modify] https://crrev.com/4b2e85648760081bed3b6df83f9f272b7007dcf0/content/browser/loader/detachable_resource_handler.cc
[modify] https://crrev.com/4b2e85648760081bed3b6df83f9f272b7007dcf0/tools/metrics/histograms/histograms.xml
►

Sign in to add a comment