Probably not blocking  bug 657117 , since this is a refactoring change.

This would be a change at https://cs.chromium.org/chromium/src/third_party/WebKit/Tools/Scripts/webkitpy/w3c/local_wpt.py&l=21; when this is done, the related unit test methods should be renamed too :-)

This reminds me of the comment at https://cs.chromium.org/chromium/src/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_expectations.py?l=1008 -- the TestExpectations constructor also does a lot of work.

Good point (I just put everything as blocking  bug 657117  by default to keep track, but maybe that's not the right way to do that).

I'm conflicted though. LocalWPT has a 'fat' constructor because the class is unusable without it. But the expectation is that constructors shouldn't be making network calls or anything with high latency. So I'm still in favor of doing this for LocalWPT and TestExpectations, as you mentioned.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1af3b5587d608b0417f3784703f8d389742cded0

commit 1af3b5587d608b0417f3784703f8d389742cded0
Author: qyearsley <qyearsley@chromium.org>
Date: Wed Jan 18 22:44:08 2017

Move WPT fetch out of the LocalWPT constructor.

BUG= 679955 

Review-Url: https://codereview.chromium.org/2627323008
Cr-Commit-Position: refs/heads/master@{#444522}

[modify] https://crrev.com/1af3b5587d608b0417f3784703f8d389742cded0/third_party/WebKit/Tools/Scripts/webkitpy/w3c/local_wpt.py
[modify] https://crrev.com/1af3b5587d608b0417f3784703f8d389742cded0/third_party/WebKit/Tools/Scripts/webkitpy/w3c/local_wpt_unittest.py
[modify] https://crrev.com/1af3b5587d608b0417f3784703f8d389742cded0/third_party/WebKit/Tools/Scripts/webkitpy/w3c/test_exporter.py