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

Unified Diff: build/sanitizers/lsan_suppressions.cc

Issue 933623002: Prune unused LSan suppressions. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/sanitizers/lsan_suppressions.cc
diff --git a/build/sanitizers/lsan_suppressions.cc b/build/sanitizers/lsan_suppressions.cc
index 410a8e3e18198b89164c36e8552774376a2242d5..a076ba02261637f1625591e490d406ec7ff9ed63 100644
--- a/build/sanitizers/lsan_suppressions.cc
+++ b/build/sanitizers/lsan_suppressions.cc
@@ -25,12 +25,9 @@ char kLSanDefaultSuppressions[] =
// Leaks in Nvidia's libGL.
"leak:libGL.so\n"
-// A small string is leaked here (57 bytes per process). http://crbug.com/46571#c9
+// A small leak in V8. http://crbug.com/46571#c9
"leak:blink::V8GCController::collectGarbage\n"
-// http://crbug.com/270180
-"leak:net::ProxyResolverV8::Context::ResolveProxy\n"
-
// TODO(earthdok): revisit NSS suppressions after the switch to BoringSSL
// NSS leaks in CertDatabaseNSSTest tests. http://crbug.com/51988
"leak:net::NSSCertDatabase::ImportFromPKCS12\n"
@@ -51,15 +48,9 @@ char kLSanDefaultSuppressions[] =
// XRandR has several one time leaks.
"leak:libxrandr\n"
-// Skia leaks GrGpuGL::ProgramCache::Entry. http://crbug.com/262934
-"leak:GrGpuGL::flushGraphicsState\n"
-
// xrandr leak. http://crbug.com/119677
"leak:XRRFindDisplay\n"
-// V8 may leak this by design in unit tests. http://crbug.com/323149
-"leak:v8::internal::Genesis::Genesis\n"
-
// Suppressions for objects which can be owned by the V8 heap. This is a
// temporary workaround until LeakSanitizer supports the V8 heap.
// Those should only fire in (browser)tests. If you see one of them in Chrome,
@@ -89,9 +80,6 @@ char kLSanDefaultSuppressions[] =
// Small test-only leak in ppapi_unittests. http://crbug.com/258113
"leak:ppapi::proxy::PPP_Instance_Private_ProxyTest_PPPInstancePrivate_Test\n"
-// http://crbug.com/318221
-"leak:base::EnsureProcessTerminated\n"
-
// http://crbug.com/322671
"leak:content::SpeechRecognitionBrowserTest::SetUpOnMainThread\n"
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698