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

Issue 2819056: Suppress a valgrind leak. (Closed)

Created:
10 years, 5 months ago by Nico
Modified:
9 years, 7 months ago
CC:
chromium-reviews, Timur Iskhodzhanov, stuartmorgan, pam+watch_chromium.org, Alexander Potapenko
Visibility:
Public.

Description

Suppress a valgrind leak. BUG=46886 TEST=net_tests goes green on memory waterfall. TBR=phajdan,willchan Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=52994

Patch Set 1 #

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

Messages

Total messages: 5 (0 generated)
Nico
10 years, 5 months ago (2010-07-20 03:53:41 UTC) #1
Lei Zhang
This is the only bot I see failing in net right now: http://build.chromium.org/buildbot/memory/builders/Chromium%20Mac%20(valgrind) and that's ...
10 years, 5 months ago (2010-07-20 04:12:54 UTC) #2
Paweł Hajdan Jr.
OK Thanks!
10 years, 5 months ago (2010-07-20 04:14:13 UTC) #3
Nico
On Mon, Jul 19, 2010 at 9:12 PM, <thestig@chromium.org> wrote: > This is the only ...
10 years, 5 months ago (2010-07-20 14:45:05 UTC) #4
Nico
10 years, 5 months ago (2010-07-20 14:49:10 UTC) #5
On Tue, Jul 20, 2010 at 7:44 AM, Nico Weber <thakis@chromium.org> wrote:
> On Mon, Jul 19, 2010 at 9:12 PM,  <thestig@chromium.org> wrote:
>> This is the only bot I see failing in net right now:
>>
>>
http://build.chromium.org/buildbot/memory/builders/Chromium%20Mac%20(valgrind)
>>
>> and that's not the right suppression for that bot.
>
> I grabbed the suppression from
>
http://build.chromium.org/buildbot/memory/builders/Chromium%20Mac%20(valgrind...

I was about to say "And net is green now", but that's because of
http://codereview.chromium.org/3015010 :-P

Thanks!

Powered by Google App Engine
This is Rietveld 408576698