New issue
Advanced search Search tips

Issue 893923 link

Starred by 1 user

Issue metadata

Status: Started
Owner:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 2
Type: Bug



Sign in to add a comment

NavigationPredictor: Understand the scenarios where navigation prediction can help

Project Member Reported by tbansal@chromium.org, Oct 10

Issue description

To speed up page loads, we have been planning to experiment with using navigation prediction to preresolve, preconnect or prefetch some of the content. However, currently, it's not really clear what are the potential benefits of doing each of these operations.

To estimate that, we should first log metrics to better understand how much each of the operations (DNS resolution, establishing connection to the server, server compute time) contribute to the slow page loads.
 
Status: Started (was: Assigned)
UKM doc is here: http://shortn/_zTLLhP8F44
Project Member

Comment 4 by bugdroid1@chromium.org, Dec 4

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/87b22dbb3eefa1cc22045f41dd8339050309ccfa

commit 87b22dbb3eefa1cc22045f41dd8339050309ccfa
Author: Tarun Bansal <tbansal@chromium.org>
Date: Tue Dec 04 05:48:48 2018

Add UKM to record socket reuse (boolean) for the main frame request

Change-Id: Ic5cb29f434b92d56277205037e1c3aa5dc54d71a
Bug: 893923
Reviewed-on: https://chromium-review.googlesource.com/c/1339721
Reviewed-by: Bryan McQuade <bmcquade@chromium.org>
Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#613442}
[modify] https://crrev.com/87b22dbb3eefa1cc22045f41dd8339050309ccfa/chrome/browser/page_load_metrics/observers/ukm_page_load_metrics_observer.cc
[modify] https://crrev.com/87b22dbb3eefa1cc22045f41dd8339050309ccfa/chrome/browser/page_load_metrics/page_load_metrics_browsertest.cc
[modify] https://crrev.com/87b22dbb3eefa1cc22045f41dd8339050309ccfa/tools/metrics/ukm/ukm.xml

Labels: Merge-Request-72 OS-Android OS-Chrome OS-Linux OS-Mac OS-Windows
Merge request for CL in #4. It's a pretty straightforward CL, and only affects metrics collection.
Project Member

Comment 6 by sheriffbot@chromium.org, Dec 6

Labels: -Merge-Request-72 Hotlist-Merge-Approved Merge-Approved-72
Your change meets the bar and is auto-approved for M72. Please go ahead and merge the CL to branch 3626 manually. Please contact milestone owner if you have questions.
Owners: govind@(Android), kariahda@(iOS), djmm@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 7 by bugdroid1@chromium.org, Dec 6

Labels: -merge-approved-72 merge-merged-3626
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7cfc796134b89493e713e282bd52be19f8371005

commit 7cfc796134b89493e713e282bd52be19f8371005
Author: Tarun Bansal <tbansal@chromium.org>
Date: Thu Dec 06 19:44:36 2018

Add UKM to record socket reuse (boolean) for the main frame request

Change-Id: Ic5cb29f434b92d56277205037e1c3aa5dc54d71a
Bug: 893923
Reviewed-on: https://chromium-review.googlesource.com/c/1339721
Reviewed-by: Bryan McQuade <bmcquade@chromium.org>
Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#613442}(cherry picked from commit 87b22dbb3eefa1cc22045f41dd8339050309ccfa)
Reviewed-on: https://chromium-review.googlesource.com/c/1366301
Cr-Commit-Position: refs/branch-heads/3626@{#122}
Cr-Branched-From: d897fb137fbaaa9355c0c93124cc048824eb1e65-refs/heads/master@{#612437}
[modify] https://crrev.com/7cfc796134b89493e713e282bd52be19f8371005/chrome/browser/page_load_metrics/observers/ukm_page_load_metrics_observer.cc
[modify] https://crrev.com/7cfc796134b89493e713e282bd52be19f8371005/chrome/browser/page_load_metrics/page_load_metrics_browsertest.cc
[modify] https://crrev.com/7cfc796134b89493e713e282bd52be19f8371005/tools/metrics/ukm/ukm.xml

Labels: Merge-Merged-72-3626
The following revision refers to this bug: 
https://chromium.googlesource.com/chromium/src.git/+/7cfc796134b89493e713e282bd52be19f8371005

Commit: 7cfc796134b89493e713e282bd52be19f8371005
Author: tbansal@chromium.org
Commiter: tbansal@chromium.org
Date: 2018-12-06 19:44:36 +0000 UTC

Add UKM to record socket reuse (boolean) for the main frame request

Change-Id: Ic5cb29f434b92d56277205037e1c3aa5dc54d71a
Bug: 893923
Reviewed-on: https://chromium-review.googlesource.com/c/1339721
Reviewed-by: Bryan McQuade <bmcquade@chromium.org>
Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#613442}(cherry picked from commit 87b22dbb3eefa1cc22045f41dd8339050309ccfa)
Reviewed-on: https://chromium-review.googlesource.com/c/1366301
Cr-Commit-Position: refs/branch-heads/3626@{#122}
Cr-Branched-From: d897fb137fbaaa9355c0c93124cc048824eb1e65-refs/heads/master@{#612437}

Sign in to add a comment