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

Issue 6627037: Suppress many Valgrind reports on the memory waterfall... (Closed)

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

Description

Suppress many Valgrind reports on the memory waterfall TBR=glider BUG=75019, 75020, 75021, 75023 TEST=Memory waterfall goes green Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=77040

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+77 lines, -2 lines) Patch
M tools/valgrind/memcheck/suppressions.txt View 3 chunks +77 lines, -2 lines 1 comment Download

Messages

Total messages: 2 (0 generated)
Timur Iskhodzhanov
9 years, 9 months ago (2011-03-05 12:14:33 UTC) #1
Alexander Potapenko
9 years, 9 months ago (2011-03-05 12:17:24 UTC) #2
LGTM

http://codereview.chromium.org/6627037/diff/1/tools/valgrind/memcheck/suppres...
File tools/valgrind/memcheck/suppressions.txt (right):

http://codereview.chromium.org/6627037/diff/1/tools/valgrind/memcheck/suppres...
tools/valgrind/memcheck/suppressions.txt:825:
fun:_ZN1}RenderViewHost16CreateRenderViewERKSbItN4base20string16_char_traitsESaItEE
s/}/4
Please fix.

Powered by Google App Engine
This is Rietveld 408576698