Failed to upload; bug 612257.
One possible workaround might be to make the change with multiple CLs. If that were done, this might cause the tests to fail on the Blink waterfall for some in between patches. Tree sheriff would have to be aware of this.
We should separate it to multiple CLs. codereview.chromium.org can't handle huge CL.
We should be able to separate it without breaking bots. e.g. If a CL moves imported/web-platform-tests/custom-elements/ to imported/wpt/custom-elements/, the CL should update imported/web-platform-tests/custom-elements entries in expectation files too.
Raise to Pri-1 because this blocks importing new revision of WPT.
Sorry everyone for the spam for the series of little patches today -- at this point, all directories are moved except for a couple:
http://crrev.com/1991603002 FileAPI
http://crrev.com/1991613002 html
After those, there should be another CL to change any leftover references to imported/web-platform-tests in Tools/Scripts/webkitpy, and then another CL can be made to finally update the imported copies.
Note, this morning Dirk gave some advice about how changes like this could be handled better in the future:
For a large refactoring CL (e.g. moving a directory), which Rietveld can't handle, it may make more sense to commit without Rietveld using git cl land (git cl land will warn if you haven't uploaded the change; so one can upload a small subset of the change, then land the whole thing, with more details about exactly what is happening in the commit description).
Comment 1 by qyears...@chromium.org
, May 13 2016