<div dir="auto"><div class="gmail_quote" dir="auto"><div dir="ltr">On Thu, May 24, 2018, 21:40 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"><br><div class="gmail_quote"><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"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div><br></div><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><div dir="auto"><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" rel="noreferrer noreferrer" target="_blank">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" rel="noreferrer noreferrer" target="_blank">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>Yep, WPT PR merged first, then WebKit patch landed.</div><div>It makes sure that whenever we are reimporting tests, we are not loosing any WebKit specific change.</div><div>Conflict resolution also happens at a time the patch is being committed in WebKit.</div><div>So far, I encountered very few conflicts.</div></div></div></blockquote></div><div dir="auto"><br></div><div dir="auto">Interesting. What happens happens if the WebKit patch then fails to land in WebKit, perhaps because some bot fails the test, a conflict, or anything else? Is resolving that a manual affair?</div><div dir="auto"><br></div><div dir="auto">When imports are automated and much more frequent, I take it the upside is that you never need to reapply still-being-exported patches like we do in Blink. But what about the above case, when the change hasn't been applied in WebKit yet? Seems like the importer instead needs to *revert* the in-flight change?</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>