From e7b66aa43aa604a9cfae0b6a5fbfa781f6103791 Mon Sep 17 00:00:00 2001
From: Marshall Greenblatt <magreenblatt@gmail.com>
Date: Fri, 4 Oct 2019 15:36:06 +0300
Subject: [PATCH] Increase URLRequestTest CloseBrowser timeout to reduce flakes
 (fixes issue #2756)

Tests for incomplete request behavior use a timeout to trigger CloseBrowser
and terminate the test case. Recent architectural changes in Chromium have
likely increased the minimum delay required to spin up a working browser
instance and initiate the request. Consequently we need to wait longer before
closing the browser.

To test:
Run `ceftests --gtest_filter=URLRequestTest.*Incomplete* --gtest_repeat=5`.
All test runs should succeed.
---
 tests/ceftests/urlrequest_unittest.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/ceftests/urlrequest_unittest.cc b/tests/ceftests/urlrequest_unittest.cc
index bf3ca3a41..ec65f9990 100644
--- a/tests/ceftests/urlrequest_unittest.cc
+++ b/tests/ceftests/urlrequest_unittest.cc
@@ -2836,7 +2836,7 @@ class RequestTestHandler : public TestHandler {
     SetSignalCompletionWhenAllBrowsersClose(false);
     CefPostDelayedTask(
         TID_UI, base::Bind(&TestHandler::CloseBrowser, GetBrowser(), false),
-        100);
+        1000);
   }
 
   // Test run is complete. It ran in either the browser or render process.