Commit 54bb0e0d authored by Donn Denman's avatar Donn Denman Committed by Commit Bot

[TTS] Fix privacy workarounds for Related Searches

We cut some corners on privacy that were helpful for the early prototype
of Related Searches. Now those have been removed.

BUG=1064141

Change-Id: I193c60f63d62e915e3dfa28e596fbb4ad6fc8d6a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2365213
Auto-Submit: Donn Denman <donnd@chromium.org>
Commit-Queue: Donn Denman <donnd@chromium.org>
Commit-Queue: Theresa  <twellington@chromium.org>
Reviewed-by: default avatarTheresa  <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#800158}
parent f3b221b0
......@@ -327,9 +327,6 @@ class ContextualSearchPolicy {
* to see if all privacy-related conditions are met to send the base page URL.
*/
boolean maySendBasePageUrl() {
// TODO(donnd): revisit for related searches privacy review. https://crbug.com/1064141.
if (isRelatedSearchesEnabled()) return true;
return !isUserUndecided();
}
......
......@@ -403,12 +403,6 @@ bool ContextualSearchDelegate::CanSendPageURL(
if (field_trial_->IsSendBasePageURLDisabled())
return false;
// TODO(donnd): privacy review needed before launch.
// See https://crbug.com/1064141.
if (base::FeatureList::IsEnabled(chrome::android::kRelatedSearches)) {
return true;
}
// Ensure that the default search provider is Google.
const TemplateURL* default_search_provider =
template_url_service->GetDefaultSearchProvider();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment