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

Unified Diff: components/search_engines/template_url.h

Issue 2639153002: Make extensions DSE persistent in browser prefs (Reland) (Closed)
Patch Set: Minor fix after review, round 7 Created 3 years, 10 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 d57aed22627266467d0f47eb6bf5b413c5bbface..92fb852c1a5238b50f94686186b15ab758b33771 100644
--- a/components/search_engines/template_url.h
+++ b/components/search_engines/template_url.h
@@ -512,6 +512,11 @@ 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& key1,
+ const std::string& key2);
+
const TemplateURLData& data() const { return data_; }
const base::string16& short_name() const { return data_.short_name(); }
@@ -587,11 +592,8 @@ class TemplateURL {
Type type() const { return type_; }
- // This setter shouldn't be used except by TemplateURLService and
- // TemplateURLServiceClient implementations.
- void set_extension_info(
- std::unique_ptr<AssociatedExtensionInfo> extension_info) {
- extension_info_ = std::move(extension_info);
+ const AssociatedExtensionInfo* GetExtensionInfoForTesting() const {
+ return extension_info_.get();
}
// Returns true if |url| supports replacement.
« 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