Bug 141928 – Merge run-launcher into run-minibrowser

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Carlos Alberto Lopez Perez]

The script Tools/Scripts/run-minibrowser don't works for the EFL or GTK ports.

I think this script should gain both a --gtk and --efl switches to execute the minibrowser from this two ports.

[Csaba Osztrogonác]

Yes, it seems it is Mac only now. But run-launcher 
already works, you can copy/paste the logic from it.

[Carlos Garcia Campos]

What is it used for? will it replace run-launcher eventually?

[Carlos Alberto Lopez Perez]

I didn't knew about run-launcher. Until now I'm executing the minibrowser manually like:

Tools/jhbuild/jhbuild-wrapper --gtk run WebKitBuild/Release/bin/MiniBrowser


What is the difference between run-launcher and run-minibrowser ? Should we merge both ?

[Csaba Osztrogonác]

It seems run-launcher is used only EFL and GTK nowadays,
run-webkit-tests calls it after tests (efl.py, gtk.py)
And run-minibrowser seems to be Mac only script now.

I'd prefer merging run-launcher to run-minibrowser.

[Michael Catanzaro]

Be sure to update the build-webkit script as well; it instructs users to use run-launcher without any arguments when the build is complete, but that just prints a message saying your OS is unsupported, so this would be a good time to fix that.

[Csaba Osztrogonác]

Created attachment 251741 [details]
Patch

additionally removing obsolete FIXMEs

[Csaba Osztrogonác]

Comment on attachment 251741 [details]
Patch

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

> Tools/Scripts/webkitpy/port/efl.py:-132
> -        # FIXME: We should find a way to share this implmentation with Gtk,
> -        # or teach run-launcher how to call run-safari and move this down to WebKitPort.

It is only a oneliner, there is no need to try to share it with GTK.

> Tools/Scripts/webkitpy/port/efl.py:-135
> -        # FIXME: old-run-webkit-tests also added ["-graphicssystem", "raster", "-style", "windows"]
> -        # FIXME: old-run-webkit-tests converted results_filename path for cygwin.

Just some comment, why can we remove them:
- -graphicssystem raster -style windows are Qt crufts
- cygwin: I can't remember if we ever supported Windows.

[Darin Adler]

Comment on attachment 251741 [details]
Patch

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

> Tools/Scripts/webkitpy/port/efl.py:131
> +        self._run_script("run-minibrowser", ["file://%s" % results_filename])

This is not the right way to construct a URL from a local file name. Isn’t there some python library function that can turn a filename into a URL that can handle things like "%" characters?

> Tools/Scripts/webkitpy/port/gtk.py:202
> +        self._run_script("run-minibrowser", ["file://%s" % results_filename])

Ditto.

[Csaba Osztrogonác]

webkitpy.common.system.path.abspath_to_uri does it,
win.py already use it in its show_results_html_file.

The results is same, but I'll change it before landing.

[Csaba Osztrogonác]

Committed r183400: <http://trac.webkit.org/changeset/183400>