In update-w3c-test-expectations, handle failure to fetch results.

In update-w3c-test-expectations, handle failure to fetch results.

Specifically, when no layout tests are fetched for a particular build,
log a warning and continue instead of raising an exception and aborting.

I noticed this error when running the script on my workstation.

This CL also adds several TODO notes.

BUG=629275
Committed: https://crrev.com/0f77323cdf30b116f8055b11ef890d6a835b2044
Cr-Commit-Position: refs/heads/master@{#414497}

[qyearsley, 2016-08-25 16:00:41]

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

[qyearsley, 2016-08-25 16:02:47]

https://codereview.chromium.org/2277303003/diff/1/third_party/WebKit/Tools/Sc...
File
third_party/WebKit/Tools/Scripts/webkitpy/w3c/update_w3c_test_expectations_unittest.py
(right):

https://codereview.chromium.org/2277303003/diff/1/third_party/WebKit/Tools/Sc...
third_party/WebKit/Tools/Scripts/webkitpy/w3c/update_w3c_test_expectations_unittest.py:16:
class UpdateW3CTestExpectationsTest(unittest.TestCase,
W3CExpectationsLineAdder):
Another TODO which I plan to do as an immediate follow-up to this CL:

Make this class not subclass W3CExpectationsLineAdder.

[Dirk Pranke, 2016-08-25 17:20:15]

lgtm

[qyearsley, 2016-08-25 17:23:10]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2016-08-25 17:23:43]

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, 2016-08-25 18:59:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-25 19:03:02]

Description was changed from

==========
In update-w3c-test-expectations, handle failure to fetch results.

Specifically, when no layout tests are fetched for a particular build,
log a warning and continue instead of raising an exception and aborting.

I noticed this error when running the script on my workstation.

This CL also adds several TODO notes.

BUG=629275
==========

to

==========
In update-w3c-test-expectations, handle failure to fetch results.

Specifically, when no layout tests are fetched for a particular build,
log a warning and continue instead of raising an exception and aborting.

I noticed this error when running the script on my workstation.

This CL also adds several TODO notes.

BUG=629275

Committed: https://crrev.com/0f77323cdf30b116f8055b11ef890d6a835b2044
Cr-Commit-Position: refs/heads/master@{#414497}
==========

[commit-bot: I haz the power, 2016-08-25 19:03:03]

Patchset 1 (id:??) landed as
https://crrev.com/0f77323cdf30b116f8055b11ef890d6a835b2044
Cr-Commit-Position: refs/heads/master@{#414497}