Port unload_uitest.cc and enable some of the tests on linux.

Make WaitForSingleProcess respect base::kNoTimeout (which is equivalent to INFINITE on windows).

Review URL: http://codereview.chromium.org/99008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14508 0039d316-1c4b-4281-b951-d872f2087c98

3 files changed, 25 insertions, 10 deletions

diff --git a/base/process_util_posix.cc b/base/process_util_posix.cc
index 562b8e9..2e3fd55 100644
--- a/base/process_util_posix.cc
+++ b/base/process_util_posix.cc
@@ -22,6 +22,7 @@
 #include "base/scoped_ptr.h"
 #include "base/sys_info.h"
 #include "base/time.h"
+#include "base/waitable_event.h"
 
 const int kMicrosecondsPerSecond = 1000000;
 
@@ -207,7 +208,7 @@ int WaitpidWithTimeout(ProcessHandle handle, int wait_milliseconds,
   // has been installed.  This means that when a SIGCHLD is sent, it will exit
   // depending on behavior external to this function.
   //
-  // This function is used primarilly for unit tests, if we want to use it in
+  // This function is used primarily for unit tests, if we want to use it in
   // the application itself it would probably be best to examine other routes.
   int status = -1;
   pid_t ret_pid = waitpid(handle, &status, WNOHANG);
@@ -243,7 +244,11 @@ int WaitpidWithTimeout(ProcessHandle handle, int wait_milliseconds,
 
 bool WaitForSingleProcess(ProcessHandle handle, int wait_milliseconds) {
   bool waitpid_success;
-  int status = WaitpidWithTimeout(handle, wait_milliseconds, &waitpid_success);
+  int status;
+  if (wait_milliseconds == base::kNoTimeout)
+    waitpid_success = (waitpid(handle, &status, 0) != -1);
+  else
+    status = WaitpidWithTimeout(handle, wait_milliseconds, &waitpid_success);
   if (status != -1) {
     DCHECK(waitpid_success);
     return WIFEXITED(status);
diff --git a/chrome/browser/unload_uitest.cc b/chrome/browser/unload_uitest.cc
index b991ebe..10a3a4a 100644
--- a/chrome/browser/unload_uitest.cc
+++ b/chrome/browser/unload_uitest.cc
@@ -3,12 +3,12 @@
 // found in the LICENSE file.
 
 #include "base/file_util.h"
-
+#include "base/platform_thread.h"
 #include "chrome/browser/automation/url_request_mock_http_job.h"
 #include "chrome/common/chrome_switches.h"
+#include "chrome/common/message_box_flags.h"
 #include "chrome/test/automation/browser_proxy.h"
 #include "chrome/test/ui/ui_test.h"
-#include "chrome/views/window/dialog_delegate.h"
 #include "net/url_request/url_request_unittest.h"
 
 const std::string NOLISTENERS_HTML =
@@ -78,7 +78,7 @@ class UnloadTest : public UITest {
     int max_wait_time = 5000;
     while (max_wait_time > 0) {
       max_wait_time -= kCheckDelayMs;
-      Sleep(kCheckDelayMs);
+      PlatformThread::Sleep(kCheckDelayMs);
       if (!IsBrowserRunning())
         break;
     }
@@ -89,7 +89,7 @@ class UnloadTest : public UITest {
     int max_wait_time = 5000;
     while (max_wait_time > 0) {
       max_wait_time -= kCheckDelayMs;
-      Sleep(kCheckDelayMs);
+      PlatformThread::Sleep(kCheckDelayMs);
       if (expected_title == GetActiveTabTitle())
         break;
     }
@@ -118,10 +118,10 @@ class UnloadTest : public UITest {
   void NavigateToNolistenersFileTwiceAsync() {
     // TODO(ojan): We hit a DCHECK in RenderViewHost::OnMsgShouldCloseACK
     // if we don't sleep here.
-    Sleep(400);
+    PlatformThread::Sleep(400);
     NavigateToURLAsync(
         URLRequestMockHTTPJob::GetMockUrl(L"title2.html"));
-    Sleep(400);
+    PlatformThread::Sleep(400);
     NavigateToURL(
         URLRequestMockHTTPJob::GetMockUrl(L"title2.html"));
 
@@ -137,17 +137,26 @@ class UnloadTest : public UITest {
   }
 
   void ClickModalDialogButton(MessageBoxFlags::DialogButton button) {
+#if defined(OS_WIN)
     bool modal_dialog_showing = false;
     MessageBoxFlags::DialogButton available_buttons;
     EXPECT_TRUE(automation()->WaitForAppModalDialog(3000));
     EXPECT_TRUE(automation()->GetShowingAppModalDialog(&modal_dialog_showing,
         &available_buttons));
     ASSERT_TRUE(modal_dialog_showing);
-    EXPECT_TRUE((button & available_buttons) != NULL);
+    EXPECT_TRUE((button & available_buttons) != 0);
     EXPECT_TRUE(automation()->ClickAppModalDialogButton(button));
+#else
+    // TODO(port): port this function if and when the tests that use it are
+    // enabled (currently they are not being run even on windows).
+    NOTIMPLEMENTED();
+#endif
   }
 };
 
+// TODO(port): these tests fail on linux because they leave a renderer process
+// lying around which holds onto the user data directory.
+#if defined(OS_WIN)
 // Navigate to a page with an infinite unload handler.
 // Then two two async crosssite requests to ensure
 // we don't get confused and think we're closing the tab.
@@ -203,6 +212,7 @@ TEST_F(UnloadTest, CrossSiteInfiniteBeforeUnloadSync) {
   NavigateToNolistenersFileTwice();
   ASSERT_TRUE(IsBrowserRunning());
 }
+#endif
 
 // Tests closing the browser on a page with no unload listeners registered.
 TEST_F(UnloadTest, BrowserCloseNoUnloadListeners) {
diff --git a/chrome/chrome.gyp b/chrome/chrome.gyp
index 8ccab4d..c88046e 100644
--- a/chrome/chrome.gyp
+++ b/chrome/chrome.gyp
@@ -2078,6 +2078,7 @@
             'browser/metrics/metrics_service_uitest.cc',
             'browser/sessions/session_restore_uitest.cc',
             'browser/tab_restore_uitest.cc',
+            'browser/unload_uitest.cc',
             'test/reliability/page_load_test.cc',
             'test/ui/layout_plugin_uitest.cc',
             'test/ui/omnibox_uitest.cc',
@@ -2098,7 +2099,6 @@
             # TODO(port)?  (Most of these include windows.h or similar.)
             'browser/printing/printing_layout_uitest.cc',
             'browser/ssl/ssl_uitest.cc',
-            'browser/unload_uitest.cc',
             'browser/views/find_bar_win_uitest.cc',
             'common/logging_chrome_uitest.cc',
             'test/accessibility/accessibility_tests.cc',