New issue
Advanced search Search tips

Issue 762406 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Sep 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

TemplateURL::IsSearchURL does not detect yandex search urls

Project Member Reported by a-...@yandex-team.ru, Sep 6 2017

Issue description

Chrome Version: 63.0.3206.0 
OS: Any

TemplateURL::IsSearchURL function  does not detect yandex search urls.
It was broken in https://chromium.googlesource.com/chromium/src/+/0ea1107bf2c1a0532d1c10a3666cafb9047abbb6
 
Status: Started (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Sep 7 2017

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

commit 5a475353d7864bb47346b1ab8975c68454cbdf1f
Author: Alexander Yashkin <a-v-y@yandex-team.ru>
Date: Thu Sep 07 10:00:55 2017

Fixed yandex search url detection

TemplateURL::IsSearchURL function does not detect yandex search urls.
It was broken in https://chromium.googlesource.com/chromium/src/+/
0ea1107bf2c1a0532d1c10a3666cafb9047abbb6.

Bug:  762406 
Change-Id: I8785f72c77cc4fce770de19c084e05080be51433
Reviewed-on: https://chromium-review.googlesource.com/651415
Commit-Queue: Alexander Yashkin <a-v-y@yandex-team.ru>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#500262}
[modify] https://crrev.com/5a475353d7864bb47346b1ab8975c68454cbdf1f/components/search_engines/template_url.cc
[modify] https://crrev.com/5a475353d7864bb47346b1ab8975c68454cbdf1f/components/search_engines/template_url_prepopulate_data_unittest.cc

Status: Verified (was: Started)

Sign in to add a comment