bluetooth: Fix leak in getCharacteristics

bluetooth: Fix leak in getCharacteristics

We were not properly removing the callbacks from the map.

WebKit Linux Leak caught a leak in the getCharacteristics LayoutTests:

({"numberOfLiveActiveDOMObjects":[2,3]})

https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Leak/17788/layout-test-results/results.html

Reproduced locally by running run_layout_tests.sh with --enable-leak-detection.

BUG=590865
Committed: https://crrev.com/542f9e5244355165647f0a6897b7c44bdc92503b
Cr-Commit-Position: refs/heads/master@{#378339}

[ortuno, 2016-02-29 21:59:02]

ortuno@chromium.org changed reviewers:
+ scheib@chromium.org

[ortuno, 2016-02-29 21:59:03]

scheib: PTAL

[ortuno, 2016-02-29 21:59:07]

The CQ bit was checked by ortuno@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-29 21:59:33]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1746973002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1746973002/1

[scheib, 2016-02-29 22:02:09]

LGTM, but please explain in description how to verify that this fixes the issue.

[ortuno, 2016-02-29 22:10:37]

Description was changed from

==========
bluetooth: Fix leak in getCharacteristics

We were not properly removing the callbacks from the map.

BUG=590865
==========

to

==========
bluetooth: Fix leak in getCharacteristics

We were not properly removing the callbacks from the map. 

WebKit Linux Leak caught a leak in the getCharacteristics LayoutTests:

({"numberOfLiveActiveDOMObjects":[2,3]})

https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Lea...

Reproduced locally by running run_layout_tests.sh with --enable-leak-detection.

BUG=590865
==========

[commit-bot: I haz the power, 2016-03-01 00:08:11]

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

[commit-bot: I haz the power, 2016-03-01 00:08:12]

Dry run: This issue passed the CQ dry run.

[ortuno, 2016-03-01 00:14:52]

The CQ bit was checked by ortuno@chromium.org

[commit-bot: I haz the power, 2016-03-01 00:15:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1746973002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1746973002/1

[commit-bot: I haz the power, 2016-03-01 00:24:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-01 00:26:30]

Description was changed from

==========
bluetooth: Fix leak in getCharacteristics

We were not properly removing the callbacks from the map. 

WebKit Linux Leak caught a leak in the getCharacteristics LayoutTests:

({"numberOfLiveActiveDOMObjects":[2,3]})

https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Lea...

Reproduced locally by running run_layout_tests.sh with --enable-leak-detection.

BUG=590865
==========

to

==========
bluetooth: Fix leak in getCharacteristics

We were not properly removing the callbacks from the map.

WebKit Linux Leak caught a leak in the getCharacteristics LayoutTests:

({"numberOfLiveActiveDOMObjects":[2,3]})

https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Lea...

Reproduced locally by running run_layout_tests.sh with --enable-leak-detection.

BUG=590865

Committed: https://crrev.com/542f9e5244355165647f0a6897b7c44bdc92503b
Cr-Commit-Position: refs/heads/master@{#378339}
==========

[commit-bot: I haz the power, 2016-03-01 00:26:31]

Patchset 1 (id:??) landed as
https://crrev.com/542f9e5244355165647f0a6897b7c44bdc92503b
Cr-Commit-Position: refs/heads/master@{#378339}