New issue
Advanced search Search tips

Issue 636128 link

Starred by 2 users
Status: WontFix
Owner:
alexilin@chromium.org
Closed: Sep 7
Cc:
csharrison@chromium.org
lizeb@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 699080


Sign in to add a comment

The net predictor should be split into two classes that each live on a single thread

Project Member Reported by csharrison@chromium.org, Aug 9 2016 
This cleanup has been a long time coming. The Predictor has an extremely complicated lifecycle and needs a thorough refactoring.

The UI thread portion should be a ProfileKeyedService and the IO thread portion (maybe) could be refcounted.
Project Member

Comment 1 by sheriffbot@chromium.org, Aug 10 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

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

Comment 2 by csharrison@chromium.org, Aug 10 2017

Cc: lizeb@chromium.org alexilin@chromium.org
Labels: -Hotlist-Recharge-Cold
Status: Available (was: Untriaged)

Comment 3 by alexilin@chromium.org, Aug 10 2017

Blockedon: 699080
Cc: -alexilin@chromium.org
Owner: alexilin@chromium.org
Status: Started (was: Available)
New LoadingPredictor has exactly this architecture. (LoadingPredictor is a KeyedService that lives in the UI thread and owns PreconnectManager that lives in the IO thread).
We want to run some experiments before replacing net predictor with the new one. Follow the status in  bug 699080 .

Comment 4 by alexilin@chromium.org, Sep 7

Status: WontFix (was: Started)
The net predictor has been removed.
►

Sign in to add a comment