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

Unified Diff: chrome/browser/push_messaging/push_messaging_notification_manager.cc

Issue 2828663002: Rewrite base::Bind to base::BindOnce with base_bind_rewriters in //chrome/browser/{i,l,m,n,p,r}* (Closed)
Patch Set: Created 3 years, 8 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/push_messaging/push_messaging_notification_manager.cc
diff --git a/chrome/browser/push_messaging/push_messaging_notification_manager.cc b/chrome/browser/push_messaging/push_messaging_notification_manager.cc
index 13068b00d5496b919e441f0dc07cdca7259d38a2..b2df4e986eb08725688b87a207fcc093a0cf3587 100644
--- a/chrome/browser/push_messaging/push_messaging_notification_manager.cc
+++ b/chrome/browser/push_messaging/push_messaging_notification_manager.cc
@@ -104,7 +104,7 @@ void PushMessagingNotificationManager::EnforceUserVisibleOnlyRequirements(
BrowserThread::PostTask(
BrowserThread::IO, FROM_HERE,
- base::Bind(
+ base::BindOnce(
&PlatformNotificationContext::
ReadAllNotificationDataForServiceWorkerRegistration,
notification_context, origin, service_worker_registration_id,
@@ -125,7 +125,7 @@ void PushMessagingNotificationManager::DidGetNotificationsFromDatabaseIOProxy(
DCHECK_CURRENTLY_ON(BrowserThread::IO);
BrowserThread::PostTask(
BrowserThread::UI, FROM_HERE,
- base::Bind(
+ base::BindOnce(
&PushMessagingNotificationManager::DidGetNotificationsFromDatabase,
ui_weak_ptr, origin, service_worker_registration_id,
message_handled_closure, success, data));
@@ -276,13 +276,13 @@ void PushMessagingNotificationManager::ProcessSilentPush(
GetStoragePartition(profile_, origin)->GetPlatformNotificationContext();
BrowserThread::PostTask(
BrowserThread::IO, FROM_HERE,
- base::Bind(&PlatformNotificationContext::WriteNotificationData,
- notification_context, origin, database_data,
- base::Bind(&PushMessagingNotificationManager::
- DidWriteNotificationDataIOProxy,
- weak_factory_.GetWeakPtr(), origin,
- database_data.notification_data,
- message_handled_closure)));
+ base::BindOnce(&PlatformNotificationContext::WriteNotificationData,
+ notification_context, origin, database_data,
+ base::Bind(&PushMessagingNotificationManager::
+ DidWriteNotificationDataIOProxy,
+ weak_factory_.GetWeakPtr(), origin,
+ database_data.notification_data,
+ message_handled_closure)));
}
// static
@@ -296,9 +296,10 @@ void PushMessagingNotificationManager::DidWriteNotificationDataIOProxy(
DCHECK_CURRENTLY_ON(BrowserThread::IO);
BrowserThread::PostTask(
BrowserThread::UI, FROM_HERE,
- base::Bind(&PushMessagingNotificationManager::DidWriteNotificationData,
- ui_weak_ptr, origin, notification_data,
- message_handled_closure, success, notification_id));
+ base::BindOnce(
+ &PushMessagingNotificationManager::DidWriteNotificationData,
+ ui_weak_ptr, origin, notification_data, message_handled_closure,
+ success, notification_id));
}
void PushMessagingNotificationManager::DidWriteNotificationData(

Powered by Google App Engine
This is Rietveld 408576698