Convert Windows Sleep to PlatformThread::Sleep...

chrome/browser/renderer_host/resource_dispatcher_host_uitest.cc

-// Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
+// Copyright (c) 2006-2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <sstream>
 #include <string>
 
 #include "base/command_line.h"
 #include "base/file_util.h"
 #include "base/path_service.h"
 #include "base/string_util.h"
@@ skipping 10 matching lines @@
 
 class ResourceDispatcherTest : public UITest {
  public:
   void CheckTitleTest(const std::wstring& file,
                       const std::wstring& expected_title) {
     NavigateToURL(URLRequestMockHTTPJob::GetMockUrl(file));
     const int kCheckDelayMs = 100;
     int max_wait_time = 5000;
     while (max_wait_time > 0) {
       max_wait_time -= kCheckDelayMs;
-      Sleep(kCheckDelayMs);
+      PlatformThread::Sleep(kCheckDelayMs);
       if (expected_title == GetActiveTabTitle())
         break;
     }
 
     EXPECT_EQ(expected_title, GetActiveTabTitle());
   }
 
  protected:
   ResourceDispatcherTest() : UITest() {
     dom_automation_enabled_ = true;
@@ skipping 15 matching lines @@
   CheckTitleTest(L"content-sniffer-test1.html", L"");
 }
 
 TEST_F(ResourceDispatcherTest, DoNotSniffHTMLFromImageGIF) {
   CheckTitleTest(L"content-sniffer-test2.html", L"");
 }
 
 TEST_F(ResourceDispatcherTest, SniffNoContentTypeNoData) {
   CheckTitleTest(L"content-sniffer-test3.html",
                  L"Content Sniffer Test 3");
-  Sleep(sleep_timeout_ms() * 2);
+  PlatformThread::Sleep(sleep_timeout_ms() * 2);
   EXPECT_EQ(1, GetTabCount());
 
   // Make sure the download shelf is not showing.
   scoped_ptr<BrowserProxy> browser(automation()->GetBrowserWindow(0));
   scoped_ptr<TabProxy> dl_tab(browser->GetTab(0));
   ASSERT_TRUE(dl_tab.get());
 
   bool visible = false;
   ASSERT_TRUE(dl_tab->IsShelfVisible(&visible));
   EXPECT_FALSE(visible);
@@ skipping 124 matching lines @@
   if (in_process_renderer())
     return;
 
   scoped_ptr<BrowserProxy> browser_proxy(automation()->GetBrowserWindow(0));
   EXPECT_TRUE(browser_proxy.get());
   scoped_ptr<TabProxy> tab(browser_proxy->GetActiveTab());
 
   // Cause the renderer to crash.
   expected_crashes_ = 1;
   tab->NavigateToURLAsync(GURL("about:crash"));
-  Sleep(sleep_timeout_ms());  // Wait for browser to notice the renderer crash.
+
+  // Wait for browser to notice the renderer crash.
+  PlatformThread::Sleep(sleep_timeout_ms());
 
   // Navigate to a new cross-site page.  The browser should not wait around for
   // the old renderer's on{before}unload handlers to run.
   CheckTitleTest(L"content-sniffer-test0.html",
                  L"Content Sniffer Test 0");
 }
 
 // Tests that cross-site navigations work when the new page does not go through
 // the BufferedEventHandler (e.g., non-http{s} URLs).  (Bug 1225872)
 TEST_F(ResourceDispatcherTest, CrossSiteNavigationNonBuffered) {
@@ skipping 37 matching lines @@
   std::wstring tab_title;
   EXPECT_TRUE(tab->GetTabTitle(&tab_title));
   EXPECT_EQ(L"set cookie on unload", tab_title);
 
   // Navigate to a new cross-site URL that results in an error page.  We must
   // wait for the error page to update the title.
   // TODO(creis): If this causes crashes or hangs, it might be for the same
   // reason as ErrorPageTest::DNSError.  See bug 1199491.
   tab->NavigateToURL(GURL(URLRequestFailedDnsJob::kTestUrl));
   for (int i = 0; i < 10; ++i) {
-    Sleep(sleep_timeout_ms());
+    PlatformThread::Sleep(sleep_timeout_ms());
     if (GetActiveTabTitle() != L"set cookie on unload") {
       // Success, bail out.
       break;
     }
   }
   EXPECT_NE(L"set cookie on unload", GetActiveTabTitle());
 
   // Check that the cookie was set, meaning that the onunload handler ran.
   std::string value_result;
   EXPECT_TRUE(tab->GetCookieByName(url, "onunloadCookie", &value_result));
   EXPECT_FALSE(value_result.empty());
   EXPECT_STREQ("foo", value_result.c_str());
 
   // Check that renderer-initiated navigations still work.  In a previous bug,
   // the ResourceDispatcherHost would think that such navigations were
   // cross-site, because we didn't clean up from the previous request.  Since
   // WebContents was in the NORMAL state, it would ignore the attempt to run
   // the onunload handler, and the navigation would fail.
   // (Test by redirecting to javascript:window.location='someURL'.)
   GURL test_url(server->TestServerPageW(L"files/title2.html"));
   std::wstring redirect_url = L"javascript:window.location='" +
       ASCIIToWide(test_url.possibly_invalid_spec()) + L"'";
   tab->NavigateToURLAsync(GURL(redirect_url));
-  Sleep(sleep_timeout_ms());  // Wait for JavaScript redirect to happen.
+
+  // Wait for JavaScript redirect to happen.
+  PlatformThread::Sleep(sleep_timeout_ms());
   EXPECT_TRUE(tab->GetTabTitle(&tab_title));
   EXPECT_EQ(L"Title Of Awesomeness", tab_title);
 }