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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 435323002: Valgrind: Remove a few obsolete suppressions. (Closed)

Created:
6 years, 4 months ago by Lei Zhang
Modified:
6 years, 4 months ago
Reviewers:
groby-ooo-7-16
CC:
chromium-reviews, glider+watch_chromium.org, timurrrr+watch_chromium.org, bruening+watch_chromium.org
Project:
chromium
Visibility:
Public.

Description

Valgrind: Remove a few obsolete suppressions. BUG=30346, 30633, 31985, 124500 NOTRY=true TBR=groby@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=287431

Patch Set 1 #

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

Messages

Total messages: 6 (0 generated)
Lei Zhang
6 years, 4 months ago (2014-08-04 23:19:25 UTC) #1
Lei Zhang
The CQ bit was checked by thestig@chromium.org
6 years, 4 months ago (2014-08-04 23:19:28 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thestig@chromium.org/435323002/1
6 years, 4 months ago (2014-08-04 23:21:26 UTC) #3
Lei Zhang
The CQ bit was unchecked by thestig@chromium.org
6 years, 4 months ago (2014-08-05 02:05:32 UTC) #4
Lei Zhang
Committed patchset #1 manually as r287431 (tree was closed).
6 years, 4 months ago (2014-08-05 02:05:45 UTC) #5
groby-ooo-7-16
6 years, 4 months ago (2014-08-05 18:02:35 UTC) #6
Message was sent while issue was closed. 
LGTM

Powered by Google App Engine
This is Rietveld 408576698