diff options
author | groby <groby@chromium.org> | 2014-09-24 18:06:14 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-09-25 01:06:25 +0000 |
commit | b49c29b0ee11c0228c0a5c210edcdff79e04bb12 (patch) | |
tree | 5d5dd0496b18028490818a93a11fcf947e24903b /components/omnibox | |
parent | ff9eea50913e4d100cb2931050a6d5c83ad6c7ba (diff) | |
download | chromium_src-b49c29b0ee11c0228c0a5c210edcdff79e04bb12.zip chromium_src-b49c29b0ee11c0228c0a5c210edcdff79e04bb12.tar.gz chromium_src-b49c29b0ee11c0228c0a5c210edcdff79e04bb12.tar.bz2 |
Fix memory leak in test
BUG=none
TBR=mpearson@chromium.org
R=jiayul@chromium.org
Review URL: https://codereview.chromium.org/601033002
Cr-Commit-Position: refs/heads/master@{#296604}
Diffstat (limited to 'components/omnibox')
-rw-r--r-- | components/omnibox/base_search_provider_unittest.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/components/omnibox/base_search_provider_unittest.cc b/components/omnibox/base_search_provider_unittest.cc index 1a64ffa..42a9f18 100644 --- a/components/omnibox/base_search_provider_unittest.cc +++ b/components/omnibox/base_search_provider_unittest.cc @@ -118,7 +118,7 @@ class BaseSearchProviderTest : public testing::Test { TEST_F(BaseSearchProviderTest, PreserveAnswersWhenDeduplicating) { TemplateURLData data; data.SetURL("http://foo.com/url?bar={searchTerms}"); - TemplateURL* template_url = new TemplateURL(data); + scoped_ptr<TemplateURL> template_url(new TemplateURL(data)); TestBaseSearchProvider::MatchMap map; base::string16 query = base::ASCIIToUTF16("weather los angeles"); @@ -128,7 +128,7 @@ TEST_F(BaseSearchProviderTest, PreserveAnswersWhenDeduplicating) { EXPECT_CALL(*provider_, GetInput(_)) .WillRepeatedly(Return(AutocompleteInput())); EXPECT_CALL(*provider_, GetTemplateURL(_)) - .WillRepeatedly(Return(template_url)); + .WillRepeatedly(Return(template_url.get())); SearchSuggestionParser::SuggestResult more_relevant( query, AutocompleteMatchType::SEARCH_HISTORY, query, base::string16(), |