Add detailed tracking for causes of invalid PageLoadTimings

Desktop / Chromium - bmcquade [chromium.org] - 17 April 2017 22:34 EDT

In https://codereview.chromium.org/2804093002 we added additional error codes to understand the root cause of ERR_BAD_TIMING_IPC. New data shows that ERR_BAD_TIMING_IPC_INVALID_TIMING is the primary cause.

We know we are receiving invalid PageLoadTiming structs, but we don't yet know the specific cause. This change adds more detailed error codes to get insight here.

BUG=712033

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

0c35320 Add detailed tracking for causes of invalid PageLoadTimings.
.../metrics_web_contents_observer_unittest.cc | 81 ++++++++++++++++
.../page_load_metrics_browsertest.cc | 8 ++
.../browser/page_load_metrics/page_load_tracker.cc | 107 +++++++++++----------
.../browser/page_load_metrics/page_load_tracker.h | 43 +++++++++
tools/metrics/histograms/histograms.xml | 33 +++++++
5 files changed, 221 insertions(+), 51 deletions(-)

Upstream: git.chromium.org


  • Share