Attachment 268574 Details for Bug 347966

[patch] jprof support for tinderbox

jprof.diff-3 (text/plain), 3.89 KB, created by Robert Helmer [:rhelmer]

(hide)

Description:

Filename:

MIME Type:

Creator: Robert Helmer [:rhelmer]

Size: 3.89 KB

patch

obsolete

>Index: build-seamonkey-util.pl
>===================================================================
>RCS file: /cvsroot/mozilla/tools/tinderbox/build-seamonkey-util.pl,v
>retrieving revision 1.360
>diff -u -r1.360 build-seamonkey-util.pl
>--- build-seamonkey-util.pl	15 Jun 2007 10:58:40 -0000	1.360
>+++ build-seamonkey-util.pl	16 Jun 2007 01:42:20 -0000
>@@ -2441,7 +2441,7 @@
>         # Settle OS.
>         run_system_cmd("sync; sleep 5", 35);
> 
>-        my @urlargs = (-chrome,"file:$build_dir/mozilla/xpfe/test/winopen.xul");
>+        my @urlargs = (-chrome,"javascript:try{JProfStartProfiling();}catch(e){};window.location='file:$build_dir/mozilla/xpfe/test/winopen.xul'");
>         if($test_result eq 'success') {
>             if ($talkback_installed) {
>                 setTestnameForTalkbackReport($talkback_ini_path,$test_name);
>@@ -2634,6 +2634,11 @@
>         }
>     }
> 
>+    if ($Settings::TestWithJprof) {
>+        print_log "Running jprof for $test_name..\n";
>+        run_shell_command("cd $binary_dir && ./jprof firefox-bin jprof-log > ../$test_name-jprof.html");
>+    }
>+
>     return $rv;
> }
> 
>@@ -2707,7 +2712,7 @@
>     my $layout_time;
>     my $layout_time_details;
>     my $binary_log = "$build_dir/$test_name.log";
>-    my $url = "http://$Settings::pageload_server/page-loader/loader.pl?delay=1000&nocache=0&maxcyc=4&timeout=$Settings::LayoutPerformanceTestPageTimeout&auto=1";
>+    my $url = "javascript:try{JProfStartProfiling();}catch(e){};window.location='http://$Settings::pageload_server/page-loader/loader.pl?delay=1000&nocache=0&maxcyc=4&timeout=$Settings::LayoutPerformanceTestPageTimeout&auto=1'";
>     
>     # Settle OS.
>     run_system_cmd("sync; sleep 5", 35);
>@@ -2783,7 +2788,7 @@
>     my $layout_time;
>     my $layout_time_details;
>     my $binary_log = "$build_dir/$test_name.log";
>-    my $url = "http://localhost/pageload/cycler.html";
>+    my $url = "javascript:try{JProfStartProfiling();}catch(e){};window.location='http://localhost/pageload/cycler.html'";
>     
>     # Settle OS.
>     run_system_cmd("sync; sleep 5", 35);
>@@ -2839,7 +2844,7 @@
>     my $dhtml_time;
>     my $dhtml_time_details;
>     my $binary_log = "$build_dir/$test_name.log";
>-    my $url = "http://www.mozilla.org/performance/test-cases/dhtml/runTests.html";
>+    my $url = "javascript:try{JProfStartProfiling();}catch(e){};window.location='http://www.mozilla.org/performance/test-cases/dhtml/runTests.html'";
>     
>     # Settle OS.
>     run_system_cmd("sync; sleep 5", 35);
>Index: post-mozilla-rel.pl
>===================================================================
>RCS file: /cvsroot/mozilla/tools/tinderbox/post-mozilla-rel.pl,v
>retrieving revision 1.120
>diff -u -r1.120 post-mozilla-rel.pl
>--- post-mozilla-rel.pl	15 Jun 2007 10:58:40 -0000	1.120
>+++ post-mozilla-rel.pl	16 Jun 2007 01:42:21 -0000
>@@ -283,6 +283,10 @@
>       $push_raw_xpis = $Settings::push_raw_xpis;
>     }
> 
>+    if ($Settings::TestWithJprof) {
>+        TinderUtils::run_shell_command("cp $package_location/../*-jprof.html $stagedir/");
>+    }
>+
>     if (TinderUtils::is_windows() || TinderUtils::is_os2()) {
>       if ($Settings::stub_installer) {
>         TinderUtils::run_shell_command("cp $package_location/stub/*.exe $stagedir/");
>Index: tinder-defaults.pl
>===================================================================
>RCS file: /cvsroot/mozilla/tools/tinderbox/tinder-defaults.pl,v
>retrieving revision 1.111
>diff -u -r1.111 tinder-defaults.pl
>--- tinder-defaults.pl	12 Jun 2007 00:32:18 -0000	1.111
>+++ tinder-defaults.pl	16 Jun 2007 01:42:21 -0000
>@@ -82,6 +82,7 @@
> # ("network","dom","toolkit","security/manager");
> $CompareLocalesAviary     = 0;  # Should the compare-locales commands use the
>                                 # aviary directory structure?
>+$TestWithJprof            = 0;  # Use jprof profiler during tests
> 
> $TestsPhoneHome           = 0;  # Should test report back to server?
> $GraphNameOverride        = ''; # Override name built from ::hostname() and $BuildTag

Flags:

coop: review+

Actions: View | Diff | Review

Attachments on bug 347966: 268323 | 268534 | 268574 | 279674 | 282846 | 283032