Bug 76817 – [GTK] run-gtk-tests randomly fails while running the xprop comand

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Mario Sanchez Prada]

Since the patch for bug 72589 got in, the GTK bots have been randomly failing sometimes because of the run-gtk-scripts sometimes (I still don't understand the reason behind it) fails to execute the xprop command, which it needs to run to make sure that there is no ATSPI accessibility bus already launched.

As the problem is not related to the WK2GTK a11y code itself, but to the run-gtk-tests script (when it runs properly, the WK2 a11y test passes), my proposal is to create a patch now for commenting the offending lines in run-gtk-tests and keep this bug open once it gets in, so we can work on getting a not randomly failing version of that script without having the bots red in the meanwhile.

[Mario Sanchez Prada]

For reference, this is the error that is happening in the bots when this command fails:

  Skipping test unittests/testdownload
  Skipping test unittests/testwebview
  Skipping test unittests/testwebresource
  Traceback (most recent call last):
    File "./Tools/Scripts/run-gtk-tests", line 147, in <module>
      sys.exit(TestRunner().run())
    File "./Tools/Scripts/run-gtk-tests", line 97, in run
      self._executive.run_command(['xprop', '-root', '-remove', 'AT_SPI_BUS'], env=test_env)
    File "/var/lib/buildbot/build/gtk-linux-32-release/build/Tools/Scripts/webkitpy/common/system/executive.py", line 440, in run_command
      (error_handler or self.default_error_handler)(script_error)
    File "/var/lib/buildbot/build/gtk-linux-32-release/build/Tools/Scripts/webkitpy/common/system/executive.py", line 357, in default_error_handler
      raise error
  webkitpy.common.system.executive.ScriptError: Failed to run "[u'xprop', u'-root', u'-remove', u'AT_SPI_BUS']"  exit_code: 1

Surprisingly, if I log in the bot and manually run Xvfb once, the error will go away... perhaps a race condition between running Xvfb and running the xprop command?

[Mario Sanchez Prada]

Created attachment 123531 [details]
Comment ATSPI related lines in run-gtk-tests

[Mario Sanchez Prada]

Committed r105601: <http://trac.webkit.org/changeset/105601>

[Mario Sanchez Prada]

Created attachment 123546 [details]
Don't use xprop in run-gtk-test (not needed)

[Gustavo Noronha (kov)]

Comment on attachment 123546 [details]
Don't use xprop in run-gtk-test (not needed)

OK

[Mario Sanchez Prada]

Committed r105617: <http://trac.webkit.org/changeset/105617>