Extract test directory owner email addresses from W3CImportExpectations.

Extract test directory owner email addresses from W3CImportExpectations.

The purpose of this CL is to improve the list of people to CC that the
w3c-test-autoroller generates -- previously the list was based on a file
directory_owners.json, which was generated once (from a spreadsheet)
and had only a couple of (mailing list) addresses. This CL would make
it so that directory owners are extracted from W3CImportExpectations.

Another alternate approach would be to use W3CImportExpectations to
generate a JSON file, and read that, but then that way we would have to
regularly update the JSON file.


BUG=684026
Review-Url: https://codereview.chromium.org/2649363003
Cr-Commit-Position: refs/heads/master@{#446157}
Committed: https://chromium.googlesource.com/chromium/src/+/4edc8aac446bd6bc319b9430ed1bed30391f1ff1

[qyearsley, 2017-01-25 00:35:45]

Description was changed from

==========
Extract test directory owner email addresses from W3CImportExpectations.

BUG=684026
==========

to

==========
Extract test directory owner email addresses from W3CImportExpectations.

The purpose of this CL is to improve the list of people to CC that the
w3c-test-autoroller generates -- previously the list was based on a file
directory_owners.json, which was generated once (from a spreadsheet) and had
only a couple of (mailing list) addresses. This CL would make it so that
directory owners are extracted from W3CImportExpectations.

Another alternate approach would be to use W3CImportExpectations to generate a
JSON file, and read that, but then that way we would have to regularly update
the JSON file.

BUG=684026
==========

[qyearsley, 2017-01-25 00:35:45]

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

[qyearsley, 2017-01-25 00:36:30]

Description was changed from

==========
Extract test directory owner email addresses from W3CImportExpectations.

The purpose of this CL is to improve the list of people to CC that the
w3c-test-autoroller generates -- previously the list was based on a file
directory_owners.json, which was generated once (from a spreadsheet) and had
only a couple of (mailing list) addresses. This CL would make it so that
directory owners are extracted from W3CImportExpectations.

Another alternate approach would be to use W3CImportExpectations to generate a
JSON file, and read that, but then that way we would have to regularly update
the JSON file.

BUG=684026
==========

to

==========
Extract test directory owner email addresses from W3CImportExpectations.

The purpose of this CL is to improve the list of people to CC that the
w3c-test-autoroller generates -- previously the list was based on a file
directory_owners.json, which was generated once (from a spreadsheet)
and had only a couple of (mailing list) addresses. This CL would make
it so that directory owners are extracted from W3CImportExpectations.

Another alternate approach would be to use W3CImportExpectations to
generate a JSON file, and read that, but then that way we would have to
regularly update the JSON file.


BUG=684026
==========

[qyearsley, 2017-01-25 00:36:35]

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

[commit-bot: I haz the power, 2017-01-25 00:37:31]

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

[tkent, 2017-01-25 01:02:13]

lgtm

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

https://codereview.chromium.org/2649363003/diff/1/third_party/WebKit/Tools/Sc...
third_party/WebKit/Tools/Scripts/webkitpy/w3c/directory_owners_extractor_unittest.py:57:
self.assertEqual(self.extractor.extract_owners('## Owners: a@x.com,b@x.com'),
['a@x.com', 'b@x.com'])
Add a testcase where we have a space after a comma.  e.g. '## Owners: a@x.com,
b@x.com'

[commit-bot: I haz the power, 2017-01-25 01:51:08]

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

[commit-bot: I haz the power, 2017-01-25 01:51:09]

Dry run: This issue passed the CQ dry run.

[qyearsley, 2017-01-25 21:10:03]

The CQ bit was checked by qyearsley@chromium.org

[qyearsley, 2017-01-25 21:10:03]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/2649363003/#ps20001
(title: "Handle spaces after commas; remove now-unused script.")

[commit-bot: I haz the power, 2017-01-25 21:11:01]

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

[qyearsley, 2017-01-25 23:01:25]

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

https://codereview.chromium.org/2649363003/diff/1/third_party/WebKit/Tools/Sc...
third_party/WebKit/Tools/Scripts/webkitpy/w3c/directory_owners_extractor_unittest.py:57:
self.assertEqual(self.extractor.extract_owners('## Owners: a@x.com,b@x.com'),
['a@x.com', 'b@x.com'])
On 2017/01/25 at 01:02:13, tkent wrote:
> Add a testcase where we have a space after a comma.  e.g. '## Owners: a@x.com,
b@x.com'

Done; thanks

[commit-bot: I haz the power, 2017-01-25 23:10:44]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1485378603191930,
"parent_rev": "b8cf426b59b931395bfe5114c7c6952473f210ce", "commit_rev":
"4edc8aac446bd6bc319b9430ed1bed30391f1ff1"}

[commit-bot: I haz the power, 2017-01-25 23:11:19]

Description was changed from

==========
Extract test directory owner email addresses from W3CImportExpectations.

The purpose of this CL is to improve the list of people to CC that the
w3c-test-autoroller generates -- previously the list was based on a file
directory_owners.json, which was generated once (from a spreadsheet)
and had only a couple of (mailing list) addresses. This CL would make
it so that directory owners are extracted from W3CImportExpectations.

Another alternate approach would be to use W3CImportExpectations to
generate a JSON file, and read that, but then that way we would have to
regularly update the JSON file.


BUG=684026
==========

to

==========
Extract test directory owner email addresses from W3CImportExpectations.

The purpose of this CL is to improve the list of people to CC that the
w3c-test-autoroller generates -- previously the list was based on a file
directory_owners.json, which was generated once (from a spreadsheet)
and had only a couple of (mailing list) addresses. This CL would make
it so that directory owners are extracted from W3CImportExpectations.

Another alternate approach would be to use W3CImportExpectations to
generate a JSON file, and read that, but then that way we would have to
regularly update the JSON file.


BUG=684026

Review-Url: https://codereview.chromium.org/2649363003
Cr-Commit-Position: refs/heads/master@{#446157}
Committed:
https://chromium.googlesource.com/chromium/src/+/4edc8aac446bd6bc319b9430ed1b...
==========

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

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/4edc8aac446bd6bc319b9430ed1b...