<div dir="ltr">I think WebKitten should be able to choose between 1 and 2 on a per patch basis.<div><div><br></div><div>In both cases, Tools/Script/export-w3c-test-changes can be used to help upstreaming to WPT.<br><div>The WPT exporter can push the changes to a branch in a GitHub WPT clone.</div><div>It can also create the WPT PR on behalf of the WebKitten.</div><div><br></div><div>As requested by some folks, integration of WPT exporter with webkit-patch is on its way (<a href="https://bugs.webkit.org/show_bug.cgi?id=184914">https://bugs.webkit.org/show_bug.cgi?id=184914</a>)<div><br></div><div>Any WPT PR that is reviewed downstream in WebKit can be merged without further WPT review.</div><div>At the moment, given the WPT infra and the way we currently create WebKit PR, not every WebKitten is allowed to merge such PRs directly.</div><div>Please cc/ping me in such a case.</div><div>    Y</div><div><br></div><div><div class="gmail_quote"><div dir="ltr">Le mer. 23 mai 2018 à 02:41, Frédéric Wang <<a href="mailto:fwang@igalia.com">fwang@igalia.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<br>
When implementing new platform features in WebKit, some of us write new<br>
WPT tests in order to verify spec conformance. One of the reason to use<br>
this format is that the tests can then be easily shared with web engine<br>
developers and people involved in standardization, an important feature<br>
for web platform developers.<br>
<br>
Youenn has shared various ideas to export such WPT tests to the W3C<br>
GitHub repository in the past [1] [2] [3] but I'm not sure the WebKit<br>
community has reached an agreement.<br>
<br>
There are basically two ways to proceed:<br>
<br>
1) Either submit WPT tests to GitHub (or Mozilla / Chromium) repository,<br>
get them reviewed and do the import within the patch that implements the<br>
feature in WebKit.<br>
<br>
2) Or submit the patch with new WPT tests to WebKit, get it reviewed and<br>
export it to GitHub (later sync with Mozilla / Chromium).<br>
<br>
The first option has the advantage to get more people involved in test<br>
review, which sometimes gives faster replies or useful advice from<br>
people who more familiar with the spec. However, the second option has<br>
the advantage of not splitting the coding & testing parts and is better<br>
for the WebKit implementers and reviewers. That latter option also<br>
aligns with what Mozilla and Chromium developers do.<br>
<br>
Can we please agree on a way to proceed? And add documentation somewhere<br>
for developers/reviewers?<br>
<br>
Thanks!<br>
<br>
<br>
[1] <a href="https://lists.webkit.org/pipermail/webkit-dev/2017-March/028884.html" rel="noreferrer" target="_blank">https://lists.webkit.org/pipermail/webkit-dev/2017-March/028884.html</a><br>
[2] <a href="https://lists.webkit.org/pipermail/webkit-dev/2017-April/028932.html" rel="noreferrer" target="_blank">https://lists.webkit.org/pipermail/webkit-dev/2017-April/028932.html</a><br>
[3] <a href="https://lists.webkit.org/pipermail/webkit-dev/2017-December/029837.html" rel="noreferrer" target="_blank">https://lists.webkit.org/pipermail/webkit-dev/2017-December/029837.html</a><br>
<br>
-- <br>
Frédéric Wang - <a href="http://frederic-wang.fr" rel="noreferrer" target="_blank">frederic-wang.fr</a><br>
<br>
<br>
_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
<a href="https://lists.webkit.org/mailman/listinfo/webkit-dev" rel="noreferrer" target="_blank">https://lists.webkit.org/mailman/listinfo/webkit-dev</a><img alt="" src="https://ssl.gstatic.com/bt/C3341AA7A1A076756462EE2E5CD71C11/1x/btw_ic_close_white_12dp.png" style="font-family: -webkit-standard;"><br>
</blockquote></div></div></div></div></div></div>