New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 590865 link

Starred by 1 user
Status: Fixed
Owner:
█ ortuno@chromium.org
Last visit > 30 days ago
Closed: Mar 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug

Blocking:
issue 436283



Sign in to add a comment

bluetooth: Fix getCharacteristics memory leak

Project Member Reported by ortuno@chromium.org, Feb 29 2016 
getCharacteristics is not properly deleting callbacks.
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 1 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/542f9e5244355165647f0a6897b7c44bdc92503b

commit 542f9e5244355165647f0a6897b7c44bdc92503b
Author: ortuno <ortuno@chromium.org>
Date: Tue Mar 01 00:24:37 2016

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 

Review URL: https://codereview.chromium.org/1746973002

Cr-Commit-Position: refs/heads/master@{#378339}

[modify] https://crrev.com/542f9e5244355165647f0a6897b7c44bdc92503b/content/renderer/bluetooth/bluetooth_dispatcher.cc

Comment 2 by ortuno@chromium.org, Mar 1 2016

Status: Fixed (was: Started)
►

Sign in to add a comment