[EFL][webkitpy] Define the 'wrapper' option in EflPort instead of creating a EflDriver.
authorrakuco@webkit.org <rakuco@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 10 May 2012 18:15:04 +0000 (18:15 +0000)
committerrakuco@webkit.org <rakuco@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 10 May 2012 18:15:04 +0000 (18:15 +0000)
https://bugs.webkit.org/show_bug.cgi?id=86117

Reviewed by Adam Barth.

Simplify the code used to wrap driver calls with jhbuild: we can
just define the 'wrapper' option instead of having to create a
Driver class just for that.

* Scripts/webkitpy/layout_tests/port/efl.py:
(EflPort.__init__):
(EflPort._port_flag_for_scripts):

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

Tools/ChangeLog
Tools/Scripts/webkitpy/layout_tests/port/efl.py

index 11325bb..7e0e431 100644 (file)
@@ -1,3 +1,18 @@
+2012-05-10  Raphael Kubo da Costa  <rakuco@webkit.org>
+
+        [EFL][webkitpy] Define the 'wrapper' option in EflPort instead of creating a EflDriver.
+        https://bugs.webkit.org/show_bug.cgi?id=86117
+
+        Reviewed by Adam Barth.
+
+        Simplify the code used to wrap driver calls with jhbuild: we can
+        just define the 'wrapper' option instead of having to create a
+        Driver class just for that.
+
+        * Scripts/webkitpy/layout_tests/port/efl.py:
+        (EflPort.__init__):
+        (EflPort._port_flag_for_scripts):
+
 2012-05-10  Sheriff Bot  <webkit.review.bot@gmail.com>
 
         Unreviewed, rolling out r116633.
index db7be80..6545cfa 100644 (file)
@@ -32,25 +32,21 @@ import signal
 import subprocess
 
 from webkitpy.layout_tests.models.test_configuration import TestConfiguration
-from webkitpy.layout_tests.port.webkit import WebKitDriver, WebKitPort
+from webkitpy.layout_tests.port.webkit import WebKitPort
 from webkitpy.layout_tests.port.pulseaudio_sanitizer import PulseAudioSanitizer
 
 
-class EflDriver(WebKitDriver):
-    def cmd_line(self, pixel_tests, per_test_args):
-        wrapper_path = self._port.path_from_webkit_base("Tools", "efl", "run-with-jhbuild")
-        return [wrapper_path] + WebKitDriver.cmd_line(self, pixel_tests, per_test_args)
-
-
 class EflPort(WebKitPort, PulseAudioSanitizer):
     port_name = 'efl'
 
+    def __init__(self, *args, **kwargs):
+        WebKitPort.__init__(self, *args, **kwargs)
+
+        self.set_option_default('wrapper', self.path_from_webkit_base('Tools', 'efl', 'run-with-jhbuild'))
+
     def _port_flag_for_scripts(self):
         return "--efl"
 
-    def _driver_class(self):
-        return EflDriver
-
     def setup_test_run(self):
         self._unload_pulseaudio_module()