Store unique_ptr instead of raw poiter in extension callback map

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}
Committed: https://chromium.googlesource.com/chromium/src/+/fbaad662ea5f8e62e03ab2febe39b7cd7946d047

[Finnur, 2017-04-27 14:48:26]

LGTM

[apisarev, 2017-04-27 14:51:32]

The CQ bit was checked by apisarev@yandex-team.ru

[commit-bot: I haz the power, 2017-04-27 14:52:16]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-27 17:53:11]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-27 17:53:12]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[apisarev, 2017-04-28 08:09:27]

The CQ bit was checked by apisarev@yandex-team.ru

[commit-bot: I haz the power, 2017-04-28 08:09:40]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-28 08:36:55]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493366967455980, "parent_rev":
"030745885a307ced7964ab2288cbd36ecd9ec969", "commit_rev":
"fbaad662ea5f8e62e03ab2febe39b7cd7946d047"}

[commit-bot: I haz the power, 2017-04-28 08:37:01]

Description was changed from

==========
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
==========

to

==========
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}
Committed:
https://chromium.googlesource.com/chromium/src/+/fbaad662ea5f8e62e03ab2febe39...
==========

[commit-bot: I haz the power, 2017-04-28 08:37:02]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/fbaad662ea5f8e62e03ab2febe39...