Title: [105673] trunk/Tools
Revision
105673
Author
[email protected]
Date
2012-01-23 18:05:17 -0800 (Mon, 23 Jan 2012)

Log Message

run-webkit-tests needs to propagate --chromium
https://bugs.webkit.org/show_bug.cgi?id=76870

Reviewed by Eric Seidel.

run-webkit-tests removes '--chromium' argument from @ARGV when
determining which port to run, which means that that doesn't
propagate to new-run-webkit-tests. That's bad (and is handled
for the other ports by re-adding the flag, but apparently we're
just now noticing for Chromium).

* Scripts/run-webkit-tests:

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (105672 => 105673)


--- trunk/Tools/ChangeLog	2012-01-24 02:00:48 UTC (rev 105672)
+++ trunk/Tools/ChangeLog	2012-01-24 02:05:17 UTC (rev 105673)
@@ -1,3 +1,18 @@
+2012-01-23  Dirk Pranke  <[email protected]>
+
+        run-webkit-tests needs to propagate --chromium
+        https://bugs.webkit.org/show_bug.cgi?id=76870
+
+        Reviewed by Eric Seidel.
+
+        run-webkit-tests removes '--chromium' argument from @ARGV when
+        determining which port to run, which means that that doesn't
+        propagate to new-run-webkit-tests. That's bad (and is handled
+        for the other ports by re-adding the flag, but apparently we're
+        just now noticing for Chromium).
+
+        * Scripts/run-webkit-tests:
+
 2012-01-23  Brian Weinstein  <[email protected]>
 
         More build fixing after r105646.

Modified: trunk/Tools/Scripts/run-webkit-tests (105672 => 105673)


--- trunk/Tools/Scripts/run-webkit-tests	2012-01-24 02:00:48 UTC (rev 105672)
+++ trunk/Tools/Scripts/run-webkit-tests	2012-01-24 02:05:17 UTC (rev 105673)
@@ -110,6 +110,8 @@
     push(@ARGV, "--gtk");
 } elsif (isEfl()) {
     push(@ARGV, "--efl");
+} elsif (isChromium()) {
+    push(@ARGV, "--chromium")
 }
 
 my $harnessPath = File::Spec->catfile(relativeScriptsDir(), $harnessName);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to