New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 787746 link

Starred by 1 user
Status: WontFix
Owner:
sdy@chromium.org
Closed: Nov 2017
Cc:
rsesek@chromium.org
█ msrchandra@chromium.org
mac-bugs-priority@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 1
Type: Bug



Sign in to add a comment

[Missing Test]: Possible MD downloads memory leak?

Project Member Reported by msrchandra@chromium.org, Nov 22 2017 
Automated tests for the below commit have been missing.Please add test coverage ASAP to avoid regressions in future.

CL :: 
======
https://chromium.googlesource.com/chromium/src.git/+/03cf015b3fe853f59ce609cb643e4647ebc0fe80

Reference Bug ::
=================
https://bugs.chromium.org/p/chromium/issues/detail?id=787571

Thank You.

Comment 1 by sdy@chromium.org, Nov 29 2017

Cc: rsesek@chromium.org
Status: WontFix (was: Assigned)
I don’t think this is really testable except as part of more general memory usage/leak testing. +rsesek@ for a second opinion, but also closing.

Comment 2 by rsesek@chromium.org, Nov 29 2017 

We do have ASan bots that can and do catch leaks: https://ci.chromium.org/buildbot/chromium.memory/Mac%20ASan%2064%20Tests%20%281%29/

If a unittest exercised the code with a leak, it should be reported.
►

Sign in to add a comment