Bug 187322 - [WPE][GTK]: Call gtk platform 'gtk' by default as this is what PortFactory expects
Summary: [WPE][GTK]: Call gtk platform 'gtk' by default as this is what PortFactory ex...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Thibault Saunier
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-07-04 08:47 PDT by Thibault Saunier
Modified: 2018-07-05 01:08 PDT (History)
5 users (show)

See Also:


Attachments
Patch (1.14 KB, patch)
2018-07-04 08:50 PDT, Thibault Saunier
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews124 for ios-simulator-wk2 (2.24 MB, application/zip)
2018-07-04 10:44 PDT, EWS Watchlist
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Thibault Saunier 2018-07-04 08:47:32 PDT
[WPE][GTK]: Call gtk platform 'gtk' by default as this is what PortFactory expects
Comment 1 Thibault Saunier 2018-07-04 08:50:08 PDT
Created attachment 344288 [details]
Patch
Comment 2 EWS Watchlist 2018-07-04 10:44:10 PDT
Comment on attachment 344288 [details]
Patch

Attachment 344288 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: https://webkit-queues.webkit.org/results/8437569

New failing tests:
animations/needs-layout.html
Comment 3 EWS Watchlist 2018-07-04 10:44:11 PDT
Created attachment 344293 [details]
Archive of layout-test-results from ews124 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews124  Port: ios-simulator-wk2  Platform: Mac OS X 10.13.4
Comment 4 Fujii Hironori 2018-07-04 18:48:52 PDT
Some scripts fails if flatpak is not used.

> $ ./Tools/Scripts/run-webkit-tests --gtk --release --no-new-test-results fast
> Traceback (most recent call last):
>   File "./Tools/Scripts/run-webkit-tests", line 39, in <module>
>     flatpakutils.run_in_sandbox_if_available(sys.argv)
>   File "./Tools/flatpak/flatpakutils.py", line 829, in run_in_sandbox_if_available
>     if not flatpak_runner.clean_args():
>   File "./Tools/flatpak/flatpakutils.py", line 561, in clean_args
>     port = factory.get(self.platform)
>   File "/home/fujii/work/webkit/ga/Tools/Scripts/webkitpy/port/factory.py", line 135, in get
>     raise NotImplementedError('unsupported platform: "%s"' % port_name)
> NotImplementedError: unsupported platform: "GTK"
Comment 5 Fujii Hironori 2018-07-04 18:50:28 PDT
Comment on attachment 344288 [details]
Patch

I also tested and confirmed it works. I'm not a reviewer. Informal LGTM.
Comment 6 Fujii Hironori 2018-07-04 23:14:49 PDT
Zan fixed.
https://trac.webkit.org/changeset/233511/webkit
Comment 7 Zan Dobersek 2018-07-05 00:41:08 PDT
Missed this bug. Rolled out my change in r233513 since this addresses it properly.
https://trac.webkit.org/changeset/233513/webkit
Comment 8 WebKit Commit Bot 2018-07-05 01:07:48 PDT
Comment on attachment 344288 [details]
Patch

Clearing flags on attachment: 344288

Committed r233514: <https://trac.webkit.org/changeset/233514>
Comment 9 WebKit Commit Bot 2018-07-05 01:07:49 PDT
All reviewed patches have been landed.  Closing bug.
Comment 10 Radar WebKit Bug Importer 2018-07-05 01:08:48 PDT
<rdar://problem/41839155>