Issue 10546025: Delete empty NotificationObserverLists. Fixes leak.

lgtm http://codereview.chromium.org/10546025/diff/1/content/browser/notification_service_impl.cc File content/browser/notification_service_impl.cc (right): http://codereview.chromium.org/10546025/diff/1/content/browser/notification_service_impl.cc#newcode85 content/browser/notification_service_impl.cc:85: observer_list = NULL; nit: unnecessary

8 years, 6 months ago (2012-06-06 16:48:05 UTC) #2

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/leng@google.com/10546025/4001

8 years, 6 months ago (2012-06-07 14:08:05 UTC) #3

lengG

Thanks for the review! https://chromiumcodereview.appspot.com/10546025/diff/1/content/browser/notification_service_impl.cc File content/browser/notification_service_impl.cc (right): https://chromiumcodereview.appspot.com/10546025/diff/1/content/browser/notification_service_impl.cc#newcode85 content/browser/notification_service_impl.cc:85: observer_list = NULL; On 2012/06/06 ...

8 years, 6 months ago (2012-06-07 14:08:51 UTC) #4

Change committed as 141010

Expand Messages | Collapse Messages

Issue 10546025: Delete empty NotificationObserverLists. Fixes leak. (Closed)

Description

Patch Set 1 #

Patch Set 2 : CR feedback: removed unnecessary = NULL. #

Messages