New issue
Advanced search Search tips

Issue 617688 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Mar 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

The predictor uses inconsistent url canonicalization

Project Member Reported by csharrison@chromium.org, Jun 6 2016

Issue description

There are places in the net predictor that use different types of canonicalization.

The two most common ones are url.GetWithEmptyPath() and Predictor::CanonicalizeUrl(url).

Sometimes urls canonicalized in two different ways are compared, which is non-ideal.

The predictor should be updated to be consistent, and use only one canonicalization scheme.
 
Status: WontFix (was: Started)
The predictor is being replaced

Sign in to add a comment