Removed suppressions for http://crbug.com/35318 and http://crbug.com/58449.

Removed suppressions for http://crbug.com/35318 and http://crbug.com/58449.

BUG=35318
BUG=58449
R=phajdan.jr@chromium.org
TEST=None


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=88580

[Randy Smith (Not in Mondays), 2011-06-03 21:12:01]

Pawel, willing to give me an LGTM on this?  for 35318, I ran the browser tests
under valgrind with the suppression removed, and didn't get anything (not
surprising, since we converted from ui tests to browser tests since this was
created).  For 58449, I get a timeout when I try to run the ui tests.  It's a
memory leak of the DownloadManager itself, which is mostly likely explained by a
DownloadFile not being trashed or by a posted task being lost, both of which are
areas in which we've been playing, so I want to see if the problem comes back if
I remove the suppression.

[Paweł Hajdan Jr., 2011-06-04 09:02:41]

Sure, LGTM (I think that kind of change can be TBR-ed). Please make sure to run
this through valgrind trybots _first_ .

[Randy Smith (Not in Mondays), 2011-06-05 14:58:07]

On 2011/06/04 09:02:41, Paweł Hajdan Jr. wrote:
> Sure, LGTM (I think that kind of change can be TBR-ed). Please make sure to
run
> this through valgrind trybots _first_ .

Whoops; I don't do memory leak investigations often enough.  Thanks for the
reminder.

[commit-bot: I haz the power, 2011-06-07 21:43:04]

No LGTM from valid reviewers yet.

[M-A Ruel, 2011-06-08 01:55:11]

lgtm

The reviewer check is a whitelist based on a list generated automatically. Maybe
Pawel was left out of it. I'm LGTM'ing the change in the meantime so you can use
the commit queue if you want, I'll verify tomorrow.

[commit-bot: I haz the power, 2011-06-09 20:22:46]

Change committed as 88580