Chromium Code Reviews
Help | Chromium Project | Sign in
(45)

Issue 3192009: Add Valgrind suppressions for CreateFullscreenWidget.... (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 9 months ago by Lei Zhang
Modified:
4 years ago
Reviewers:
stuartmorgan, TVL
CC:
chromium-reviews, Timur Iskhodzhanov, Alexander Potapenko, pam+watch_chromium.org, stuartmorgan+watch_chromium.org
Visibility:
Public.

Description

Add Valgrind suppressions for CreateFullscreenWidget. BUG=52900 TEST=Valgrind Linux gets greener. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=57258

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -0 lines) Patch
M tools/valgrind/memcheck/suppressions.txt View 1 chunk +11 lines, -0 lines 0 comments Download
Trybot results:
Commit: CQ not working?

Messages

Total messages: 2 (0 generated)
Lei Zhang
Same leak already suppressed in the heapchecker. I'll take a LGTM from anyone really.
4 years, 9 months ago (2010-08-23 22:00:38 UTC) #1
TVL
4 years, 9 months ago (2010-08-23 22:05:42 UTC) #2
lg
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld ec887be