New issue
Advanced search Search tips

Issue 841820 link

Starred by 3 users

Issue metadata

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

Blocked on:
issue 721403



Sign in to add a comment

Remove PageLoadMetrics non network service path

Project Member Reported by jcivelli@chromium.org, May 10 2018

Issue description

The page load metrics code as 2 code paths:
- the non network service one, that relies on the ResourceDispatcherHost (see chrome_resource_dispatcher_host_delegate.cc NotifyUIThreadOfRequestComplete)
- the network service one, that used the WebContentsObserver notifications (see MetricsWebContentsObserver::ResourceLoadComplete)


We can't switch to the network service one yet as we still need the data reduction proxy data which is not available yet with the network service.

Once it is available, we should hook it in in MetricsWebContentsObserver::ResourceLoadComplete and removed the non network service code path.

 

Comment 1 by dxie@chromium.org, May 14 2018

Labels: -Pri-3 Proj-Servicification-Canary OS-Android OS-Chrome OS-Linux OS-Mac OS-Windows Pri-1
Status: Started (was: Untriaged)
*triaged* - jcivelli is working on it.

Comment 2 by dxie@google.com, Jun 29 2018

Labels: -Proj-Servicification-Canary Proj-Servicification
Owner: ----
Status: Available (was: Started)
Labels: Hotlist-KnownIssue

Sign in to add a comment