Chromium Code Reviews| Index: components/omnibox/browser/shortcuts_backend_unittest.cc |
| diff --git a/components/omnibox/browser/shortcuts_backend_unittest.cc b/components/omnibox/browser/shortcuts_backend_unittest.cc |
| index 2cd6235e40977bf61874e3b1cb5871f77b92748b..e50e81f9f0598284e214474485c38df34dce09e8 100644 |
| --- a/components/omnibox/browser/shortcuts_backend_unittest.cc |
| +++ b/components/omnibox/browser/shortcuts_backend_unittest.cc |
| @@ -104,9 +104,10 @@ void ShortcutsBackendTest::SetSearchProvider() { |
| data.SetShortName(base::UTF8ToUTF16("foo")); |
| data.SetKeyword(base::UTF8ToUTF16("foo")); |
| - TemplateURL* template_url = new TemplateURL(data); |
| - // Takes ownership of |template_url|. |
| - template_url_service_->Add(template_url); |
| + std::unique_ptr<TemplateURL> template_url_ptr = |
| + base::MakeUnique<TemplateURL>(data); |
| + TemplateURL* template_url = template_url_ptr.get(); |
| + template_url_service_->Add(std::move(template_url_ptr)); |
|
Peter Kasting
2016/08/31 04:12:56
Nit: Just use the old code and add a WrapUnique()
Avi (use Gerrit)
2016/09/01 00:34:26
Done.
|
| template_url_service_->SetUserSelectedDefaultSearchProvider(template_url); |
| } |