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

Unified Diff: build/sanitizers/lsan_suppressions.cc

Issue 2613653003: Move some basic early process init into Service Manager (Closed)
Patch Set: fix mac shutdown order Created 3 years, 9 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 | content/DEPS » ('j') | 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 27a2426ad72ae69cd338d689d43ab3b4424e4f28..4b75528ad79c181a4b5da8e6f4b579d25abfbe6b 100644
--- a/build/sanitizers/lsan_suppressions.cc
+++ b/build/sanitizers/lsan_suppressions.cc
@@ -14,83 +14,85 @@
// newlines. See http://dev.chromium.org/developers/testing/leaksanitizer
// for the instructions on writing suppressions.
char kLSanDefaultSuppressions[] =
-// Intentional leak used as sanity test for Valgrind/memcheck.
-"leak:base::ToolsSanityTest_MemoryLeak_Test::TestBody\n"
-
-// ================ Leaks in third-party code ================
-
-// False positives in libfontconfig. http://crbug.com/39050
-"leak:libfontconfig\n"
-// eglibc-2.19/string/strdup.c creates false positive leak errors because of the
-// same reason as crbug.com/39050. The leak error stack trace, when unwind on
-// malloc, includes a call to libfontconfig. But the default stack trace is too
-// short in leak sanitizer bot to make the libfontconfig suppression works.
-// http://crbug.com/605286
-"leak:__strdup\n"
-
-// Leaks in Nvidia's libGL.
-"leak:libGL.so\n"
-
-// TODO(eugenis): revisit NSS suppressions after the switch to BoringSSL
-// NSS leaks in CertDatabaseNSSTest tests. http://crbug.com/51988
-"leak:net::NSSCertDatabase::ImportFromPKCS12\n"
-"leak:net::NSSCertDatabase::ListCerts\n"
-"leak:net::NSSCertDatabase::DeleteCertAndKey\n"
-"leak:crypto::ScopedTestNSSDB::ScopedTestNSSDB\n"
-// Another leak due to not shutting down NSS properly. http://crbug.com/124445
-"leak:error_get_my_stack\n"
-// The NSS suppressions above will not fire when the fast stack unwinder is
-// used, because it can't unwind through NSS libraries. Apply blanket
-// suppressions for now.
-"leak:libnssutil3\n"
-"leak:libnspr4\n"
-"leak:libnss3\n"
-"leak:libplds4\n"
-"leak:libnssckbi\n"
-
-// XRandR has several one time leaks.
-"leak:libxrandr\n"
-
-// xrandr leak. http://crbug.com/119677
-"leak:XRRFindDisplay\n"
-
-// http://crbug.com/431213, http://crbug.com/416665
-"leak:gin/object_template_builder.h\n"
-
-// Leaks in swrast_dri.so. http://crbug.com/540042
-"leak:swrast_dri.so\n"
-
-// Leak in glibc's gconv caused by fopen(..., "r,ccs=UNICODE")
-"leak:__gconv_lookup_cache\n"
-
-// ================ Leaks in Chromium code ================
-// PLEASE DO NOT ADD SUPPRESSIONS FOR NEW LEAKS.
-// Instead, commits that introduce memory leaks should be reverted. Suppressing
-// the leak is acceptable in some cases when reverting is impossible, i.e. when
-// enabling leak detection for the first time for a test target with
-// pre-existing leaks.
-
-// Small test-only leak in ppapi_unittests. http://crbug.com/258113
-"leak:ppapi::proxy::PPP_Instance_Private_ProxyTest_PPPInstancePrivate_Test\n"
-
-// http://crbug.com/322671
-"leak:content::SpeechRecognitionBrowserTest::SetUpOnMainThread\n"
-
-// http://crbug.com/355641
-"leak:TrayAccessibilityTest\n"
-
-// http://crbug.com/354644
-"leak:CertificateViewerUITest::ShowModalCertificateViewer\n"
-
-// http://crbug.com/356306
-"leak:content::SetProcessTitleFromCommandLine\n"
-
-// http://crbug.com/601435
-"leak:mojo/edk/js/handle.h\n"
-
-// PLEASE READ ABOVE BEFORE ADDING NEW SUPPRESSIONS.
-
-// End of suppressions.
-; // Please keep this semicolon.
+ // Intentional leak used as sanity test for Valgrind/memcheck.
+ "leak:base::ToolsSanityTest_MemoryLeak_Test::TestBody\n"
+
+ // ================ Leaks in third-party code ================
+
+ // False positives in libfontconfig. http://crbug.com/39050
+ "leak:libfontconfig\n"
+ // eglibc-2.19/string/strdup.c creates false positive leak errors because of
+ // the same reason as crbug.com/39050. The leak error stack trace, when
+ // unwind on malloc, includes a call to libfontconfig. But the default stack
+ // trace is too short in leak sanitizer bot to make the libfontconfig
+ // suppression works. http://crbug.com/605286
+ "leak:__strdup\n"
+
+ // Leaks in Nvidia's libGL.
+ "leak:libGL.so\n"
+
+ // TODO(eugenis): revisit NSS suppressions after the switch to BoringSSL
+ // NSS leaks in CertDatabaseNSSTest tests. http://crbug.com/51988
+ "leak:net::NSSCertDatabase::ImportFromPKCS12\n"
+ "leak:net::NSSCertDatabase::ListCerts\n"
+ "leak:net::NSSCertDatabase::DeleteCertAndKey\n"
+ "leak:crypto::ScopedTestNSSDB::ScopedTestNSSDB\n"
+ // Another leak due to not shutting down NSS properly.
+ // http://crbug.com/124445
+ "leak:error_get_my_stack\n"
+ // The NSS suppressions above will not fire when the fast stack unwinder is
+ // used, because it can't unwind through NSS libraries. Apply blanket
+ // suppressions for now.
+ "leak:libnssutil3\n"
+ "leak:libnspr4\n"
+ "leak:libnss3\n"
+ "leak:libplds4\n"
+ "leak:libnssckbi\n"
+
+ // XRandR has several one time leaks.
+ "leak:libxrandr\n"
+
+ // xrandr leak. http://crbug.com/119677
+ "leak:XRRFindDisplay\n"
+
+ // http://crbug.com/431213, http://crbug.com/416665
+ "leak:gin/object_template_builder.h\n"
+
+ // Leaks in swrast_dri.so. http://crbug.com/540042
+ "leak:swrast_dri.so\n"
+
+ // Leak in glibc's gconv caused by fopen(..., "r,ccs=UNICODE")
+ "leak:__gconv_lookup_cache\n"
+
+ // ================ Leaks in Chromium code ================
+ // PLEASE DO NOT ADD SUPPRESSIONS FOR NEW LEAKS.
+ // Instead, commits that introduce memory leaks should be reverted.
+ // Suppressing the leak is acceptable in some cases when reverting is
+ // impossible, i.e. when enabling leak detection for the first time for a
+ // test target with pre-existing leaks.
+
+ // Small test-only leak in ppapi_unittests. http://crbug.com/258113
+ "leak:ppapi::proxy::PPP_Instance_Private_ProxyTest_PPPInstancePrivate_"
+ "Test\n"
+
+ // http://crbug.com/322671
+ "leak:content::SpeechRecognitionBrowserTest::SetUpOnMainThread\n"
+
+ // http://crbug.com/355641
+ "leak:TrayAccessibilityTest\n"
+
+ // http://crbug.com/354644
+ "leak:CertificateViewerUITest::ShowModalCertificateViewer\n"
+
+ // http://crbug.com/356306
+ "leak:service_manager::SetProcessTitleFromCommandLine\n"
+
+ // http://crbug.com/601435
+ "leak:mojo/edk/js/handle.h\n"
+
+ // PLEASE READ ABOVE BEFORE ADDING NEW SUPPRESSIONS.
+
+ // End of suppressions.
+ ; // Please keep this semicolon.
#endif // LEAK_SANITIZER
« no previous file with comments | « no previous file | content/DEPS » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698