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

Unified Diff: components/autofill/core/browser/autocomplete_history_manager_unittest.cc

Issue 1859453002: components/autofill: scoped_ptr -> unique_ptr (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Comments addressed Created 4 years, 8 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/autofill/core/browser/autocomplete_history_manager_unittest.cc
diff --git a/components/autofill/core/browser/autocomplete_history_manager_unittest.cc b/components/autofill/core/browser/autocomplete_history_manager_unittest.cc
index 601e5748cda9bb680ff192d6347e3f456957cee7..90906d76632569f8396fe36b8fe3f7c3b02fa01e 100644
--- a/components/autofill/core/browser/autocomplete_history_manager_unittest.cc
+++ b/components/autofill/core/browser/autocomplete_history_manager_unittest.cc
@@ -57,7 +57,7 @@ class MockAutofillClient : public TestAutofillClient {
private:
scoped_refptr<MockWebDataService> web_data_service_;
- scoped_ptr<PrefService> prefs_;
+ std::unique_ptr<PrefService> prefs_;
DISALLOW_COPY_AND_ASSIGN(MockAutofillClient);
};
@@ -80,9 +80,9 @@ class AutocompleteHistoryManagerTest : public testing::Test {
base::MessageLoop message_loop_;
scoped_refptr<MockWebDataService> web_data_service_;
- scoped_ptr<AutocompleteHistoryManager> autocomplete_manager_;
- scoped_ptr<AutofillDriver> autofill_driver_;
- scoped_ptr<MockAutofillClient> autofill_client_;
+ std::unique_ptr<AutocompleteHistoryManager> autocomplete_manager_;
+ std::unique_ptr<AutofillDriver> autofill_driver_;
+ std::unique_ptr<MockAutofillClient> autofill_client_;
};
// Tests that credit card numbers are not sent to the WebDatabase to be saved.
@@ -222,11 +222,9 @@ TEST_F(AutocompleteHistoryManagerTest, ExternalDelegate) {
TestAutocompleteHistoryManager autocomplete_history_manager(
autofill_driver_.get(), autofill_client_.get());
- scoped_ptr<AutofillManager> autofill_manager(
- new AutofillManager(autofill_driver_.get(),
- autofill_client_.get(),
- "en-US",
- AutofillManager::ENABLE_AUTOFILL_DOWNLOAD_MANAGER));
+ std::unique_ptr<AutofillManager> autofill_manager(new AutofillManager(
+ autofill_driver_.get(), autofill_client_.get(), "en-US",
+ AutofillManager::ENABLE_AUTOFILL_DOWNLOAD_MANAGER));
MockAutofillExternalDelegate external_delegate(autofill_manager.get(),
autofill_driver_.get());
@@ -242,11 +240,9 @@ TEST_F(AutocompleteHistoryManagerTest, NoAutocompleteSuggestionsForTextarea) {
TestAutocompleteHistoryManager autocomplete_history_manager(
autofill_driver_.get(), autofill_client_.get());
- scoped_ptr<AutofillManager> autofill_manager(
- new AutofillManager(autofill_driver_.get(),
- autofill_client_.get(),
- "en-US",
- AutofillManager::ENABLE_AUTOFILL_DOWNLOAD_MANAGER));
+ std::unique_ptr<AutofillManager> autofill_manager(new AutofillManager(
+ autofill_driver_.get(), autofill_client_.get(), "en-US",
+ AutofillManager::ENABLE_AUTOFILL_DOWNLOAD_MANAGER));
MockAutofillExternalDelegate external_delegate(autofill_manager.get(),
autofill_driver_.get());
« no previous file with comments | « components/autofill/core/browser/address_i18n_unittest.cc ('k') | components/autofill/core/browser/autofill_client.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698