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

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

Issue 2971733002: Change CookieStore::DeleteCallback to take uint32_t. (Closed)
Patch Set: Got rid of rest of <cstdint> stuff. 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..e4b2d2638e010e8e3713506e758285f91e91df9a 100644
--- a/chrome/browser/sessions/session_data_deleter.cc
+++ b/chrome/browser/sessions/session_data_deleter.cc
@@ -3,6 +3,7 @@
// found in the LICENSE file.
#include <stddef.h>
+#include <stdint.h>
#include "base/bind.h"
#include "base/command_line.h"
@@ -26,8 +27,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 +60,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 +135,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