Bug 100102

Summary: [WK2] [GTK] TestWebKitAccessibility is not being run
Product: WebKit Reporter: Mario Sanchez Prada <mario>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: apinheiro, cgarcia, jdiggs
Priority: P2 Keywords: Gtk
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch proposal cgarcia: review+

Description Mario Sanchez Prada 2012-10-23 03:43:37 PDT
It seems a line got in run-gtk-tests by mistake along with SVN r120944, causing this test not to be run in WK2 bots:

     def _start_accessibility_daemons(self):
         return False
         spi_bus_launcher_path = self._lookup_atspi2_binary('at-spi-bus-launcher')
         spi_registryd_path = self._lookup_atspi2_binary('at-spi2-registryd')
         [...]

Additionally, there seems to be a mistake in configure.ac as well, since the following block is now _after_ the definition of enable_webkit2, meaning that the PKG_CHECK_MODULES is never executed:

  # check if atspi2 is available (only needed for Webkit2 unit tests)
  if test "$enable_webkit2" = "yes"; then
     PKG_CHECK_MODULES([ATSPI2],
                       [atspi-2 >= $ATSPI2_REQUIRED_VERSION],
                       [have_atspi2=yes], [have_atspi2=no])

     AC_SUBST([ATSPI2_CFLAGS])
     AC_SUBST([ATSPI2_LIBS])
  fi

This needs to be fixed
Comment 1 Mario Sanchez Prada 2012-10-23 03:53:02 PDT
Created attachment 170110 [details]
Patch proposal
Comment 2 Carlos Garcia Campos 2012-10-24 00:00:19 PDT
Comment on attachment 170110 [details]
Patch proposal

Thanks!
Comment 3 Mario Sanchez Prada 2012-10-24 01:13:06 PDT
Committed r132328: <http://trac.webkit.org/changeset/132328>