Attachment 339560 Details for Bug 185304 – Patch

[patch] Patch

bug-185304-20180504141813.patch (text/plain), 1.75 KB, created by Leo Balter on 2018-05-04 10:18:14 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Leo Balter

Created: 2018-05-04 10:18:14 PDT

Size: 1.75 KB

patch

obsolete

>Subversion Revision: 231356
>diff --git a/Tools/ChangeLog b/Tools/ChangeLog
>index 717973039824042379641466e7e4990a1c720936..99ddafe306310995fd241ad40206537d5039ef49 100644
>--- a/Tools/ChangeLog
>+++ b/Tools/ChangeLog
>@@ -1,3 +1,13 @@
>+2018-05-04  Leo Balter  <leonardo.balter@gmail.com>
>+
>+        Test262-Runner: place the processCLI inside the main call
>+        https://bugs.webkit.org/show_bug.cgi?id=185304
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * Scripts/test262/Runner.pm:
>+        (main):
>+
> 2018-05-04  Carlos Garcia Campos  <cgarcia@igalia.com>
> 
>         [GTK] Epiphany (GNOME Web) says "Error downloading: Service Unavailable." when trying to download an image from discogs.com
>diff --git a/Tools/Scripts/test262/Runner.pm b/Tools/Scripts/test262/Runner.pm
>index 4237601987d97a53b40ddd8112f8a917c9c5a7c7..2d0a3275033f3ad82148136636e6763941203a7c 100755
>--- a/Tools/Scripts/test262/Runner.pm
>+++ b/Tools/Scripts/test262/Runner.pm
>@@ -92,20 +92,12 @@ my @results;
> my @files;
> 
> my $tempdir = tempdir();
>-
>-my @default_harnesses = (
>-    "$harnessDir/sta.js",
>-    "$harnessDir/assert.js",
>-    "$harnessDir/doneprintHandle.js",
>-    "$Bin/agent.js"
>-);
>-
> my ($deffh, $deffile) = getTempFile();
>-print $deffh getHarness(<@default_harnesses>);
>+
>+my @default_harnesses;
> 
> my $startTime = time();
> 
>-processCLI();
> main();
> 
> sub processCLI {
>@@ -223,6 +215,15 @@ sub processCLI {
> }
> 
> sub main {
>+    processCLI();
>+
>+    @default_harnesses = (
>+        "$harnessDir/sta.js",
>+        "$harnessDir/assert.js",
>+        "$harnessDir/doneprintHandle.js",
>+        "$Bin/agent.js"
>+    );
>+    print $deffh getHarness(<@default_harnesses>);
> 
>     # If not commandline test path supplied, use the root directory of all tests.
>     push(@cliTestDirs, 'test') if not @cliTestDirs;

Actions: View | Formatted Diff | Diff

Attachments on bug 185304: 339560