Fix obvious leak in chrome_web_ui_data_source_unittest.cc, and revert associated suppresions.

Fix obvious leak in chrome_web_ui_data_source_unittest.cc, and revert associated suppresions.

BUG=88094
TEST=heapchecks clean about ChromeWebUIDataSource
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=91303

[Tom Sepez, 2011-06-30 22:35:26]

On 2011/06/30 22:34:58, Tom Sepez wrote:

Is there a good way to test that this fixes all the issues prior to commit? 
Thanks.

[Tom Sepez, 2011-06-30 23:08:39]

http://build.chromium.org/p/tryserver.chromium/builders/linux_valgrind/builds...

In particular, this try job seems useless.  Does it need a --testfilter on the
try command?

[Tom Sepez, 2011-06-30 23:11:23]

Nevermind.  Test names are different between valgrind and normal.

linux (not valgrind) try job:
http://build.chromium.org/p/tryserver.chromium/builders/linux/builds/35000

[oshima, 2011-06-30 23:15:46]

LGTM

thanks!

On Thu, Jun 30, 2011 at 4:11 PM, <tsepez@chromium.org> wrote:

> Nevermind.  Test names are different between valgrind and normal.
>
> linux (not valgrind) try job:
> http://build.chromium.org/p/**tryserver.chromium/builders/**
>
linux/builds/35000<http://build.chromium.org/p/tryserver.chromium/builders/linux/builds/35000>
>
>
>
http://codereview.chromium.**org/7278008/<http://codereview.chromium.org/7278...
>

[Tom Sepez, 2011-06-30 23:28:44]

http://build.chromium.org/p/tryserver.chromium/builders/mac/builds/35557

http://build.chromium.org/p/tryserver.chromium/builders/win/builds/40291

green unit-test only runs on other platforms

[Tom Sepez, 2011-07-01 00:14:32]

http://build.chromium.org/p/tryserver.chromium/builders/linux_chromeos/builds...

green unit-test only run on chromeos