<div dir="auto"><div class="gmail_quote" dir="auto"><div dir="ltr">On Fri, May 25, 2018, 00:52 Ryosuke Niwa <<a href="mailto:rniwa@webkit.org">rniwa@webkit.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Thu, May 24, 2018 at 3:22 AM, Philip Jägenstedt <span dir="ltr"><<a href="mailto:foolip@chromium.org" target="_blank" rel="noreferrer">foolip@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div><div class="gmail_quote"><div dir="ltr">On Wed, May 23, 2018, 23:43 youenn fablet <<a href="mailto:youennf@gmail.com" target="_blank" rel="noreferrer">youennf@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span>Le mer. 23 mai 2018 à 14:11, Frédéric Wang <<a href="mailto:fwang@igalia.com" rel="noreferrer noreferrer" target="_blank">fwang@igalia.com</a>> a écrit :<br></span><div class="gmail_quote"><span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 23/05/2018 22:50, Ryosuke Niwa wrote:<br>
> As we have preciously discussed, we should NEVER commit new tests into<br>
> LayoutTests/imported/w3c/web-platform-tests.<br></blockquote><div><br></div></span><div>Ryosuke, correct me if I am wrong, I think you are pointing out the following rule:</div><div>Changes to LayoutTests/imported/w3c/web-platform-tests tests should land first in WPT repository, then in WebKit repository.</div></div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Oh, that is surprising.</div><div dir="auto"><br></div><div dir="auto"><a href="https://github.com/w3c/web-platform-tests/pull/10964" target="_blank" rel="noreferrer">https://github.com/w3c/web-platform-tests/pull/10964</a> is a recent WebKit export, and <a href="https://trac.webkit.org/changeset/231788/webkit" target="_blank" rel="noreferrer">https://trac.webkit.org/changeset/231788/webkit</a> did modify the test in place. Do you mean that the WPT PR was merged first, or should be in general? Chromium and Gecko do it in the other order, and I'd be interested to understand the trade-offs of flipping the order.</div></div></blockquote><div><br></div><div>Yes, it's okay for a WebKit commit to merge the change which got merged into WPT but it's never okay to first commit the test into WebKit and then later upstream it to WPT...</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div dir="auto">Any process like this where changes end up in WebKit trunk via anything except a full WPT import will mean that divergence is possible.</div></div></blockquote><div><br></div><div>Precisely to avoid these problems.</div></div></div></div></blockquote></div><div dir="auto"><br></div><div dir="auto">I'm saying that there's temporary divergence with the WPT-then-WebKit order as well.</div><div dir="auto"><br></div><div dir="auto">Whichever side is merged first, the other side can fail to merge because any number of reasons, and something/someone then needs to deal with that. (See question in reply to Youenn.)</div><div dir="auto"><br></div><div dir="auto">However, one thing I really like about your order is that it makes it more straightforward to make WPT repo failures block browser repo merging. In the WPT Travis job, we already detect flakiness for Chrome and Firefox and I think we should also detect harness errors, and eventually also flag when a test goes from passing everywhere to failing everywhere, which is probably a mistake. We haven't really figured out how to make that block Chromium's CQ yet, and maybe flipping the order would do it.</div><div class="gmail_quote" dir="auto"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
</blockquote></div></div>