WPT Auto-import: fix for owner list in import CLs.

WPT Auto-import: fix for owner list in import CLs.

This is a follow-up fix for http://crrev.com/2684663003.

The mistake was: In one part of the code I was assuming that
get_directory_owners() would return a map from owner to list of
directories, but in reality it returned a dict from owner to single
directory (string).

We want to have lists of directories, since a single import CL
will often have modifications to many directories for one owner.

This resulted in silly-looking CL descriptions like
http://crrev.com/2687343002.

BUG=687270
Review-Url: https://codereview.chromium.org/2684423004
Cr-Commit-Position: refs/heads/master@{#449666}
Committed: https://chromium.googlesource.com/chromium/src/+/0ccb652d00c6fbd072608a7bdbb9f1fbe17fd87d

[qyearsley, 2017-02-10 17:21:03]

Description was changed from

==========
WPT Auto-import: fix for owner list in import CLs.

This is a follow-up fix for http://crrev.com/2684663003.

The mistake was: In one part of the code I was assuming that
get_directory_owners() would return a map from owner to list of
directories, but in reality it returned a dict from owner to single
directory (string).

This resulted in silly-looking CL descriptions like
http://crrev.com/2687343002.

BUG=687270
==========

to

==========
WPT Auto-import: fix for owner list in import CLs.

This is a follow-up fix for http://crrev.com/2684663003.

The mistake was: In one part of the code I was assuming that
get_directory_owners() would return a map from owner to list of
directories, but in reality it returned a dict from owner to single
directory (string).

We want to have lists of directories, since a single import CL
will often have modifications to many directories for one owner.

This resulted in silly-looking CL descriptions like
http://crrev.com/2687343002.

BUG=687270
==========

[qyearsley, 2017-02-10 17:21:03]

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

[qyearsley, 2017-02-10 17:21:13]

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

[commit-bot: I haz the power, 2017-02-10 17:21:50]

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

[jeffcarp, 2017-02-10 17:26:34]

lgtm

[qyearsley, 2017-02-10 17:33:24]

The CQ bit was unchecked by qyearsley@chromium.org

[qyearsley, 2017-02-10 17:33:25]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2017-02-10 17:33:41]

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-02-10 18:34:38]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486748005930560, "parent_rev":
"a446224d4d9f2ffd3b0598d5c7647c886890a8ce", "commit_rev":
"0ccb652d00c6fbd072608a7bdbb9f1fbe17fd87d"}

[commit-bot: I haz the power, 2017-02-10 18:36:20]

Description was changed from

==========
WPT Auto-import: fix for owner list in import CLs.

This is a follow-up fix for http://crrev.com/2684663003.

The mistake was: In one part of the code I was assuming that
get_directory_owners() would return a map from owner to list of
directories, but in reality it returned a dict from owner to single
directory (string).

We want to have lists of directories, since a single import CL
will often have modifications to many directories for one owner.

This resulted in silly-looking CL descriptions like
http://crrev.com/2687343002.

BUG=687270
==========

to

==========
WPT Auto-import: fix for owner list in import CLs.

This is a follow-up fix for http://crrev.com/2684663003.

The mistake was: In one part of the code I was assuming that
get_directory_owners() would return a map from owner to list of
directories, but in reality it returned a dict from owner to single
directory (string).

We want to have lists of directories, since a single import CL
will often have modifications to many directories for one owner.

This resulted in silly-looking CL descriptions like
http://crrev.com/2687343002.

BUG=687270

Review-Url: https://codereview.chromium.org/2684423004
Cr-Commit-Position: refs/heads/master@{#449666}
Committed:
https://chromium.googlesource.com/chromium/src/+/0ccb652d00c6fbd072608a7bdbb9...
==========

[commit-bot: I haz the power, 2017-02-10 18:36:21]

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