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

Unified Diff: third_party/WebKit/LayoutTests/LeakExpectations

Issue 2277663004: Add a caution to LeakExpectations (Closed)
Patch Set: Created 4 years, 4 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: third_party/WebKit/LayoutTests/LeakExpectations
diff --git a/third_party/WebKit/LayoutTests/LeakExpectations b/third_party/WebKit/LayoutTests/LeakExpectations
index f18d367ca0e32e8068466c833d57bf96be7d57b9..7b3940b646c526aee6bf31252595d47b13f58cc2 100644
--- a/third_party/WebKit/LayoutTests/LeakExpectations
+++ b/third_party/WebKit/LayoutTests/LeakExpectations
@@ -1,9 +1,11 @@
# This file is used for tests that only need to be suppressed on the Chromium
# WebKit Leak bots.
-#
-# This list isn't comprehensive yet, and Leak Detector bots are not tree closers atm.
-# Gardeners are welcome to touch this file, but you can ignore the failures
-# and wait for kouhei@ or hajimehoshi@ to maintain this file.
+
+###########################################################################
+# WARNING: Memory leaks must be fixed asap. Sheriff is expected to revert #
+# culprit CLs instead of suppressing the leaks. If you have any question, #
+# ask hajimehoshi@. #
+###########################################################################
# view-source:// tests are not supported by the leak detector
crbug.com/364398 fast/frames/viewsource/viewsource-1.html [ Leak ]
@@ -119,3 +121,9 @@ crbug.com/619060 [ Linux ] imported/wpt/pointerevents/pointerevent_pointerleave_
# Leaks that must be fixed asap.
crbug.com/640886 [ Linux ] http/tests/images/png-partial-load-as-document.html is leaking [ Leak ]
+
+###########################################################################
+# WARNING: Memory leaks must be fixed asap. Sheriff is expected to revert #
+# culprit CLs instead of suppressing the leaks. If you have any question, #
+# ask hajimehoshi@. #
+###########################################################################
« 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