Commit eec38c27 authored by Alex Moshchuk's avatar Alex Moshchuk Committed by Commit Bot

Add IsLockedProcess histograms for HTTP(S) navigations only.

Bug: 1004371
Change-Id: I62c91186f046b6701c8fdd1fded90f38d45188c5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1817286Reviewed-by: default avatarIlya Sherman <isherman@chromium.org>
Reviewed-by: default avatarŁukasz Anforowicz <lukasza@chromium.org>
Commit-Queue: Alex Moshchuk <alexmos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#699054}
parent b40b6611
......@@ -469,11 +469,20 @@ void RecordReadyToCommitMetrics(
UMA_HISTOGRAM_BOOLEAN("Navigation.RequiresDedicatedProcess",
new_rfh->GetSiteInstance()->RequiresDedicatedProcess());
if (common_params.url.SchemeIsHTTPOrHTTPS()) {
UMA_HISTOGRAM_BOOLEAN(
"Navigation.RequiresDedicatedProcess.HTTPOrHTTPS",
new_rfh->GetSiteInstance()->RequiresDedicatedProcess());
}
ChildProcessSecurityPolicyImpl* policy =
ChildProcessSecurityPolicyImpl::GetInstance();
GURL process_lock = policy->GetOriginLock(new_rfh->GetProcess()->GetID());
UMA_HISTOGRAM_BOOLEAN("Navigation.IsLockedProcess", !process_lock.is_empty());
if (common_params.url.SchemeIsHTTPOrHTTPS()) {
UMA_HISTOGRAM_BOOLEAN("Navigation.IsLockedProcess.HTTPOrHTTPS",
!process_lock.is_empty());
}
if (common_params.transition & ui::PAGE_TRANSITION_FORWARD_BACK) {
UMA_HISTOGRAM_BOOLEAN("Navigation.IsSameProcess.BackForward",
......
......@@ -71436,6 +71436,17 @@ uploading your change for review.
</summary>
</histogram>
<histogram name="Navigation.IsLockedProcess.HTTPOrHTTPS"
enum="NavigationIsLockedProcess" expires_after="2020-12-31">
<owner>alexmos@chromium.org</owner>
<owner>lukasza@chromium.org</owner>
<summary>
Whether the navigation commits in a process that is locked to an origin.
Logged at ready-to-commit time for every navigation that commits with a HTTP
or HTTPS URL scheme, excluding same-document navigations and errors.
</summary>
</histogram>
<histogram name="Navigation.IsMobileOptimized" enum="BooleanIsMobileOptimized"
expires_after="2018-03-05">
<obsolete>
......@@ -71708,6 +71719,18 @@ uploading your change for review.
</summary>
</histogram>
<histogram name="Navigation.RequiresDedicatedProcess.HTTPOrHTTPS"
enum="NavigationRequiresDedicatedProcess" expires_after="2020-12-31">
<owner>alexmos@chromium.org</owner>
<owner>lukasza@chromium.org</owner>
<summary>
Whether the navigation commits in a SiteInstance that requires a dedicated
process. Logged at ready-to-commit time for every navigation that commits
with a HTTP or HTTPS URL scheme, excluding same-document navigations and
errors.
</summary>
</histogram>
<histogram
name="Navigation.ResourceHandler.ProceedWithResponseUntilFirstReadCompleted"
units="ms">
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment