Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5076)

Unified Diff: chrome/browser/search/search.h

Issue 2211983002: Remove search::IsQueryExtractionEnabled (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@remove_prefetch_non_default
Patch Set: remove more tests Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/search/search.h
diff --git a/chrome/browser/search/search.h b/chrome/browser/search/search.h
index bd8a27549ca375558770a486c8ad335799d3ff2b..b32da509da92f7af78201fb0927a042239c1695d 100644
--- a/chrome/browser/search/search.h
+++ b/chrome/browser/search/search.h
@@ -39,10 +39,9 @@ enum CacheableNTPLoad {
// Returns whether the suggest is enabled for the given |profile|.
bool IsSuggestPrefEnabled(Profile* profile);
-// Extracts and returns search terms from |url|. Does not consider
-// IsQueryExtractionEnabled() and Instant support state of the page and does
-// not check for a privileged process, so most callers should use
-// GetSearchTerms() below instead.
+// Extracts and returns search terms from |url|. Does not consider Instant
+// support state of the page and does not check for a privileged process, so
+// most callers should use GetSearchTerms() below instead.
base::string16 ExtractSearchTermsFromURL(Profile* profile, const GURL& url);
// Returns true if it is okay to extract search terms from |url|. |url| must
@@ -51,9 +50,8 @@ base::string16 ExtractSearchTermsFromURL(Profile* profile, const GURL& url);
bool IsQueryExtractionAllowedForURL(Profile* profile, const GURL& url);
// Returns the search terms attached to a specific NavigationEntry, or empty
-// string otherwise. Does not consider IsQueryExtractionEnabled() and does not
-// check Instant support, so most callers should use GetSearchTerms() below
-// instead.
+// string otherwise. Does not check Instant support, so most callers should use
+// GetSearchTerms() below instead.
base::string16 GetSearchTermsFromNavigationEntry(
const content::NavigationEntry* entry);

Powered by Google App Engine
This is Rietveld 408576698