Issue metadata
Sign in to add a comment
|
Default search engine should be in list of all sites. |
||||||||||||||||||||||||
Issue descriptionAt the moment it isn't but as it has geolocation by default it should be listed.
,
Jun 19 2017
Hmm yes. Emily, any comments about just adding the dse in these two places?
,
Jun 21 2017
SGTM
,
Aug 1 2017
Punting to Tim
,
Aug 10 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ed44c4013f117b63bba25b8c1af3500120c06228 commit ed44c4013f117b63bba25b8c1af3500120c06228 Author: Timothy Loh <timloh@chromium.org> Date: Thu Aug 10 05:28:06 2017 Ensure the DSE is in the list of all sites / sites with location permissions Bug: 734469 Change-Id: I4a29e439697ceac725b23354fd4dee38a03f5192 Reviewed-on: https://chromium-review.googlesource.com/604774 Reviewed-by: Ben Wells <benwells@chromium.org> Reviewed-by: Bernhard Bauer <bauerb@chromium.org> Commit-Queue: Timothy Loh <timloh@chromium.org> Cr-Commit-Position: refs/heads/master@{#493270} [modify] https://crrev.com/ed44c4013f117b63bba25b8c1af3500120c06228/chrome/browser/android/preferences/website_preference_bridge.cc [modify] https://crrev.com/ed44c4013f117b63bba25b8c1af3500120c06228/chrome/browser/android/search_geolocation/search_geolocation_service.cc [modify] https://crrev.com/ed44c4013f117b63bba25b8c1af3500120c06228/chrome/browser/android/search_geolocation/search_geolocation_service.h
,
Aug 16 2017
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by dullweber@chromium.org
, Jun 19 2017