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

Unified Diff: net/cookies/cookie_store.cc

Issue 2937963003: Shift cookie system callbacks to OnceCallback to impedance match mojo. (Closed)
Patch Set: Finish Merge Created 3 years, 6 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: net/cookies/cookie_store.cc
diff --git a/net/cookies/cookie_store.cc b/net/cookies/cookie_store.cc
index 20f2dad64f8722bcabd45e252f01c3b90bd8bd9b..ffe939bf6be6ed99eb42d4abd4a5b9653640d5af 100644
--- a/net/cookies/cookie_store.cc
+++ b/net/cookies/cookie_store.cc
@@ -48,23 +48,23 @@ std::string CookieStore::BuildCookieLine(
return cookie_line;
}
-void CookieStore::DeleteAllAsync(const DeleteCallback& callback) {
- DeleteAllCreatedBetweenAsync(base::Time(), base::Time::Max(), callback);
+void CookieStore::DeleteAllAsync(DeleteCallback callback) {
+ DeleteAllCreatedBetweenAsync(base::Time(), base::Time::Max(),
+ std::move(callback));
}
void CookieStore::SetForceKeepSessionState() {
// By default, do nothing.
}
-void CookieStore::GetAllCookiesForURLAsync(
- const GURL& url,
- const GetCookieListCallback& callback) {
+void CookieStore::GetAllCookiesForURLAsync(const GURL& url,
+ GetCookieListCallback callback) {
CookieOptions options;
options.set_include_httponly();
options.set_same_site_cookie_mode(
CookieOptions::SameSiteCookieMode::INCLUDE_STRICT_AND_LAX);
options.set_do_not_update_access_time();
- GetCookieListWithOptionsAsync(url, options, callback);
+ GetCookieListWithOptionsAsync(url, options, std::move(callback));
}
void CookieStore::SetChannelIDServiceID(int id) {

Powered by Google App Engine
This is Rietveld 408576698