Comment #7 on issue 15818 by bugdro...@chromium.org: Mem leak in
BlockedPopupContainerView::Create
http://code.google.com/p/chromium/issues/detail?id=15818
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=19944
------------------------------------------------------------------------
r19944 | pinker...@chromium.org | 2009-07-06 08:32:57 -0700 (Mon, 06 Jul
2009) | 4 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/blocked_popup_container.cc?r1=19944&r2=19943
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/blocked_popup_container.h?r1=19944&r2=19943
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/blocked_popup_container_controller.h?r1=19944&r2=19943
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/blocked_popup_container_controller.mm?r1=19944&r2=19943
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/blocked_popup_container_controller_unittest.mm?r1=19944&r2=19943
Add remaining functionality for popup blocker: popup menu to unblock
individual popups and whitelist sites. Also fixes intermittant leak on
valgrind bots from poorly constructed unit test.
BUG=13160, 15818
TEST=unblocking popups, whitelisting sites, and unwhitelisting them. Green
valgrind bot.
Review URL: http://codereview.chromium.org/149145
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---
