Make all Blink API OWNERS be Source/OWNERS

Make all Blink API OWNERS be Source/OWNERS

We had an explicit list of these folks for modules,
but given that these individuals are charged with
Blink's long-term stewardship, it makes sense that
they should be able to review all changes across
Blink's source.

This also makes this list publicly documented
which we had failed to do so in the past.

R=abarth@chromium.org, dglazkov@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=179975

[dglazkov, 2014-08-05 21:32:50]

lgtm

[abarth-chromium, 2014-08-05 21:33:42]

LGTM

[commit-bot: I haz the power, 2014-08-05 21:35:41]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/eseidel@chromium.org/441083002/1

[Peter Beverloo, 2014-08-05 23:41:11]

On 2014/08/05 21:35:41, I haz the power (commit-bot) wrote:
> CQ is trying da patch. Follow status at
>  https://chromium-status.appspot.com/cq/eseidel@chromium.org/441083002/1

He's also OWNER of //Tools and //public. It makes more sense to just move him in
the main OWNERS file, since there is a grand total of four *files* in the
repository he's not an OWNER of :-).

[Peter Beverloo, 2014-08-05 23:42:14]

On 2014/08/05 23:41:11, Peter Beverloo wrote:
> On 2014/08/05 21:35:41, I haz the power (commit-bot) wrote:
> > CQ is trying da patch. Follow status at
> >  https://chromium-status.appspot.com/cq/eseidel@chromium.org/441083002/1
> 
> He's also OWNER of //Tools and //public. It makes more sense to just move him
in
> the main OWNERS file, since there is a grand total of four *files* in the
> repository he's not an OWNER of :-).

Err, clicked on the wrong link in the e-mail for
https://codereview.chromium.org/347703002/ :).

[commit-bot: I haz the power, 2014-08-06 02:47:38]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/1...)

[commit-bot: I haz the power, 2014-08-06 03:23:53]

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

[commit-bot: I haz the power, 2014-08-06 03:23:53]

Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/1...)

[eseidel, 2014-08-11 17:13:02]

The CQ bit was checked by eseidel@chromium.org

[commit-bot: I haz the power, 2014-08-11 17:13:21]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/eseidel@chromium.org/441083002/1

[commit-bot: I haz the power, 2014-08-11 17:39:44]

Change committed as 179975

[jamesr, 2014-08-11 17:58:40]

I like this, but it's a little confusing now that the 'Blink API owners'
documented in Source/OWNERS is different from the owners of the Blink public C++
API (aka public/OWNERS).  The sets overlap except for Pavel, Eric and I.  Maybe
you could put a link to the document describing what 'Blink API OWNERS' means in
Source/OWNERS?  We have http://www.chromium.org/blink/public-c-api and a README
for public/

[eseidel, 2014-08-11 18:03:59]

Agreed.  There is a doc here: http://www.chromium.org/blink/#TOC-API-Owners but
it's wrong. :/