[EFL][webkitpy] Do not redefine check_build() in EflPort.
authorrakuco@webkit.org <rakuco@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 10 May 2012 19:28:18 +0000 (19:28 +0000)
committerrakuco@webkit.org <rakuco@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 10 May 2012 19:28:18 +0000 (19:28 +0000)
https://bugs.webkit.org/show_bug.cgi?id=86124

Reviewed by Dirk Pranke.

Remove an old FIXME and do not redefine check_build in EflPort;
once build-dumprendertree recognizes --efl, we can use the
check_build implementation in WebKitPort.

* Scripts/build-dumprendertree:
* Scripts/webkitpy/layout_tests/port/efl.py:
(EflPort.check_build):

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

Tools/ChangeLog
Tools/Scripts/build-dumprendertree
Tools/Scripts/webkitpy/layout_tests/port/efl.py

index 1d02e79..3c43029 100644 (file)
@@ -1,3 +1,18 @@
+2012-05-10  Raphael Kubo da Costa  <rakuco@webkit.org>
+
+        [EFL][webkitpy] Do not redefine check_build() in EflPort.
+        https://bugs.webkit.org/show_bug.cgi?id=86124
+
+        Reviewed by Dirk Pranke.
+
+        Remove an old FIXME and do not redefine check_build in EflPort;
+        once build-dumprendertree recognizes --efl, we can use the
+        check_build implementation in WebKitPort.
+
+        * Scripts/build-dumprendertree:
+        * Scripts/webkitpy/layout_tests/port/efl.py:
+        (EflPort.check_build):
+
 2012-05-10  Tommy Widenflycht  <tommyw@google.com>
 
         [chromium] MediaStream API: Fix the ExtraData functionality in WebMediaStreamDescriptor
 2012-05-10  Tommy Widenflycht  <tommyw@google.com>
 
         [chromium] MediaStream API: Fix the ExtraData functionality in WebMediaStreamDescriptor
index 717f934..4e206f4 100755 (executable)
@@ -46,6 +46,7 @@ Usage: $programName [options] [options to pass to build system]
   --qt          Build the Qt port
   --wx          Build the wxWindows port
   --chromium    Build the Chromium port
   --qt          Build the Qt port
   --wx          Build the wxWindows port
   --chromium    Build the Chromium port
+  --efl         Build the EFL port
 EOF
 
 GetOptions(
 EOF
 
 GetOptions(
@@ -70,8 +71,8 @@ if (isAppleMacWebKit()) {
     $result = buildXCodeProject("DumpRenderTree", $clean, XcodeOptions(), @ARGV);
 } elsif (isAppleWinWebKit()) {
     $result = buildVisualStudioProject("DumpRenderTree.sln", $clean);
     $result = buildXCodeProject("DumpRenderTree", $clean, XcodeOptions(), @ARGV);
 } elsif (isAppleWinWebKit()) {
     $result = buildVisualStudioProject("DumpRenderTree.sln", $clean);
-} elsif (isQt() || isGtk() || isWx() || isChromium()) {
-    # Qt, Gtk wxWindows, and Chromium build everything in one shot. No need to build anything here.
+} elsif (isQt() || isGtk() || isWx() || isChromium() || isEfl()) {
+    # Qt, Gtk wxWindows, Chromium and EFL build everything in one shot. No need to build anything here.
     $result = 0;
 } else {
     die "Building not defined for this platform!\n";
     $result = 0;
 } else {
     die "Building not defined for this platform!\n";
index 6545cfa..f2121bd 100644 (file)
@@ -62,10 +62,6 @@ class EflPort(WebKitPort, PulseAudioSanitizer):
     def _path_to_image_diff(self):
         return self._build_path('bin', 'ImageDiff')
 
     def _path_to_image_diff(self):
         return self._build_path('bin', 'ImageDiff')
 
-    # FIXME: I doubt EFL wants to override this method.
-    def check_build(self, needs_http):
-        return self._check_driver()
-
     def _path_to_webcore_library(self):
         static_path = self._build_path('WebCore', 'libwebcore_efl.a')
         dyn_path = self._build_path('WebCore', 'libwebcore_efl.so')
     def _path_to_webcore_library(self):
         static_path = self._build_path('WebCore', 'libwebcore_efl.a')
         dyn_path = self._build_path('WebCore', 'libwebcore_efl.so')