After r210523: <http://trac.webkit.org/changeset/210523> we are ready to add the new release test bot for Wayland (WebKitGTK+). This bot will run the tests under a real wayland session (using the machine native GPU -- an Intel i7-3770 GPU with Mesa and a native Weston session). Since we would have to restart the master to make the changes effective and to avoid more restarts than needed, we seize this bug also to start passing --display-server=xorg to the GTK perf bot so we can stop using the environment variable USE_NATIVE_XDISPLAY=1 to tell the perf bot to use the driver for native X11 on the perf-test step.
Created attachment 298421 [details] Patch
Comment on attachment 298421 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=298421&action=review > Tools/BuildSlaveSupport/build.webkit.org-config/config.json:297 > + "additionalArguments": ["--display-server=xorg"], I would go ahead and remove the FIXME you left in the previous commit now, since you're squeezing this in here anyway.
(In reply to comment #2) > Comment on attachment 298421 [details] > Patch > Thanks for the review > View in context: > https://bugs.webkit.org/attachment.cgi?id=298421&action=review > > > Tools/BuildSlaveSupport/build.webkit.org-config/config.json:297 > > + "additionalArguments": ["--display-server=xorg"], > > I would go ahead and remove the FIXME you left in the previous commit now, > since you're squeezing this in here anyway. I intend to do that in an unreviewed follow-up patch after this patch lands and the master is rebooted. Otherwise I will be risking having the bot running the perf tests with xvfb in the window between [this_path_lands, master_is_rebooted]
Comment on attachment 298421 [details] Patch Clearing flags on attachment: 298421 Committed r210539: <http://trac.webkit.org/changeset/210539>
All reviewed patches have been landed. Closing bug.
(In reply to comment #3) > > I would go ahead and remove the FIXME you left in the previous commit now, > > since you're squeezing this in here anyway. > > I intend to do that in an unreviewed follow-up patch after this patch lands > and the master is rebooted. Otherwise I will be risking having the bot > running the perf tests with xvfb in the window between [this_path_lands, > master_is_rebooted] Committed r210547: <http://trac.webkit.org/changeset/210547>