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

Issue 9309038: Additional suppressions for bug_112315 v8 memory leak (Closed)

Created:
8 years, 10 months ago by Brad Chen
Modified:
8 years, 10 months ago
Reviewers:
danno
CC:
chromium-reviews, Timur Iskhodzhanov, Alexander Potapenko, pam+watch_chromium.org, stuartmorgan+watch_chromium.org
Visibility:
Public.

Description

Additional suppressions for bug_112315 v8 memory leak TBR=danno@chromium.org BUG=112315 TEST=Chromium OS Memcheck bots Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=120082

Patch Set 1 #

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

Messages

Total messages: 2 (0 generated)
Brad Chen
8 years, 10 months ago (2012-02-01 18:53:53 UTC) #1
Alexander Potapenko
8 years, 10 months ago (2012-02-17 13:16:03 UTC) #2
Brad,

you don't need the ellipsis at the end of the suppression stack,
because a suppression is applied if it matches the top of the stack,
and the ellipsis stands for "match any number of lines from here"

Powered by Google App Engine
This is Rietveld 408576698