Bug 256061 - [run-benchmark][WPE] Update cog and minibrowser-wpe browser drivers.
Summary: [run-benchmark][WPE] Update cog and minibrowser-wpe browser drivers.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Carlos Alberto Lopez Perez
URL:
Keywords: InRadar
Depends on:
Blocks: 249031
  Show dependency treegraph
 
Reported: 2023-04-27 16:07 PDT by Carlos Alberto Lopez Perez
Modified: 2023-04-28 03:42 PDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Alberto Lopez Perez 2023-04-27 16:07:04 PDT
At bug 249286 I had the idea of adding a cog-drm browser driver to run-benchmark script to be able to run perf tets on the RPi.

However I realized this doesn't scale well because we will end with several drivers for cog (one for drm, another for wayland, other for gtk4, etc..)

This also will presents problems later on the UI of https://browserperfdash.igalia.com to identify the browser (it makes little sense to have a 'cog-drm' and a 'cog-gtk4' in the browser list there like if those were different browsers.. there is already a GPU field in case of wanting to have the data split by graphics parameters)

So after landing 263463@main it is now possible to specify the platform plugin for Cog via an environment variable.

We can simplify the driver to have only one Cog browser and in the bots we can select which platform plugin to use via the env.
Comment 1 Carlos Alberto Lopez Perez 2023-04-27 17:00:36 PDT
Pull request: https://github.com/WebKit/WebKit/pull/13253
Comment 2 EWS 2023-04-28 03:41:27 PDT
Committed 263495@main (3bfd9bb7a457): <https://commits.webkit.org/263495@main>

Reviewed commits have been landed. Closing PR #13253 and removing active labels.
Comment 3 Radar WebKit Bug Importer 2023-04-28 03:42:20 PDT
<rdar://problem/108652798>