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

Unified Diff: content/browser/net/quota_policy_cookie_store.cc

Issue 2383393002: Remove stl_util's deletion functions from net/cookies/ and net/extras/. (Closed)
Patch Set: removing Created 4 years, 2 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: content/browser/net/quota_policy_cookie_store.cc
diff --git a/content/browser/net/quota_policy_cookie_store.cc b/content/browser/net/quota_policy_cookie_store.cc
index 28edfbf170f69ee65248da80d43bc564d0e290a7..b759c5b608c8f7f9fd9d0c5dfb022609cadfc2ed 100644
--- a/content/browser/net/quota_policy_cookie_store.cc
+++ b/content/browser/net/quota_policy_cookie_store.cc
@@ -97,14 +97,14 @@ void QuotaPolicyCookieStore::Flush(const base::Closure& callback) {
void QuotaPolicyCookieStore::OnLoad(
const LoadedCallback& loaded_callback,
- const std::vector<net::CanonicalCookie*>& cookies) {
- for (auto* cookie : cookies) {
+ std::vector<std::unique_ptr<net::CanonicalCookie>> cookies) {
+ for (const auto& cookie : cookies) {
net::SQLitePersistentCookieStore::CookieOrigin origin(
cookie->Domain(), cookie->IsSecure());
++cookies_per_origin_[origin];
}
- loaded_callback.Run(cookies);
+ loaded_callback.Run(std::move(cookies));
}
CookieStoreConfig::CookieStoreConfig()
« no previous file with comments | « content/browser/net/quota_policy_cookie_store.h ('k') | content/browser/net/quota_policy_cookie_store_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698