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

Unified Diff: components/omnibox/browser/autocomplete_provider_unittest.cc

Issue 2290503003: Remove use of stl_util in search_engines. (Closed)
Patch Set: fix broken test Created 4 years, 3 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: components/omnibox/browser/autocomplete_provider_unittest.cc
diff --git a/components/omnibox/browser/autocomplete_provider_unittest.cc b/components/omnibox/browser/autocomplete_provider_unittest.cc
index 93b8c837509a9135879e015424e4d6ba2228b20a..d20f4be89e2f1b5586915bb39e659d2c17daab12 100644
--- a/components/omnibox/browser/autocomplete_provider_unittest.cc
+++ b/components/omnibox/browser/autocomplete_provider_unittest.cc
@@ -302,12 +302,12 @@ void AutocompleteProviderTest::RegisterTemplateURL(
data.SetURL(template_url);
data.SetShortName(keyword);
data.SetKeyword(keyword);
- TemplateURL* default_t_url = new TemplateURL(data);
TemplateURLService* turl_model = client_->GetTemplateURLService();
- turl_model->Add(default_t_url);
- turl_model->SetUserSelectedDefaultSearchProvider(default_t_url);
+ TemplateURL* default_turl =
+ turl_model->Add(base::MakeUnique<TemplateURL>(data));
+ turl_model->SetUserSelectedDefaultSearchProvider(default_turl);
turl_model->Load();
- TemplateURLID default_provider_id = default_t_url->id();
+ TemplateURLID default_provider_id = default_turl->id();
ASSERT_NE(0, default_provider_id);
}
@@ -367,11 +367,11 @@ void AutocompleteProviderTest::ResetControllerWithKeywordAndSearchProviders() {
data.SetShortName(base::ASCIIToUTF16("default"));
data.SetKeyword(base::ASCIIToUTF16("default"));
data.SetURL("http://defaultturl/{searchTerms}");
- TemplateURL* default_t_url = new TemplateURL(data);
TemplateURLService* turl_model = client_->GetTemplateURLService();
- turl_model->Add(default_t_url);
- turl_model->SetUserSelectedDefaultSearchProvider(default_t_url);
- TemplateURLID default_provider_id = default_t_url->id();
+ TemplateURL* default_turl =
+ turl_model->Add(base::MakeUnique<TemplateURL>(data));
+ turl_model->SetUserSelectedDefaultSearchProvider(default_turl);
+ TemplateURLID default_provider_id = default_turl->id();
ASSERT_NE(0, default_provider_id);
// Create another TemplateURL for KeywordProvider.
@@ -379,9 +379,9 @@ void AutocompleteProviderTest::ResetControllerWithKeywordAndSearchProviders() {
data2.SetShortName(base::ASCIIToUTF16("k"));
data2.SetKeyword(base::ASCIIToUTF16("k"));
data2.SetURL("http://keyword/{searchTerms}");
- TemplateURL* keyword_t_url = new TemplateURL(data2);
- turl_model->Add(keyword_t_url);
- ASSERT_NE(0, keyword_t_url->id());
+ TemplateURL* keyword_turl =
+ turl_model->Add(base::MakeUnique<TemplateURL>(data2));
+ ASSERT_NE(0, keyword_turl->id());
ResetControllerWithType(AutocompleteProvider::TYPE_KEYWORD |
AutocompleteProvider::TYPE_SEARCH);
@@ -395,26 +395,24 @@ void AutocompleteProviderTest::ResetControllerWithKeywordProvider() {
data.SetShortName(base::ASCIIToUTF16("foo.com"));
data.SetKeyword(base::ASCIIToUTF16("foo.com"));
data.SetURL("http://foo.com/{searchTerms}");
- TemplateURL* keyword_t_url = new TemplateURL(data);
- turl_model->Add(keyword_t_url);
- ASSERT_NE(0, keyword_t_url->id());
+ TemplateURL* keyword_turl =
+ turl_model->Add(base::MakeUnique<TemplateURL>(data));
+ ASSERT_NE(0, keyword_turl->id());
// Make a TemplateURL for KeywordProvider that a shorter version of the
// first.
data.SetShortName(base::ASCIIToUTF16("f"));
data.SetKeyword(base::ASCIIToUTF16("f"));
data.SetURL("http://f.com/{searchTerms}");
- keyword_t_url = new TemplateURL(data);
- turl_model->Add(keyword_t_url);
- ASSERT_NE(0, keyword_t_url->id());
+ keyword_turl = turl_model->Add(base::MakeUnique<TemplateURL>(data));
+ ASSERT_NE(0, keyword_turl->id());
// Create another TemplateURL for KeywordProvider.
data.SetShortName(base::ASCIIToUTF16("bar.com"));
data.SetKeyword(base::ASCIIToUTF16("bar.com"));
data.SetURL("http://bar.com/{searchTerms}");
- keyword_t_url = new TemplateURL(data);
- turl_model->Add(keyword_t_url);
- ASSERT_NE(0, keyword_t_url->id());
+ keyword_turl = turl_model->Add(base::MakeUnique<TemplateURL>(data));
+ ASSERT_NE(0, keyword_turl->id());
ResetControllerWithType(AutocompleteProvider::TYPE_KEYWORD);
}

Powered by Google App Engine
This is Rietveld 408576698