Store unique_ptr instead of raw poiter in extension callback map
There is callback map in extension function dispatcher, which store
pointers on callback wrappers. But it force to remember to delete
wrapper after erase from map. This may lead to memory leak.
To prevent it, raw pointer in map was replaced by unique_ptr (like in
other callback map in same class).
BUG=None
R=finnur
Review-Url: https://codereview.chromium.org/2850533002
Cr-Commit-Position: refs/heads/master@{#467932}
diff --git a/extensions/browser/extension_function_dispatcher.h b/extensions/browser/extension_function_dispatcher.h
index 073ca0e..c7b1d8f 100644
--- a/extensions/browser/extension_function_dispatcher.h
+++ b/extensions/browser/extension_function_dispatcher.h
@@ -169,7 +169,8 @@
// This map doesn't own either the keys or the values. When a RenderFrameHost
// instance goes away, the corresponding entry in this map (if exists) will be
// removed.
- typedef std::map<content::RenderFrameHost*, UIThreadResponseCallbackWrapper*>
+ typedef std::map<content::RenderFrameHost*,
+ std::unique_ptr<UIThreadResponseCallbackWrapper>>
UIThreadResponseCallbackWrapperMap;
UIThreadResponseCallbackWrapperMap ui_thread_response_callback_wrappers_;
