diff options
author | levin@chromium.org <levin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-12-10 07:39:12 +0000 |
---|---|---|
committer | levin@chromium.org <levin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-12-10 07:39:12 +0000 |
commit | 7dbfefe01d85c8cc6c34d4d04a96029bf2c5ddfa (patch) | |
tree | 3cf00391912e898ce1f2f06d152ca23a1c76eb6e /chrome/worker | |
parent | 112bd4674a886777b79dcd03263088b692fb47fd (diff) | |
download | chromium_src-7dbfefe01d85c8cc6c34d4d04a96029bf2c5ddfa.zip chromium_src-7dbfefe01d85c8cc6c34d4d04a96029bf2c5ddfa.tar.gz chromium_src-7dbfefe01d85c8cc6c34d4d04a96029bf2c5ddfa.tar.bz2 |
Fix a memory leak in the worker tests.
Clean-up suppressions: remove ones that appear to have been fixed,
clarify another to be a known leak by design.
BUG=26966,22923
TEST=valgrind tests (for ui and layout tests)
Review URL: http://codereview.chromium.org/465075
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@34239 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/worker')
-rw-r--r-- | chrome/worker/worker_uitest.cc | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/chrome/worker/worker_uitest.cc b/chrome/worker/worker_uitest.cc index b5571b0..8bb903d 100644 --- a/chrome/worker/worker_uitest.cc +++ b/chrome/worker/worker_uitest.cc @@ -146,6 +146,12 @@ TEST_F(WorkerTest, WorkerFastLayoutTests) { for (size_t i = 0; i < arraysize(kLayoutTestFiles); ++i) RunLayoutTest(kLayoutTestFiles[i], false); + + // Navigate away from to a blank page so that any workers are cleaned up. This + // helps leaks trackers do a better job of reporting. + scoped_refptr<TabProxy> tab(GetActiveTab()); + GURL about_url(std::string("file://localhost/")); + ASSERT_EQ(AUTOMATION_MSG_NAVIGATION_SUCCESS, tab->NavigateToURL(about_url)); } #if defined(OS_WIN) || defined(OS_MACOSX) |