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

Side by Side Diff: chrome/test/reliability/page_load_test.cc

Issue 5967003: Refactor UITestBase/ProxyLauncher. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Add POD struct to hold some launcher variables. Created 9 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 // 4 //
5 // This file provides reliablity test which runs under UI test framework. The 5 // This file provides reliablity test which runs under UI test framework. The
6 // test is intended to run within QEMU environment. 6 // test is intended to run within QEMU environment.
7 // 7 //
8 // Usage 1: reliability_test 8 // Usage 1: reliability_test
9 // Upon invocation, it visits a hard coded list of sample URLs. This is mainly 9 // Upon invocation, it visits a hard coded list of sample URLs. This is mainly
10 // used by buildbot, to verify reliability_test itself runs ok. 10 // used by buildbot, to verify reliability_test itself runs ok.
(...skipping 313 matching lines...) Expand 10 before | Expand all | Expand 10 after
324 g_server_url.c_str(); 324 g_server_url.c_str();
325 std::string test_page_url( 325 std::string test_page_url(
326 StringPrintf("%s/page?id=%d", server, i)); 326 StringPrintf("%s/page?id=%d", server, i));
327 NavigateToURLLogResult(test_page_url, log_file, NULL); 327 NavigateToURLLogResult(test_page_url, log_file, NULL);
328 } 328 }
329 } else { 329 } else {
330 // Don't run if single process mode. 330 // Don't run if single process mode.
331 // Also don't run if running as a standalone program which is for 331 // Also don't run if running as a standalone program which is for
332 // distributed testing, to avoid mistakenly hitting web sites with many 332 // distributed testing, to avoid mistakenly hitting web sites with many
333 // instances. 333 // instances.
334 if (in_process_renderer() || g_stand_alone) 334 if (ProxyLauncher::in_process_renderer() || g_stand_alone)
335 return; 335 return;
336 // For usage 1 336 // For usage 1
337 NavigationMetrics metrics; 337 NavigationMetrics metrics;
338 if (g_timeout_ms == -1) 338 if (g_timeout_ms == -1)
339 g_timeout_ms = 2000; 339 g_timeout_ms = 2000;
340 340
341 // Though it would be nice to test the page down code path in usage 1, 341 // Though it would be nice to test the page down code path in usage 1,
342 // enabling page down adds several seconds to the test and does not seem 342 // enabling page down adds several seconds to the test and does not seem
343 // worth the tradeoff. It is also potentially disruptive when running the 343 // worth the tradeoff. It is also potentially disruptive when running the
344 // test in the background as it will send the event to the window that 344 // test in the background as it will send the event to the window that
(...skipping 372 matching lines...) Expand 10 before | Expand all | Expand 10 after
717 parsed_command_line.GetSwitchValuePath(switches::kJavaScriptFlags)); 717 parsed_command_line.GetSwitchValuePath(switches::kJavaScriptFlags));
718 if (v8_command_line.HasSwitch(kV8LogFileSwitch)) { 718 if (v8_command_line.HasSwitch(kV8LogFileSwitch)) {
719 g_v8_log_path = v8_command_line.GetSwitchValuePath(kV8LogFileSwitch); 719 g_v8_log_path = v8_command_line.GetSwitchValuePath(kV8LogFileSwitch);
720 if (!file_util::AbsolutePath(&g_v8_log_path)) 720 if (!file_util::AbsolutePath(&g_v8_log_path))
721 g_v8_log_path = FilePath(); 721 g_v8_log_path = FilePath();
722 } 722 }
723 } 723 }
724 } 724 }
725 } 725 }
726 } 726 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698