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

Issue 7134062: Insert suppression for memory leaks. (Closed)

Created:
9 years, 6 months ago by Randy Smith (Not in Mondays)
Modified:
9 years, 6 months ago
Reviewers:
jhawkins
CC:
chromium-reviews, Timur Iskhodzhanov, Alexander Potapenko, pam+watch_chromium.org, stuartmorgan+watch_chromium.org
Visibility:
Public.

Description

Insert suppression for memory leaks. BUG=58449 TBR=jhawkins Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=88624

Patch Set 1 #

Patch Set 2 : Put back in suppression I removed too quickly. #

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

Messages

Total messages: 1 (0 generated)
Randy Smith (Not in Mondays)
9 years, 6 months ago (2011-06-10 02:07:16 UTC) #1
Jay: Could you review this at your leisure (I'm certain enough I have the format
right that I'm just going to commit it).  The context here is that I was working
on bug 58449 and thought that it had gone away, so I removed the suppression,
but it hasn't, so I'm putting it back in.

Powered by Google App Engine
This is Rietveld 408576698