Add PrewarmPrerenderCoverageStatus metric and deprecate legacy metrics [chromium/src : main]

0 views
Skip to first unread message

Jiacheng Guo (Gerrit)

unread,
7:43 AM (4 hours ago) 7:43 AM
to prerendering-reviews, chromium...@chromium.org, Chromium Metrics Reviews, asvitkine...@chromium.org, bmcquad...@chromium.org, lingqi...@chromium.org, csharris...@chromium.org, speed-metrics...@chromium.org, gavin...@chromium.org, loading-rev...@chromium.org, tburkar...@chromium.org, speed-metr...@chromium.org

Jiacheng Guo has uploaded the change for review

Commit message

Add PrewarmPrerenderCoverageStatus metric and deprecate legacy metrics

Introduce the SearchPrewarmPrerenderCoverageStatus enum and UMA metric
to track the full funnel of prewarm and prerender coverage on Google
Search navigations. Deprecate legacy metrics PrewarmNavigationStatus2
and HadPriorPrewarmCommitStatus2.
Bug: 473593133
Change-Id: I121709fc076c96c560eb33c4d75fcba47bf5503c

Change diff


Change information

Files:
  • M chrome/browser/page_load_metrics/observers/gws_page_load_metrics_observer_browsertest.cc
  • M chrome/browser/page_load_metrics/observers/gws_page_load_metrics_observer_unittest.cc
  • M chrome/browser/preloading/prerender/dse_prewarm_navigation_throttle_browsertest.cc
  • M chrome/browser/preloading/prerender/prerender_manager.cc
  • M chrome/browser/preloading/prerender/search_preload_progress_service.cc
  • M chrome/browser/preloading/prerender/search_preload_progress_service.h
  • M chrome/browser/preloading/prerender/search_preload_progress_service_unittest.cc
  • M components/page_load_metrics/google/browser/BUILD.gn
  • M components/page_load_metrics/google/browser/gws_page_load_metrics_observer.cc
  • M components/page_load_metrics/google/browser/gws_page_load_metrics_observer.h
  • A components/page_load_metrics/google/browser/search_prewarm_types.cc
  • A components/page_load_metrics/google/browser/search_prewarm_types.h
  • M tools/metrics/histograms/metadata/page/enums.xml
  • M tools/metrics/histograms/metadata/page/histograms.xml
Change size: L
Delta: 14 files changed, 743 insertions(+), 92 deletions(-)
Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement is not satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: newchange
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I121709fc076c96c560eb33c4d75fcba47bf5503c
Gerrit-Change-Number: 8260261
Gerrit-PatchSet: 1
Gerrit-Owner: Jiacheng Guo <g...@google.com>
Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
Gerrit-CC: prerendering-reviews <prerenderi...@chromium.org>
satisfied_requirement
unsatisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages