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

Issue 111443009: Retire unused memcheck suppressions (Closed)

Created:
7 years ago by Eric Willigers
Modified:
6 years, 5 months ago
CC:
chromium-reviews, glider+watch_chromium.org, timurrrr+watch_chromium.org, bruening+watch_chromium.org, groby-ooo-7-16
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Visibility:
Public.

Description

Patch Set 1 #

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

Messages

Total messages: 9 (0 generated)
Eric Willigers
7 years ago (2013-12-18 05:24:26 UTC) #1
Lei Zhang
1) Can you put the bugs affected in the BUG= line? 2) You'll want NOTRY=true ...
7 years ago (2013-12-18 05:42:16 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/ericwilligers@chromium.org/111443009/1
7 years ago (2013-12-18 06:45:53 UTC) #3
commit-bot: I haz the power
Change committed as 241515
7 years ago (2013-12-18 06:46:11 UTC) #4
Eric Willigers
A revert of this CL has been created in https://codereview.chromium.org/118073002/ by ericwilligers@chromium.org. The reason for ...
7 years ago (2013-12-18 07:56:00 UTC) #5
Peter Kasting
Someone closing another bug pointed me at this CL. Was this ever relanded (partially or ...
6 years, 5 months ago (2014-07-02 18:16:29 UTC) #6
Eric Willigers
On 2014/07/02 18:16:29, Peter Kasting wrote: > Someone closing another bug pointed me at this ...
6 years, 5 months ago (2014-07-03 03:37:17 UTC) #7
Peter Kasting
On 2014/07/03 03:37:17, Eric Willigers wrote: > On 2014/07/02 18:16:29, Peter Kasting wrote: > > ...
6 years, 5 months ago (2014-07-03 04:56:57 UTC) #8
Eric Willigers
6 years, 5 months ago (2014-07-07 01:28:26 UTC) #9
Message was sent while issue was closed.
> OK.  Is there a bug tracking this?  I want to make sure this work actually
gets
> done :)

I have created bug https://code.google.com/p/chromium/issues/detail?id=391631

Powered by Google App Engine
This is Rietveld 408576698