From fe6064373414f01d180a84b2d3473d5032523174 Mon Sep 17 00:00:00 2001 From: "jam@chromium.org" Date: Sun, 1 Mar 2009 01:14:31 +0000 Subject: Revert my previous change. It fixed the memory leaks (see http://build.chromium.org/buildbot/waterfall/builders/XP%20Unit%20(purify)/builds/2378/steps/purify%20test:%20unit/logs/stdio), but DeleteURLAndFavicon started failing mysteriously after my checkin. Review URL: http://codereview.chromium.org/28304 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10688 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/renderer/render_thread_unittest.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'chrome/renderer/render_thread_unittest.cc') diff --git a/chrome/renderer/render_thread_unittest.cc b/chrome/renderer/render_thread_unittest.cc index 808bedf..3d09c51 100644 --- a/chrome/renderer/render_thread_unittest.cc +++ b/chrome/renderer/render_thread_unittest.cc @@ -24,13 +24,13 @@ class RenderThreadTest : public testing::Test { virtual void TearDown() { message_loop_.RunAllPending(); mock_process_.reset(); - // Need to fully destruct IPC::SyncChannel before the message loop goes - // away. - message_loop_.RunAllPending(); // Delete the server channel after the RenderThread so that // IPC::SyncChannel's OnChannelError doesn't fire on the context and attempt // to use the listener thread which is now gone. delete channel_; + // Need to fully destruct IPC::SyncChannel before the message loop goes + // away. + message_loop_.RunAllPending(); } protected: -- cgit v1.1