Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7191)

Unified Diff: chrome_frame/test/chrome_frame_test_utils.cc

Issue 9225051: Enlarge the CF integration test timeout values, more so for Debug builds. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Correct inverted Release/Debug values. Created 8 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome_frame/test/chrome_frame_test_utils.cc
diff --git a/chrome_frame/test/chrome_frame_test_utils.cc b/chrome_frame/test/chrome_frame_test_utils.cc
index 3324a0d64747cf17c487e5f679d2edc7564b2061..31d3e1985659c17bb8bb475ced07111c32a4a55e 100644
--- a/chrome_frame/test/chrome_frame_test_utils.cc
+++ b/chrome_frame/test/chrome_frame_test_utils.cc
@@ -49,8 +49,14 @@ const wchar_t kIEBrokerImageName[] = L"ieuser.exe";
const char kChromeImageName[] = "chrome.exe";
const wchar_t kIEProfileName[] = L"iexplore";
const wchar_t kChromeLauncher[] = L"chrome_launcher.exe";
-const int kChromeFrameLongNavigationTimeoutInSeconds = 10;
-const int kChromeFrameVeryLongNavigationTimeoutInSeconds = 30;
+
+#ifndef NDEBUG
+const int kChromeFrameLongNavigationTimeoutInSeconds = 30;
+const int kChromeFrameVeryLongNavigationTimeoutInSeconds = 90;
+#else
+const int kChromeFrameLongNavigationTimeoutInSeconds = 15;
+const int kChromeFrameVeryLongNavigationTimeoutInSeconds = 45;
+#endif
// Callback function for EnumThreadWindows.
BOOL CALLBACK CloseWindowsThreadCallback(HWND hwnd, LPARAM param) {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698