[NRWT] Fix --platform=qt-5.0 --new-baseline combo
authorossy@webkit.org <ossy@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 30 Nov 2011 07:53:47 +0000 (07:53 +0000)
committerossy@webkit.org <ossy@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 30 Nov 2011 07:53:47 +0000 (07:53 +0000)
https://bugs.webkit.org/show_bug.cgi?id=72489

Patch by Balazs Ankes <Ankes.Balazs@stud.u-szeged.hu> on 2011-11-29
Reviewed by Eric Seidel.

Before the this fix always added "qt" option for the option list
* Scripts/run-webkit-tests:
* Scripts/webkitpy/layout_tests/port/qt.py:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@101451 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/run-webkit-tests
Tools/Scripts/webkitpy/layout_tests/port/qt.py

index 28cbdf5..9aa722d 100644 (file)
@@ -1,3 +1,14 @@
+2011-11-29  Balazs Ankes  <Ankes.Balazs@stud.u-szeged.hu>
+
+        [NRWT] Fix --platform=qt-5.0 --new-baseline combo
+        https://bugs.webkit.org/show_bug.cgi?id=72489
+
+        Reviewed by Eric Seidel.
+
+        Before the this fix always added "qt" option for the option list
+        * Scripts/run-webkit-tests:
+        * Scripts/webkitpy/layout_tests/port/qt.py:
+
 2011-11-29  David Levin  <levin@chromium.org>
 
         Add a way to revert a variable to its previous value after leaving a scope.
index e4d9706..d8b19f4 100755 (executable)
@@ -99,7 +99,14 @@ if (useNewRunWebKitTests()) {
 # webkitdirs.pm strips --qt and --gtk from @ARGV when we call isQt/isGtk.
 # We have to add back any --PORT arguments which may have been removed by isPort() checks above.
 if (isQt()) {
-    push(@ARGV, "--qt");
+    my $isPlatformSet = 0;
+    for (@ARGV){
+        # Pass --qt if platform isn't passed explicitly (eg. qt-5.0, qt-wk2, ...)
+        if(/^--platform.*/){
+            $isPlatformSet = 1;
+        }
+    }
+    push(@ARGV, "--qt") if(!$isPlatformSet);
 } elsif (isGtk()) {
     push(@ARGV, "--gtk");
 }
index 9840953..a1dbcb2 100644 (file)
@@ -59,17 +59,22 @@ class QtPort(WebKitPort):
         return None
 
     # sys_platform exists only for unit testing.
-    def __init__(self, host, sys_platform=None, **kwargs):
-        WebKitPort.__init__(self, host, **kwargs)
+    def __init__(self, host, port_name=None, sys_platform=None, **kwargs):
+        port_name = port_name or self.port_name
+        WebKitPort.__init__(self, host, port_name=None, **kwargs)
         self._operating_system = self._operating_system_for_platform(sys_platform or sys.platform)
         self._version = self._operating_system
 
         # FIXME: This will allow WebKitPort.baseline_search_path and WebKitPort._skipped_file_search_paths
         # to do the right thing, but doesn't include support for qt-4.8 or qt-arm (seen in LayoutTests/platform) yet.
-        name_components = [self.port_name]
-        if self._operating_system:
-            name_components.append(self._operating_system)
-        self._name = "-".join(name_components)
+
+        if port_name != self.port_name:
+            self._name = port_name
+        else:
+            name_components = [self.port_name]
+            if self._operating_system:
+                name_components.append(self._operating_system)
+            self._name = "-".join(name_components)
 
     def _generate_all_test_configurations(self):
         configurations = []