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

Issue 6268009: Add suppression for leak in gfx::CanvasSkia::SizeStringInt (Closed)

Created:
9 years, 11 months ago by Ilya Sherman
Modified:
9 years, 7 months ago
Reviewers:
James Hawkins
CC:
chromium-reviews, pam+watch_chromium.org
Visibility:
Public.

Description

Add suppression for leak in gfx::CanvasSkia::SizeStringInt BUG=70327 TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=72098

Patch Set 1 #

Total comments: 1

Patch Set 2 : Less is more #

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

Messages

Total messages: 2 (0 generated)
James Hawkins
http://codereview.chromium.org/6268009/diff/1/tools/heapcheck/suppressions.txt File tools/heapcheck/suppressions.txt (right): http://codereview.chromium.org/6268009/diff/1/tools/heapcheck/suppressions.txt#newcode1221 tools/heapcheck/suppressions.txt:1221: fun:ui::TextEliderTest_TestGeneralEliding_Test::TestBody Remove the stack below this line.
9 years, 11 months ago (2011-01-21 02:33:57 UTC) #1
James Hawkins
9 years, 11 months ago (2011-01-21 02:44:49 UTC) #2
LGTM after that though.

Powered by Google App Engine
This is Rietveld 408576698