On W3C test import, always skip large files that may fail to upload.

On W3C test import, always skip large files that may fail to upload.

This CL also removes a skip from W3CImportExpectations that should no
longer be necessary.

BUG=676491
Review-Url: https://codereview.chromium.org/2628913003
Cr-Commit-Position: refs/heads/master@{#443163}
Committed: https://chromium.googlesource.com/chromium/src/+/dff0c445aee94798e03534796a0af11eb459d3a9

[qyearsley, 2017-01-11 19:02:40]

qyearsley@chromium.org changed reviewers:
+ jeffcarp@chromium.org

[qyearsley, 2017-01-11 19:02:40]

This is a more general fix for the issue that the test importer ran into the
past few days with domxpath/xml_xpath_tests.xml.

This isn't urgent, but should hopefully prevent the importer getting stuck on a
very similar issue in the future.

[qyearsley, 2017-01-11 19:02:45]

The CQ bit was checked by qyearsley@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-11 19:03:48]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-11 20:20:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-11 20:20:34]

Dry run: This issue passed the CQ dry run.

[jeffcarp, 2017-01-12 02:43:07]

lgtm

[qyearsley, 2017-01-12 05:47:36]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2017-01-12 05:47:56]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-12 05:51:57]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484200056971800, "parent_rev":
"f77fda1001064005346121f403bb80b3414cbc00", "commit_rev":
"dff0c445aee94798e03534796a0af11eb459d3a9"}

[commit-bot: I haz the power, 2017-01-12 05:52:48]

Description was changed from

==========
On W3C test import, always skip large files that may fail to upload.

This CL also removes a skip from W3CImportExpectations that should no
longer be necessary.

BUG=676491
==========

to

==========
On W3C test import, always skip large files that may fail to upload.

This CL also removes a skip from W3CImportExpectations that should no
longer be necessary.

BUG=676491

Review-Url: https://codereview.chromium.org/2628913003
Cr-Commit-Position: refs/heads/master@{#443163}
Committed:
https://chromium.googlesource.com/chromium/src/+/dff0c445aee94798e03534796a0a...
==========

[commit-bot: I haz the power, 2017-01-12 05:52:49]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/dff0c445aee94798e03534796a0a...

[qyearsley, 2017-01-13 20:28:19]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2630813002/ by qyearsley@chromium.org.

The reason for reverting is: I believe that this didn't actually solve the
problem properly - 
In the latest import, some tests are failing because they depend on files that
are too large that were already checked in (in media/*), and after this change,
the files are removed but the tests are still imported, causing this failure:
https://build.chromium.org/p/chromium.infra.cron/builders/w3c-test-autoroller....