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

Unified Diff: chrome/browser/sessions/session_data_deleter.cc

Issue 2971733002: Change CookieStore::DeleteCallback to take uint32_t. (Closed)
Patch Set: Fixed Android webview compilation errors. Created 3 years, 5 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: chrome/browser/sessions/session_data_deleter.cc
diff --git a/chrome/browser/sessions/session_data_deleter.cc b/chrome/browser/sessions/session_data_deleter.cc
index 72af6aaa101e5cac5c96ec312cf69debb4211c3b..a1be97d90ad87f0b08e667e56b49e13a71e66819 100644
--- a/chrome/browser/sessions/session_data_deleter.cc
+++ b/chrome/browser/sessions/session_data_deleter.cc
@@ -4,6 +4,8 @@
#include <stddef.h>
+#include <cstdint>
+
#include "base/bind.h"
#include "base/command_line.h"
#include "base/macros.h"
@@ -26,8 +28,8 @@
namespace {
-void CookieDeleted(int num_cookies_deleted) {
- DCHECK_EQ(1, num_cookies_deleted);
+void CookieDeleted(uint32_t num_cookies_deleted) {
+ DCHECK_EQ(1u, num_cookies_deleted);
}
class SessionDataDeleter
@@ -59,7 +61,7 @@ class SessionDataDeleter
// Called when all session-only cookies have been deleted.
void DeleteSessionCookiesDone(net::CookieStore* cookie_store,
- int num_deleted);
+ uint32_t num_deleted);
// Deletes the cookies in |cookies| that are for origins which are
// session-only.
@@ -134,7 +136,7 @@ void SessionDataDeleter::DeleteSessionCookiesOnIOThread(
void SessionDataDeleter::DeleteSessionCookiesDone(
net::CookieStore* cookie_store,
- int num_deleted) {
+ uint32_t num_deleted) {
// If these callbacks are invoked, |cookie_store| is gauranteed to still
// exist, since deleting the CookieStore will cancel pending callbacks.
cookie_store->GetAllCookiesAsync(

Powered by Google App Engine
This is Rietveld 408576698