0

Add a metrics to show if NavigationURLLoader is the main path

This is a follow up of:
https://chromium-review.googlesource.com/c/chromium/src/+/6301690/comment/7e850da7_03f20d85/

To understand the ratio of cases that need to send the IPC to the
network service, let me add a metrics.

Bug: 394974691
Change-Id: I66faffea649461bb79cb8b5ddc92c20218125e52
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6304405
Reviewed-by: Minoru Chikamune <chikamune@chromium.org>
Reviewed-by: Takashi Toyoshima <toyoshim@chromium.org>
Commit-Queue: Yoshisato Yanagisawa <yyanagisawa@chromium.org>
Auto-Submit: Yoshisato Yanagisawa <yyanagisawa@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1426941}
This commit is contained in:
Yoshisato Yanagisawa
2025-03-02 21:22:32 -08:00
committed by Chromium LUCI CQ
parent 3ff43726a6
commit 64ec936b04
2 changed files with 14 additions and 0 deletions
content/browser/loader
tools/metrics/histograms/metadata/navigation

@@ -1518,6 +1518,8 @@ void NavigationURLLoaderImpl::ParseHeaders(
// - Network
// - ServiceWorker
// - WebUI
base::UmaHistogramBoolean("Navigation.URLLoader.InMainPath",
static_cast<bool>(head->parsed_headers));
if (head->parsed_headers) {
#ifndef NDEBUG
// In debug mode, force reparsing the headers and check that they match.

@@ -2629,6 +2629,18 @@ chromium-metrics-reviews@google.com.
</summary>
</histogram>
<histogram name="Navigation.URLLoader.InMainPath" enum="Boolean"
expires_after="2025-09-25">
<owner>yyanagisawa@chromium.org</owner>
<owner>chrome-loading@google.com</owner>
<summary>
Records NavigationURLLoader is executed as the main path or not.
This is recorded on each main resource load on navigation. Multiple events
will be recorded for redirect cases.
</summary>
</histogram>
<histogram name="Navigation.URLLoader.ParseHeaders.{Case}" units="microseconds"
expires_after="2025-09-25">
<owner>yyanagisawa@chromium.org</owner>