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

Unified Diff: chrome/test/ui/ui_test.cc

Issue 209022: Enable page load test on linux (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: dont compile page load test on mac Created 11 years, 3 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 | « chrome/test/reliability/page_load_test.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/ui/ui_test.cc
===================================================================
--- chrome/test/ui/ui_test.cc (revision 26501)
+++ chrome/test/ui/ui_test.cc (working copy)
@@ -46,9 +46,16 @@
static const int kWaitForTerminateMsec = 30000;
const wchar_t UITest::kFailedNoCrashService[] =
+#if defined(OS_WIN)
L"NOTE: This test is expected to fail if crash_service.exe is not "
L"running. Start it manually before running this test (see the build "
L"output directory).";
+#elif defined(OS_LINUX)
+ L"NOTE: This test is expected to fail if breakpad is not built in "
+ L"or if chromium is not running headless (try CHROME_HEADLESS=1).";
+#else
+ L"NOTE: Crash service not ported to this platform!";
+#endif
bool UITest::in_process_renderer_ = false;
bool UITest::no_sandbox_ = false;
bool UITest::full_memory_dump_ = false;
@@ -63,7 +70,6 @@
std::wstring UITest::js_flags_ = L"";
std::wstring UITest::log_level_ = L"";
-
// Specify the time (in milliseconds) that the ui_tests should wait before
// timing out. This is used to specify longer timeouts when running under Purify
// which requires much more time.
« no previous file with comments | « chrome/test/reliability/page_load_test.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698