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

Unified Diff: components/search_engines/template_url.h

Issue 2479113002: Make extensions DSE persistent in browser prefs (Closed)
Patch Set: Fixed default extension keywords conflicts problem Created 3 years, 11 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
« no previous file with comments | « components/search_engines/search_engines_test_util.cc ('k') | components/search_engines/template_url.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/search_engines/template_url.h
diff --git a/components/search_engines/template_url.h b/components/search_engines/template_url.h
index af5f71f24db4953e6e74fe14b6ec34df0afe8c6b..8a8fd94ee2c24c32b24e4e21cc8ac4f3e323ab0f 100644
--- a/components/search_engines/template_url.h
+++ b/components/search_engines/template_url.h
@@ -512,10 +512,6 @@ class TemplateURL {
// Whether the search engine is supposed to be default.
bool wants_to_be_default_engine;
-
- // Used to resolve conflicts when there are multiple extensions specifying
- // the default search engine. The most recently-installed wins.
- base::Time install_time;
};
explicit TemplateURL(const TemplateURLData& data, Type type = NORMAL);
@@ -535,6 +531,12 @@ class TemplateURL {
const TemplateURLData* data,
const SearchTermsData& search_terms_data);
+ // Special case for search_terms_replacement_key comparison, because of
+ // its special initialization in TemplateURL constructor.
+ static bool SearchTermsReplacementKeysMatch(
+ const std::string& search_terms_replacement_key1,
+ const std::string& search_terms_replacement_key2);
+
const TemplateURLData& data() const { return data_; }
const base::string16& short_name() const { return data_.short_name(); }
@@ -617,6 +619,10 @@ class TemplateURL {
extension_info_ = std::move(extension_info);
}
+ const AssociatedExtensionInfo* GetExtensionInfoForTesting() const {
+ return extension_info_.get();
+ }
+
// Returns true if |url| supports replacement.
bool SupportsReplacement(const SearchTermsData& search_terms_data) const;
« no previous file with comments | « components/search_engines/search_engines_test_util.cc ('k') | components/search_engines/template_url.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698