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

Issue 219021: Removing suppressions that have not triggered in the last 100 runs. (Closed)

Created:
11 years, 3 months ago by mattm
Modified:
9 years, 7 months ago
Reviewers:
James Hawkins
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Removing valgrind suppressions that have not triggered in the last 100 runs. (Doesn't cover suppressions_mac.txt) Some suppressions didn't trigger because the test was disabled. It is assumed that before re-enabling such a test someone would fix it and check that it doesn't still need any suppressions: BUG=6522 BUG=11838 Not reproducible, bug is already closed: BUG=10750 BUG=11117 BUG=12539, 12549, 12661 Not reproducible (or not enough info to try): BUG=4609 BUG=16103 BUG=16129 (probably fixed by r20100) BUG=16578 BUG=16584 BUG=16586 BUG=16625 BUG=16908 BUG=18654 BUG=19385 BUG=19490 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=27168

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -313 lines) Patch
M chrome/browser/importer/toolbar_importer_unittest.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/tab_restore_uitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/data/purify/unit_tests.exe.gtest.txt View 1 chunk +0 lines, -3 lines 0 comments Download
M tools/valgrind/memcheck/suppressions.txt View 11 chunks +0 lines, -309 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
mattm
11 years, 3 months ago (2009-09-23 22:34:54 UTC) #1
James Hawkins
11 years, 3 months ago (2009-09-23 22:36:02 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698