[webkit-reviews] review granted: [Bug 229620] [GTK] Simplify run-gtk-tests handling of the a11y service : [Attachment 436653] Patch

[bugzilla-daemon at webkit.org]

Michael Catanzaro <mcatanzaro at gnome.org> has granted Patrick Griffis
<pgriffis at igalia.com>'s request for review:
Bug 229620: [GTK] Simplify run-gtk-tests handling of the a11y service
https://bugs.webkit.org/show_bug.cgi?id=229620

Attachment 436653: Patch

https://bugs.webkit.org/attachment.cgi?id=436653&action=review




--- Comment #4 from Michael Catanzaro <mcatanzaro at gnome.org> ---
Comment on attachment 436653
  --> https://bugs.webkit.org/attachment.cgi?id=436653
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=436653&action=review

> Tools/Scripts/run-gtk-tests:56
> +	       sys.stderr.write("Failed to talk to the accessibility service
{}\n".format(e))

Let's add a colon: "Failed to talk to the accessibility service: {}\n"

Also, can you check to ensure passing the error directly works? It will really
print a nice error string and not some object ID? There is no e.message
property that needs to be used instead?