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

Issue 7834039: Removed suppression for 95449. (Closed)

Created:
9 years, 3 months ago by Joao da Silva
Modified:
9 years, 3 months ago
CC:
chromium-reviews, Timur Iskhodzhanov, Alexander Potapenko, pam+watch_chromium.org, stuartmorgan+watch_chromium.org
Visibility:
Public.

Description

Removed suppression for 95449, fixed in http://src.chromium.org/viewvc/chrome?view=rev&revision=99703 BUG=95449 TEST=The memcheck shouldn't trigger again Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=99783

Patch Set 1 #

Patch Set 2 : Rebased #

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

Messages

Total messages: 4 (0 generated)
Joao da Silva
Please review, thanks.
9 years, 3 months ago (2011-09-06 11:25:53 UTC) #1
Nico
lgtm
9 years, 3 months ago (2011-09-06 14:20:57 UTC) #2
commit-bot: I haz the power
Can't apply patch for file tools/valgrind/memcheck/suppressions.txt. While running patch -p1 --forward --force; patching file tools/valgrind/memcheck/suppressions.txt ...
9 years, 3 months ago (2011-09-06 17:22:16 UTC) #3
commit-bot: I haz the power
9 years, 3 months ago (2011-09-06 19:09:58 UTC) #4
Change committed as 99783

Powered by Google App Engine
This is Rietveld 408576698