Add DataReductionProxy UMA on page size usage |
||||
Issue descriptionUMA should be added to Page_load_metrics (data_reduction_proxy observer) to track the per-page data usage of data reduction proxy along with information about compression of the page.
,
Jan 18 2017
Thanks, this is in reviews, and can hopefully land before branch or merge back shortly after.
,
Jan 19 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/485df9ed214f7924eaf8ed067070146e3d12aa58 commit 485df9ed214f7924eaf8ed067070146e3d12aa58 Author: ryansturm <ryansturm@chromium.org> Date: Thu Jan 19 22:17:24 2017 Adding UMA for data reduction proxy page load size/savings This Adds UMA to track bytes savings, percent savings, total requests, percent requests throug the proxy, byte usage, etc. This UMA will be compared against existing PLM data as well as sliced by various experiments in data reduction proxy. BUG= 682022 Review-Url: https://codereview.chromium.org/2642553002 Cr-Commit-Position: refs/heads/master@{#444859} [modify] https://crrev.com/485df9ed214f7924eaf8ed067070146e3d12aa58/chrome/browser/page_load_metrics/observers/data_reduction_proxy_metrics_observer.cc [modify] https://crrev.com/485df9ed214f7924eaf8ed067070146e3d12aa58/chrome/browser/page_load_metrics/observers/data_reduction_proxy_metrics_observer.h [modify] https://crrev.com/485df9ed214f7924eaf8ed067070146e3d12aa58/chrome/browser/page_load_metrics/observers/data_reduction_proxy_metrics_observer_unittest.cc [modify] https://crrev.com/485df9ed214f7924eaf8ed067070146e3d12aa58/tools/metrics/histograms/histograms.xml
,
Jan 19 2017
,
Jan 19 2017
[Auto-generated comment by a script] We noticed that this issue is targeted for M-57; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-57 label, otherwise remove Merge-TBD label. Thanks.
,
Mar 3 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by ajha@chromium.org
, Jan 18 2017