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

Issue 17287009: [Memory Sheriff] Widen FileStream suppression. (Closed)

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

Description

[Memory Sheriff] Widen FileStream suppression. NOTRY=true TBR=thestig@chromium.org, earthdok@chromium.org BUG=251004 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=207385

Patch Set 1 #

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

Messages

Total messages: 5 (0 generated)
groby-ooo-7-16
7 years, 6 months ago (2013-06-19 18:34:39 UTC) #1
earthdok
We already have this in heapcheck/suppressions.txt: { FileStream::Context can leak through WorkerPool by design (CrOS) ...
7 years, 6 months ago (2013-06-19 18:36:54 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/groby@chromium.org/17287009/1
7 years, 6 months ago (2013-06-19 18:44:41 UTC) #3
groby-ooo-7-16
As I said on the bug, I am not sure if that is indeed the ...
7 years, 6 months ago (2013-06-19 18:44:54 UTC) #4
commit-bot: I haz the power
7 years, 6 months ago (2013-06-20 10:08:34 UTC) #5
Message was sent while issue was closed.
Change committed as 207385

Powered by Google App Engine
This is Rietveld 408576698