From 2ce26c438a2cd219348eefa324a64c15d1bf8cf2 Mon Sep 17 00:00:00 2001 From: "phajdan.jr@chromium.org" Date: Mon, 19 Sep 2011 17:08:12 +0000 Subject: Wait properly for renderer crashes This replaces a Sleep in automation with a wait for renderer crash. It turns out that our IPC on POSIX had one loophole that caused it not to notice very early crashes, so I also fixed that. The problem was that when the child process died before connecting to the parent's IPC channel, the parent wouldn't notice the crash because the child end of the IPC pipe was kept open for too long. This change makes the code close the child end of the pipe right after forking the child. This might also help with automation not noticing the browser crash during initial launch, or at least should be a good step toward fixing that problem. BUG=38497,90489 Review URL: http://codereview.chromium.org/7870008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101760 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/test/automation/automation_proxy.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) (limited to 'chrome/test/automation/automation_proxy.h') diff --git a/chrome/test/automation/automation_proxy.h b/chrome/test/automation/automation_proxy.h index a41689d..0fccd90 100644 --- a/chrome/test/automation/automation_proxy.h +++ b/chrome/test/automation/automation_proxy.h @@ -222,9 +222,7 @@ class AutomationProxy : public IPC::Channel::Listener, const std::string& password) WARN_UNUSED_RESULT; #endif -#if defined(OS_POSIX) - base::file_handle_mapping_vector fds_to_map() const; -#endif + IPC::SyncChannel* channel(); // AutomationMessageSender implementation. virtual bool Send(IPC::Message* message) WARN_UNUSED_RESULT; -- cgit v1.1