New issue
Advanced search Search tips

Issue 899098 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 13
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug

Blocking:
issue 850624



Sign in to add a comment

NavigationPredictor: Compute navigation score based on whether the anchor element is same host or not

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

Issue description

The navigation prediction score currently does not take into account if the host of the target anchor element is same host as the source document or a different host. We should take that into account when computing the navigation score.
 
Blocking: 850624
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 8

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

commit 4926f67876d23d563314970c6cfc1ac52e210bce
Author: Tarun Bansal <tbansal@chromium.org>
Date: Thu Nov 08 07:35:40 2018

Determine navigation score based on host of the target element

For default search engine pages, give higher weight to cross-origin
anchor elements.
For non-default search engine pages, give higher weight to same-origin
anchor elements.

Change-Id: I6de3037d9b28dbb770032255f8f22a7c2e6342a6
Bug:  899098 
Reviewed-on: https://chromium-review.googlesource.com/c/1300594
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#606375}
[modify] https://crrev.com/4926f67876d23d563314970c6cfc1ac52e210bce/chrome/browser/navigation_predictor/navigation_predictor.cc
[modify] https://crrev.com/4926f67876d23d563314970c6cfc1ac52e210bce/chrome/browser/navigation_predictor/navigation_predictor.h
[modify] https://crrev.com/4926f67876d23d563314970c6cfc1ac52e210bce/chrome/browser/navigation_predictor/navigation_predictor_browsertest.cc
[modify] https://crrev.com/4926f67876d23d563314970c6cfc1ac52e210bce/testing/variations/fieldtrial_testing_config.json
[modify] https://crrev.com/4926f67876d23d563314970c6cfc1ac52e210bce/tools/metrics/histograms/enums.xml
[modify] https://crrev.com/4926f67876d23d563314970c6cfc1ac52e210bce/tools/metrics/histograms/histograms.xml

Status: Fixed (was: Started)

Sign in to add a comment