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

Unified Diff: content/child/push_messaging/push_provider.h

Issue 2496653002: Part 2 of base::IDMap refactor to eliminate IDMapOwnPointer/IDMapExternalPointer modes (Closed)
Patch Set: Rebase Created 4 years 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/child/push_messaging/push_provider.h
diff --git a/content/child/push_messaging/push_provider.h b/content/child/push_messaging/push_provider.h
index 70471ff6424685c1a6c851f36a19cddde17b2bff..9d554fe8ed61251204077ffa083030dd0ec1a00d 100644
--- a/content/child/push_messaging/push_provider.h
+++ b/content/child/push_messaging/push_provider.h
@@ -100,17 +100,17 @@ class PushProvider : public blink::WebPushProvider,
// Stores the subscription callbacks with their request ids. This class owns
// the callbacks.
- IDMap<blink::WebPushSubscriptionCallbacks, IDMapOwnPointer>
+ IDMap<std::unique_ptr<blink::WebPushSubscriptionCallbacks>>
subscription_callbacks_;
// Stores the permission status callbacks with their request ids. This class
// owns the callbacks.
- IDMap<blink::WebPushPermissionStatusCallbacks, IDMapOwnPointer>
+ IDMap<std::unique_ptr<blink::WebPushPermissionStatusCallbacks>>
permission_status_callbacks_;
// Stores the unsubscription callbacks with their request ids. This class owns
// the callbacks.
- IDMap<blink::WebPushUnsubscribeCallbacks, IDMapOwnPointer>
+ IDMap<std::unique_ptr<blink::WebPushUnsubscribeCallbacks>>
unsubscribe_callbacks_;
DISALLOW_COPY_AND_ASSIGN(PushProvider);

Powered by Google App Engine
This is Rietveld 408576698