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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/ui/passwords/password_manager_presenter.h

Issue 866983003: GetLoginsRequest: Use ScopedVector to express ownership of forms (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@324291_scopedvector
Patch Set: Second fix of the rebase Created 5 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 | « chrome/browser/sync/test/integration/performance/passwords_sync_perf_test.cc ('k') | chrome/browser/ui/passwords/password_manager_presenter.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/passwords/password_manager_presenter.h
diff --git a/chrome/browser/ui/passwords/password_manager_presenter.h b/chrome/browser/ui/passwords/password_manager_presenter.h
index 52523cf000bead5a576da0e5e6f6b0b4513952e5..e3d7922cea0a4d425bf7fd837dfcb81ee2beaded 100644
--- a/chrome/browser/ui/passwords/password_manager_presenter.h
+++ b/chrome/browser/ui/passwords/password_manager_presenter.h
@@ -95,7 +95,7 @@ class PasswordManagerPresenter
// Send the password store's reply back to the handler.
void OnGetPasswordStoreResults(
- const std::vector<autofill::PasswordForm*>& results) override;
+ ScopedVector<autofill::PasswordForm> results) override;
};
// A short class to mediate requests to the password store for exceptions.
@@ -108,7 +108,7 @@ class PasswordManagerPresenter
// Send the password store's reply back to the handler.
void OnGetPasswordStoreResults(
- const std::vector<autofill::PasswordForm*>& results) override;
+ ScopedVector<autofill::PasswordForm> results) override;
};
// Password store consumer for populating the password list and exceptions.
« no previous file with comments | « chrome/browser/sync/test/integration/performance/passwords_sync_perf_test.cc ('k') | chrome/browser/ui/passwords/password_manager_presenter.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698