Update Blink OWNERS file for tooling info.

Update Blink OWNERS file for tooling info.

This CL updates the OWNERS information for
//third_party/WebKit/Tools and //blink/tools to be current
(including adding TEAM and COMPONENT info), and
also adds a preparatory OWNER stub for //blink and tweaks
a few lines in //third_party/WebKit.

This CL removes jochen@, peter@, and tony@ as owners, as they haven't
committed anything to the tools code in some time, and adds
the current contributors. I should probably remove myself as
an owner as well, but I'll let that go for a few more months.

This CL also removes the wildcard "*" owner from Tools/Scripts, as there's
no needd for that to be a wildcard these days, and the blink infra folks
should review all changes.

R=qyearsley@chromium.org, jeffcarp@chromium.org, tansell@chromium.org, wangxianzhu@chromium.org, tkent@chromium.org, jochen@chromium.org, peter@chromium.org

Review-Url: https://codereview.chromium.org/2800863004
Cr-Commit-Position: refs/heads/master@{#463002}
Committed: https://chromium.googlesource.com/chromium/src/+/f0967b500becdadb10fa3b91bea592105fb74730

[Dirk Pranke, 2017-04-07 01:46:06]

Description was changed from

==========
Update Blink OWNERS file for tooling info.

This CL updates the OWNERS information for
//third_party/WebKit/Tools and //blink/tools to be current
(including adding TEAM and COMPONENT info), and
also adds a preparatory OWNER stub for //blink and tweaks
a few lines in //third_party/WebKit.

This CL removes jochen@ and peter@ as owners, as they haven't
committed anything to the tools code in some time, and adds
the current contributors. I should probably remove myself as
an owner as well, but I'll let that go for a few more months.

R=qyearsley@chromium.org, jeffcarp@chromium.org, tansell@chromium.org,
wangxianzhu@chromium.org, tkent@chromium.org, jochen@chromium.org,
peter@chromium.org
==========

to

==========
Update Blink OWNERS file for tooling info.

This CL updates the OWNERS information for
//third_party/WebKit/Tools and //blink/tools to be current
(including adding TEAM and COMPONENT info), and
also adds a preparatory OWNER stub for //blink and tweaks
a few lines in //third_party/WebKit.

This CL removes jochen@ and peter@ as owners, as they haven't
committed anything to the tools code in some time, and adds
the current contributors. I should probably remove myself as
an owner as well, but I'll let that go for a few more months.

R=qyearsley@chromium.org, jeffcarp@chromium.org, tansell@chromium.org,
wangxianzhu@chromium.org, tkent@chromium.org, jochen@chromium.org,
peter@chromium.org
==========

[Dirk Pranke, 2017-04-07 01:47:10]

Description was changed from

==========
Update Blink OWNERS file for tooling info.

This CL updates the OWNERS information for
//third_party/WebKit/Tools and //blink/tools to be current
(including adding TEAM and COMPONENT info), and
also adds a preparatory OWNER stub for //blink and tweaks
a few lines in //third_party/WebKit.

This CL removes jochen@ and peter@ as owners, as they haven't
committed anything to the tools code in some time, and adds
the current contributors. I should probably remove myself as
an owner as well, but I'll let that go for a few more months.

R=qyearsley@chromium.org, jeffcarp@chromium.org, tansell@chromium.org,
wangxianzhu@chromium.org, tkent@chromium.org, jochen@chromium.org,
peter@chromium.org
==========

to

==========
Update Blink OWNERS file for tooling info.

This CL updates the OWNERS information for
//third_party/WebKit/Tools and //blink/tools to be current
(including adding TEAM and COMPONENT info), and
also adds a preparatory OWNER stub for //blink and tweaks
a few lines in //third_party/WebKit.

This CL removes jochen@, peter@, and tony@ as owners, as they haven't
committed anything to the tools code in some time, and adds
the current contributors. I should probably remove myself as
an owner as well, but I'll let that go for a few more months.

This CL also removes the wildcard "*" owner from Tools/Scripts, as there's
no needd for that to be a wildcard these days, and the blink infra folks
should review all changes.

R=qyearsley@chromium.org, jeffcarp@chromium.org, tansell@chromium.org,
wangxianzhu@chromium.org, tkent@chromium.org, jochen@chromium.org,
peter@chromium.org
==========

[Dirk Pranke, 2017-04-07 01:47:30]

Hi all, 

Please let me know if you have any objections to any of these changes.

[tkent, 2017-04-07 01:50:42]

We won't use //blink, but anyway this CL lgtm.

[Peter Beverloo, 2017-04-07 01:52:09]

also lgtm

[mithro, 2017-04-07 06:40:29]

lgtm

[jochen (gone - plz use gerrit), 2017-04-07 06:52:18]

hum, not sure whether contributing code should be a precondition for remaining
an owner

In general, I'd prefer if we had owner coverage around the globe, but you're now
removing everybody from emea.

otoh, I'm still in the top-level file, so practically speaking, it doesn't
matter..

[qyearsley, 2017-04-07 16:01:20]

No objections, lgtm

[jeffcarp, 2017-04-07 16:29:03]

lgtm

[Dirk Pranke, 2017-04-07 18:18:58]

On 2017/04/07 06:52:18, jochen wrote:
> hum, not sure whether contributing code should be a precondition for remaining
> an owner

It seems like a pretty good reason to me, since people not involved with the
codebase
may not know what changes are consistent w/ where the codebase is headed.

> In general, I'd prefer if we had owner coverage around the globe, but you're
now
> removing everybody from emea.

I'm fine w/ leaving you in. I agree that it would be good to have emea coverage,
but
on the other hand there isn't anyone in emea working on this code these days.
 
> otoh, I'm still in the top-level file, so practically speaking, it doesn't
> matter..

Yeah :). It's also good to have the OWNERS files closest to the code try to
point 
you to the best people to review things.

[Dirk Pranke, 2017-04-07 18:19:01]

The CQ bit was checked by dpranke@chromium.org

[commit-bot: I haz the power, 2017-04-07 18:20:03]

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-04-07 21:09:55]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491589141673970, "parent_rev":
"bbf439562c0b21668040b2c10e560f32dcd42048", "commit_rev":
"f0967b500becdadb10fa3b91bea592105fb74730"}

[commit-bot: I haz the power, 2017-04-07 21:10:48]

Description was changed from

==========
Update Blink OWNERS file for tooling info.

This CL updates the OWNERS information for
//third_party/WebKit/Tools and //blink/tools to be current
(including adding TEAM and COMPONENT info), and
also adds a preparatory OWNER stub for //blink and tweaks
a few lines in //third_party/WebKit.

This CL removes jochen@, peter@, and tony@ as owners, as they haven't
committed anything to the tools code in some time, and adds
the current contributors. I should probably remove myself as
an owner as well, but I'll let that go for a few more months.

This CL also removes the wildcard "*" owner from Tools/Scripts, as there's
no needd for that to be a wildcard these days, and the blink infra folks
should review all changes.

R=qyearsley@chromium.org, jeffcarp@chromium.org, tansell@chromium.org,
wangxianzhu@chromium.org, tkent@chromium.org, jochen@chromium.org,
peter@chromium.org
==========

to

==========
Update Blink OWNERS file for tooling info.

This CL updates the OWNERS information for
//third_party/WebKit/Tools and //blink/tools to be current
(including adding TEAM and COMPONENT info), and
also adds a preparatory OWNER stub for //blink and tweaks
a few lines in //third_party/WebKit.

This CL removes jochen@, peter@, and tony@ as owners, as they haven't
committed anything to the tools code in some time, and adds
the current contributors. I should probably remove myself as
an owner as well, but I'll let that go for a few more months.

This CL also removes the wildcard "*" owner from Tools/Scripts, as there's
no needd for that to be a wildcard these days, and the blink infra folks
should review all changes.

R=qyearsley@chromium.org, jeffcarp@chromium.org, tansell@chromium.org,
wangxianzhu@chromium.org, tkent@chromium.org, jochen@chromium.org,
peter@chromium.org

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

[commit-bot: I haz the power, 2017-04-07 21:10:49]

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

[mithro, 2017-04-10 01:35:19]

Maybe someone like Pawel or Sergiy could help provide EMEA coverage. They aren't
really working on Blink tooling but do have a lot of context about more general
Chrome Infa.

[jochen (gone - plz use gerrit), 2017-04-10 07:49:31]

On 2017/04/10 at 01:35:19, tansell wrote:
> Maybe someone like Pawel or Sergiy could help provide EMEA coverage. They
aren't really working on Blink tooling but do have a lot of context about more
general Chrome Infa.

I don't think that makes sense - I'd claim to still have more context about the
stuff there (and as I noted in comment #7, I'm still an owner anyways, as I'm in
the blink/OWNERS file)