Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(265)

Unified Diff: tools/valgrind/memcheck/suppressions.txt

Issue 10808054: Don't leak ChromeDownloadManagerDelegate in DownloadService (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Actual fix Created 8 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tools/valgrind/memcheck/suppressions.txt
diff --git a/tools/valgrind/memcheck/suppressions.txt b/tools/valgrind/memcheck/suppressions.txt
index 7927f216d1f1922fa468ab223ec05baaf1c8a428..0599ac18fa9708b52ed8f398e74f567100389e52 100644
--- a/tools/valgrind/memcheck/suppressions.txt
+++ b/tools/valgrind/memcheck/suppressions.txt
@@ -2039,13 +2039,6 @@
fun:_ZN11webkit_glue16WebURLLoaderImpl7Context18OnReceivedResponseERKNS_20ResourceResponseInfoEb
}
{
- bug_58449
- Memcheck:Leak
- fun:_Znw*
- ...
- fun:_ZN11ProfileImpl18GetDownloadManagerEv
-}
-{
bug_58546
Memcheck:Leak
fun:_Znw*
@@ -4062,41 +4055,6 @@
fun:_ZN3gfx31JPEGCodec_EncodeDecodeRGBA_Test8TestBodyEv
}
{
- bug_102255
- Memcheck:Leak
- fun:_Znw*
- fun:_ZN15DownloadService18GetDownloadManagerEv
- fun:_ZN29ExtensionDownloadsEventRouterC*
- fun:_ZN16ExtensionService16InitEventRoutersEv
- fun:_ZN16ExtensionService7ObserveEiRKN7content18NotificationSourceERKNS0_19NotificationDetailsE
- ...
- fun:_ZN23NotificationServiceImpl6NotifyEiRKN7content18NotificationSourceERKNS0_19NotificationDetailsE
- fun:_ZN14ProfileManager16OnImportFinishedEP7Profile
- fun:_ZN22ChromeBrowserMainParts25PreMainMessageLoopRunImplEv
-}
-{
- bug_102255b
- Memcheck:Leak
- fun:_Znw*
- ...
- fun:_ZN15DownloadService18GetDownloadManagerEv
- fun:_ZN11DownloadsUIC1EPN7content5WebUIE
- fun:_ZN12_GLOBAL__N_18NewWebUII11DownloadsUIEEPN7content15WebUIControllerEPNS2_5WebUIERK4GURL
- fun:_ZNK28ChromeWebUIControllerFactory27CreateWebUIControllerForURLEPN7content5WebUIERK4GURL
- fun:_ZNK32TestChromeWebUIControllerFactory27CreateWebUIControllerForURLEPN7content5WebUIERK4GURL
- fun:_ZN15WebContentsImpl11CreateWebUIERK4GURL
- fun:_ZN15WebContentsImpl27CreateWebUIForRenderManagerERK4GURL
- fun:_ZN21RenderViewHostManager30UpdateRendererStateForNavigateERKN7content19NavigationEntryImplE
- fun:_ZN21RenderViewHostManager8NavigateERKN7content19NavigationEntryImplE
- ...
- fun:_ZN24NavigationControllerImpl9LoadEntryEPN7content19NavigationEntryImplE
- fun:_ZN24NavigationControllerImpl7LoadURLERK4GURLRKN7content8ReferrerENS3_14PageTransitionERKSs
- fun:_ZN12_GLOBAL__N_117LoadURLInContentsEPN7content11WebContentsERK4GURLPN7browser14NavigateParamsERKSs
- fun:_ZN7browser8NavigateEPNS_14NavigateParamsE
- fun:_ZN7Browser30ShowSingletonTabOverwritingNTPERKN7browser14NavigateParamsE
- fun:_ZN7Browser16ShowDownloadsTabEv
-}
-{
bug_102257
Memcheck:Uninitialized
...
@@ -4990,18 +4948,6 @@
fun:_ZN13sync_notifier12_GLOBAL__N_124InvalidationNotifierTest5SetUpEv
}
{
- bug_chromium_os_28149
- Memcheck:Leak
- fun:_Znw*
- fun:_ZN15DownloadService18GetDownloadManagerEv
- fun:_ZN5gdata18GDataSystemService10InitializeEv
- fun:_ZNK5gdata25GDataSystemServiceFactory23BuildServiceInstanceForEP7Profile
- fun:_ZN26ProfileKeyedServiceFactory20GetServiceForProfileEP7Profileb
- fun:_ZN5gdata25GDataSystemServiceFactory13GetForProfileEP7Profile
- fun:_ZN22FileBrowserEventRouter23ObserveFileSystemEventsEv
- fun:_ZN16ExtensionService16InitEventRoutersEv
-}
-{
bug_120146
Memcheck:Leak
fun:_Znw*
« chrome/browser/download/download_service.cc ('K') | « chrome/browser/download/download_service.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698