New issue
Advanced search Search tips

Issue 726058 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Task



Sign in to add a comment

Record the percentage of main frame requests that could not be fetched from data reduction proxy due to missing/invalid config

Project Member Reported by tbansal@chromium.org, May 24 2017

Issue description

DataReductionProxy.ConfigService.HTTPRequests records the number of requests that could not be fetched due to missing/invalid client config. Separately, we should record this metric for main frame requests.
 

Comment 1 by bengr@chromium.org, May 26 2017

Status: Available (was: Untriaged)

Comment 2 by bengr@chromium.org, Nov 30 2017

Refreshed during triage.
Refreshed during triage.
Owner: tbansal@chromium.org
Status: Assigned (was: Available)
Owner: robertogden@chromium.org
Project Member

Comment 6 by bugdroid1@chromium.org, Apr 18 2018

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

commit 484d380e914ece73e5c6f946706a7e210243fc6e
Author: Robert Ogden <robertogden@chromium.org>
Date: Wed Apr 18 16:57:49 2018

Record main frame requests not handled by DRP because of config issues

Adds a new content type on the DRP resource type provider to accomplish
this.

Bug:  726058 
Change-Id: I9a5c3934c293279894a396e8d7e879e8ffe60ec5
Reviewed-on: https://chromium-review.googlesource.com/1015766
Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Commit-Queue: Robert Ogden <robertogden@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551720}
[modify] https://crrev.com/484d380e914ece73e5c6f946706a7e210243fc6e/components/data_reduction_proxy/content/browser/content_resource_type_provider.cc
[modify] https://crrev.com/484d380e914ece73e5c6f946706a7e210243fc6e/components/data_reduction_proxy/content/browser/content_resource_type_provider_unittest.cc
[modify] https://crrev.com/484d380e914ece73e5c6f946706a7e210243fc6e/components/data_reduction_proxy/core/browser/data_reduction_proxy_delegate.cc
[modify] https://crrev.com/484d380e914ece73e5c6f946706a7e210243fc6e/components/data_reduction_proxy/core/common/data_reduction_proxy_server.cc
[modify] https://crrev.com/484d380e914ece73e5c6f946706a7e210243fc6e/components/data_reduction_proxy/core/common/resource_type_provider.h
[modify] https://crrev.com/484d380e914ece73e5c6f946706a7e210243fc6e/tools/metrics/histograms/enums.xml
[modify] https://crrev.com/484d380e914ece73e5c6f946706a7e210243fc6e/tools/metrics/histograms/histograms.xml

Status: Fixed (was: Assigned)

Sign in to add a comment