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 23902038: Valgrind: Fix up some more suppressions: (Closed)

Created:
7 years, 3 months ago by Lei Zhang
Modified:
7 years, 3 months ago
Reviewers:
Nico
CC:
chromium-reviews, glider+watch_chromium.org, timurrrr+watch_chromium.org, bruening+watch_chromium.org
Visibility:
Public.

Description

Valgrind: Fix up some more suppressions: - Move a WontFix leak to the intentional section, and update the suppression to match the only current instance of the leak. - Widen a suppression for an inspector memory error. - Add new suppression for a WebCore::Document::decoder NULL deref. BUG=73299, 288801, 293088 NOTRY=true TBR=thakis@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=223496

Patch Set 1 #

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

Messages

Total messages: 3 (0 generated)
Lei Zhang
TBR
7 years, 3 months ago (2013-09-17 00:59:30 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thestig@chromium.org/23902038/1
7 years, 3 months ago (2013-09-17 01:01:07 UTC) #2
commit-bot: I haz the power
7 years, 3 months ago (2013-09-17 01:03:56 UTC) #3
Message was sent while issue was closed.
Change committed as 223496

Powered by Google App Engine
This is Rietveld 408576698