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

Issue 7497053: Add suppression for valgrind failures in PhishingTermFeatureExtractorTest. (Closed)

Created:
9 years, 4 months ago by Garrett Casto
Modified:
9 years, 4 months ago
Reviewers:
dhollowa, Brian Ryner
CC:
noelutz, chromium-reviews, Timur Iskhodzhanov, Alexander Potapenko, pam+watch_chromium.org, stuartmorgan+watch_chromium.org
Visibility:
Public.

Description

Add suppression for valgrind failures in PhishingTermFeatureExtractorTest. BUG=92267 TEST=None Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=96113

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+52 lines, -0 lines) Patch
M tools/valgrind/memcheck/suppressions.txt View 1 chunk +52 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Garrett Casto
9 years, 4 months ago (2011-08-10 01:50:54 UTC) #1
Brian Ryner
LGTM
9 years, 4 months ago (2011-08-10 01:54:59 UTC) #2
dhollowa
9 years, 4 months ago (2011-08-10 03:57:46 UTC) #3
It looks like additional suppressions are needed for Mac and CrOS bots.

Powered by Google App Engine
This is Rietveld 408576698