[chromium]: Annotate StringImpl::createStatic leak for LeakSanitizer.
Changes to enable annotating known leaks in blink so that leak sanitizer doesn't raise alarm. Removing addition of StringImpl::createStatic from tools/lsan/suppressions.txt This patch currently just handles Leak Sanitizer only. The changes for Heap Checker would be done in a follow up patch. BUG=268258 TEST=Running content_unittests with Leak Sanitizer enabled. Output of Leak Sanitizer before the fix: http://pastebin.com/niDgRNAQ Output of Leak Sanitizer after the fix: http://pastebin.com/v6hF43Lj Review URL: https://chromiumcodereview.appspot.com/23460014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@221586 0039d316-1c4b-4281-b951-d872f2087c98
This commit is contained in:
1
AUTHORS
1
AUTHORS
@ -213,6 +213,7 @@ Ramkumar Gokarnesan <ramkumar.gokarnesan@gmail.com>
|
||||
Ramkumar Ramachandra <artagnon@gmail.com>
|
||||
Randy Posynick <randy.posynick@gmail.com>
|
||||
Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com>
|
||||
Ravi Phaneendra Kasibhatla <r.kasibhatla@samsung.com>
|
||||
Ravi Phaneendra Kasibhatla <ravi.kasibhatla@motorola.com>
|
||||
Rene Bolldorf <rb@radix.io>
|
||||
Rijubrata Bhaumik <rijubrata.bhaumik@intel.com>
|
||||
|
@ -3283,6 +3283,7 @@
|
||||
],
|
||||
'defines': [
|
||||
'LEAK_SANITIZER',
|
||||
'WTF_USE_LEAK_SANITIZER=1',
|
||||
],
|
||||
}],
|
||||
],
|
||||
|
@ -4,9 +4,6 @@ leak:base::ToolsSanityTest_MemoryLeak_Test::TestBody
|
||||
# False positives in libfontconfig. http://crbug.com/39050
|
||||
leak:libfontconfig
|
||||
|
||||
# Intentionally leaked "static" strings in Blink. http://crbug.com/268258
|
||||
leak:WTF::StringImpl::createStatic
|
||||
|
||||
# V8 leak, expect 1024 bytes leaked per process. http://crbug.com/99304
|
||||
leak:v8::internal::V8::Initialize
|
||||
|
||||
|
Reference in New Issue
Block a user