Chromium Code Reviews Chromium Code Reviews
Issue 603243005:
Add an LSAN suppression for gin/object_template_builder.h. (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| OLD | NEW |
|---|---|
| 1 # HeapCheck sanity test | 1 # HeapCheck sanity test |
| 2 leak:base::ToolsSanityTest_MemoryLeak_Test::TestBody | 2 leak:base::ToolsSanityTest_MemoryLeak_Test::TestBody |
| 3 | 3 |
| 4 #### Third-party leaks #### | 4 #### Third-party leaks #### |
| 5 | 5 |
| 6 # False positives in libfontconfig. http://crbug.com/39050 | 6 # False positives in libfontconfig. http://crbug.com/39050 |
| 7 leak:libfontconfig | 7 leak:libfontconfig |
| 8 | 8 |
| 9 # Leaks in Nvidia's libGL. | 9 # Leaks in Nvidia's libGL. |
| 10 leak:libGL.so | 10 leak:libGL.so |
| (...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 49 # Those should only fire in (browser)tests. If you see one of them in Chrome, | 49 # Those should only fire in (browser)tests. If you see one of them in Chrome, |
| 50 # then it's a real leak. | 50 # then it's a real leak. |
| 51 # http://crbug.com/328552 | 51 # http://crbug.com/328552 |
| 52 leak:WTF::StringImpl::createUninitialized | 52 leak:WTF::StringImpl::createUninitialized |
| 53 leak:WTF::StringImpl::create8BitIfPossible | 53 leak:WTF::StringImpl::create8BitIfPossible |
| 54 leak:blink::MouseEvent::create | 54 leak:blink::MouseEvent::create |
| 55 leak:blink::WindowProxy::initializeIfNeeded | 55 leak:blink::WindowProxy::initializeIfNeeded |
| 56 leak:blink::*::*GetterCallback | 56 leak:blink::*::*GetterCallback |
| 57 leak:blink::CSSComputedStyleDeclaration::create | 57 leak:blink::CSSComputedStyleDeclaration::create |
| 58 leak:blink::V8PerIsolateData::ensureDomInJSContext | 58 leak:blink::V8PerIsolateData::ensureDomInJSContext |
| 59 leak:CreateTemplate | |
|
Yoyo Zhou
2014/09/30 01:21:54
I think we can do leak:gin/object_template_builder
Sam McNally
2014/09/30 01:28:20
Done.
| |
| 59 | 60 |
| 60 # http://crbug.com/356785 | 61 # http://crbug.com/356785 |
| 61 leak:content::RenderViewImplTest_DecideNavigationPolicyForWebUI_Test::TestBody | 62 leak:content::RenderViewImplTest_DecideNavigationPolicyForWebUI_Test::TestBody |
| 62 | 63 |
| 63 #### Actual bugs in Chromium code #### | 64 #### Actual bugs in Chromium code #### |
| 64 # PLEASE DO NOT ADD SUPPRESSIONS FOR NEW LEAKS. | 65 # PLEASE DO NOT ADD SUPPRESSIONS FOR NEW LEAKS. |
| 65 # Instead, commits that introduce memory leaks should be reverted. Suppressing | 66 # Instead, commits that introduce memory leaks should be reverted. Suppressing |
| 66 # the leak is acceptable in some cases when reverting is impossible, i.e. when | 67 # the leak is acceptable in some cases when reverting is impossible, i.e. when |
| 67 # enabling leak detection for the first time for a test target with pre-existing | 68 # enabling leak detection for the first time for a test target with pre-existing |
| 68 # leaks. | 69 # leaks. |
| (...skipping 10 matching lines...) Expand all Loading... | |
| 79 # http://crbug.com/355641 | 80 # http://crbug.com/355641 |
| 80 leak:TrayAccessibilityTest | 81 leak:TrayAccessibilityTest |
| 81 | 82 |
| 82 # http://crbug.com/354644 | 83 # http://crbug.com/354644 |
| 83 leak:CertificateViewerUITest::ShowModalCertificateViewer | 84 leak:CertificateViewerUITest::ShowModalCertificateViewer |
| 84 | 85 |
| 85 # http://crbug.com/356306 | 86 # http://crbug.com/356306 |
| 86 leak:content::SetProcessTitleFromCommandLine | 87 leak:content::SetProcessTitleFromCommandLine |
| 87 | 88 |
| 88 # PLEASE READ ABOVE BEFORE ADDING NEW SUPPRESSIONS. | 89 # PLEASE READ ABOVE BEFORE ADDING NEW SUPPRESSIONS. |
| OLD | NEW |
