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

Unified Diff: content/child/service_worker/web_service_worker_registration_impl.h

Issue 2480293004: Mandate unique_ptr for base::IDMap in IDMapOwnPointer mode. (Closed)
Patch Set: Make changes requested by danakj, fix a few more headers Created 4 years, 1 month 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/service_worker/web_service_worker_registration_impl.h
diff --git a/content/child/service_worker/web_service_worker_registration_impl.h b/content/child/service_worker/web_service_worker_registration_impl.h
index ce30fb00eae8dec1cf096f951825d3d01fa9f00f..46144444da79d7e0309ff9e7ab6a488b10ec8d4c 100644
--- a/content/child/service_worker/web_service_worker_registration_impl.h
+++ b/content/child/service_worker/web_service_worker_registration_impl.h
@@ -49,10 +49,12 @@ class CONTENT_EXPORT WebServiceWorkerRegistrationImpl
void setProxy(blink::WebServiceWorkerRegistrationProxy* proxy) override;
blink::WebServiceWorkerRegistrationProxy* proxy() override;
blink::WebURL scope() const override;
- void update(blink::WebServiceWorkerProvider* provider,
- WebServiceWorkerUpdateCallbacks* callbacks) override;
+ void update(
+ blink::WebServiceWorkerProvider* provider,
+ std::unique_ptr<WebServiceWorkerUpdateCallbacks> callbacks) override;
void unregister(blink::WebServiceWorkerProvider* provider,
- WebServiceWorkerUnregistrationCallbacks* callbacks) override;
+ std::unique_ptr<WebServiceWorkerUnregistrationCallbacks>
+ callbacks) override;
void enableNavigationPreload(
bool enable,
blink::WebServiceWorkerProvider* provider,

Powered by Google App Engine
This is Rietveld 408576698