Drop self from API_OWNERS, OWNERS, etc.

Drop self from API_OWNERS, OWNERS, etc.

tkent@ takes over web-platform-tests/dom and chcunningham@ joins
wolenetz@ in mediasource/OWNERS.

Committed: https://crrev.com/c8e42b0a1c8fcd6389738414543f21ec5b2f4aed
Cr-Commit-Position: refs/heads/master@{#390604}

[philipj_slow, 2016-04-27 14:06:42]

Description was changed from

==========
Drop self from API_OWNERS, OWNERS and WATCHLISTS

R=rbyers@chromium.org
==========

to

==========
Drop self from API_OWNERS, OWNERS, etc.

R=rbyers@chromium.org
==========

[philipj_slow, 2016-04-27 14:07:01]

Description was changed from

==========
Drop self from API_OWNERS, OWNERS, etc.

R=rbyers@chromium.org
==========

to

==========
Drop self from API_OWNERS, OWNERS, etc.
==========

[philipj_slow, 2016-04-27 14:07:44]

philipj@opera.com changed reviewers:
+ jochen@chromium.org

[philipj_slow, 2016-04-27 14:07:45]

Last week at Opera, can I have an lgtm on this so that I can land it on Friday?

[Rick Byers, 2016-04-27 14:29:37]

rbyers@chromium.org changed reviewers:
+ wolenetz@chromium.org

[Rick Byers, 2016-04-27 14:31:08]

Grudgingly LGTM :-(

https://codereview.chromium.org/1919183004/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/W3CImportExpectations (right):

https://codereview.chromium.org/1919183004/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/W3CImportExpectations:212: ## Owners: TBD
tkent has been working in this space, perhaps he should be listed as an owner?

https://codereview.chromium.org/1919183004/diff/1/third_party/WebKit/Source/m...
File third_party/WebKit/Source/modules/mediasource/OWNERS (left):

https://codereview.chromium.org/1919183004/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/mediasource/OWNERS:1: philipj@opera.com
Is there someone else you could add here?  In general we don't want to have
OWNERS files with just a single entry (vacation etc.).  /cc wolenetz@ any
suggestions?

[jochen (gone - plz use gerrit), 2016-04-27 14:34:20]

lgtm

[philipj_slow, 2016-04-27 14:43:32]

philipj@opera.com changed reviewers:
+ tkent@chromium.org

[philipj_slow, 2016-04-27 14:43:33]

https://codereview.chromium.org/1919183004/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/W3CImportExpectations (right):

https://codereview.chromium.org/1919183004/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/W3CImportExpectations:212: ## Owners: TBD
On 2016/04/27 14:31:08, Rick Byers wrote:
> tkent has been working in this space, perhaps he should be listed as an owner?

tkent@, can I add you here?

[tkent, 2016-04-27 16:02:49]

https://codereview.chromium.org/1919183004/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/W3CImportExpectations (right):

https://codereview.chromium.org/1919183004/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/W3CImportExpectations:212: ## Owners: TBD
On 2016/04/27 at 14:43:33, philipj_slow wrote:
> On 2016/04/27 14:31:08, Rick Byers wrote:
> > tkent has been working in this space, perhaps he should be listed as an
owner?
> 
> tkent@, can I add you here?

yes, I'll take it over.

[wolenetz, 2016-04-27 20:18:57]

I don't have a strong suggestion for an MSE modules co-owner:

https://codereview.chromium.org/1919183004/diff/1/third_party/WebKit/Source/m...
File third_party/WebKit/Source/modules/mediasource/OWNERS (left):

https://codereview.chromium.org/1919183004/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/mediasource/OWNERS:1: philipj@opera.com
On 2016/04/27 14:31:08, Rick Byers wrote:
> Is there someone else you could add here?  In general we don't want to have
> OWNERS files with just a single entry (vacation etc.).  /cc wolenetz@ any
> suggestions?

Hmm. servolk@ works mainly in downstream Chromecast, but has made contributions
to MSE, though possibly not qualifying as OWNER yet. I'm thinking this MSE
module would continue to benefit from having an OWNER that is strong in their
Blink/OILPAN/etc experience. philipj@: might sigbjornf@opera.com qualify here?

[wolenetz, 2016-04-27 21:22:38]

wolenetz@chromium.org changed reviewers:
+ chcunningham@chromium.org

[wolenetz, 2016-04-27 21:23:33]

On 2016/04/27 20:18:57, wolenetz wrote:
> I don't have a strong suggestion for an MSE modules co-owner:
> 
>
https://codereview.chromium.org/1919183004/diff/1/third_party/WebKit/Source/m...
> File third_party/WebKit/Source/modules/mediasource/OWNERS (left):
> 
>
https://codereview.chromium.org/1919183004/diff/1/third_party/WebKit/Source/m...
> third_party/WebKit/Source/modules/mediasource/OWNERS:1:
mailto:philipj@opera.com
> On 2016/04/27 14:31:08, Rick Byers wrote:
> > Is there someone else you could add here?  In general we don't want to have
> > OWNERS files with just a single entry (vacation etc.).  /cc wolenetz@ any
> > suggestions?
> 
> Hmm. servolk@ works mainly in downstream Chromecast, but has made
contributions
> to MSE, though possibly not qualifying as OWNER yet. I'm thinking this MSE
> module would continue to benefit from having an OWNER that is strong in their
> Blink/OILPAN/etc experience. philipj@: might mailto:sigbjornf@opera.com
qualify here?

On further reflection, I myself don't have enough Blink/OILPAN/etc experience,
though I do have deep experience with the MSE spec. So for a committer that
could be an OWNER here based on similar criteria, I recommend chcunningham@ as
MSE modules co-owner due to his experience with the MSE spec, too. We will
include blink experts as appropriate on CRs in MSE modules.

[philipj_slow, 2016-04-28 09:36:43]

Description was changed from

==========
Drop self from API_OWNERS, OWNERS, etc.
==========

to

==========
Drop self from API_OWNERS, OWNERS, etc.

tkent@ takes over web-platform-tests/dom and chcunningham@ joins
wolenetz@ in mediasource/OWNERS.
==========

[philipj_slow, 2016-04-28 09:37:42]

On 2016/04/27 21:23:33, wolenetz wrote:
> On 2016/04/27 20:18:57, wolenetz wrote:
> > I don't have a strong suggestion for an MSE modules co-owner:
> > 
> >
>
https://codereview.chromium.org/1919183004/diff/1/third_party/WebKit/Source/m...
> > File third_party/WebKit/Source/modules/mediasource/OWNERS (left):
> > 
> >
>
https://codereview.chromium.org/1919183004/diff/1/third_party/WebKit/Source/m...
> > third_party/WebKit/Source/modules/mediasource/OWNERS:1:
> mailto:philipj@opera.com
> > On 2016/04/27 14:31:08, Rick Byers wrote:
> > > Is there someone else you could add here?  In general we don't want to
have
> > > OWNERS files with just a single entry (vacation etc.).  /cc wolenetz@ any
> > > suggestions?
> > 
> > Hmm. servolk@ works mainly in downstream Chromecast, but has made
> contributions
> > to MSE, though possibly not qualifying as OWNER yet. I'm thinking this MSE
> > module would continue to benefit from having an OWNER that is strong in
their
> > Blink/OILPAN/etc experience. philipj@: might mailto:sigbjornf@opera.com
> qualify here?
> 
> On further reflection, I myself don't have enough Blink/OILPAN/etc experience,
> though I do have deep experience with the MSE spec. So for a committer that
> could be an OWNER here based on similar criteria, I recommend chcunningham@ as
> MSE modules co-owner due to his experience with the MSE spec, too. We will
> include blink experts as appropriate on CRs in MSE modules.

I've added chcunningham@ here and tkent@ for web-platform-tests/dom, so if there
are no further comments I'll land this tomorrow.

[philipj_slow, 2016-04-29 08:17:38]

The CQ bit was checked by philipj@opera.com

[philipj_slow, 2016-04-29 08:17:39]

The patchset sent to the CQ was uploaded after l-g-t-m from jochen@chromium.org,
rbyers@chromium.org
Link to the patchset: https://codereview.chromium.org/1919183004/#ps20001
(title: "tkent+chcunningham")

[commit-bot: I haz the power, 2016-04-29 08:17:46]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1919183004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1919183004/20001

[commit-bot: I haz the power, 2016-04-29 09:18:54]

Description was changed from

==========
Drop self from API_OWNERS, OWNERS, etc.

tkent@ takes over web-platform-tests/dom and chcunningham@ joins
wolenetz@ in mediasource/OWNERS.
==========

to

==========
Drop self from API_OWNERS, OWNERS, etc.

tkent@ takes over web-platform-tests/dom and chcunningham@ joins
wolenetz@ in mediasource/OWNERS.
==========

[commit-bot: I haz the power, 2016-04-29 09:18:55]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-30 17:24:41]

Patchset 2 (id:??) landed as
https://crrev.com/c8e42b0a1c8fcd6389738414543f21ec5b2f4aed
Cr-Commit-Position: refs/heads/master@{#390604}

[commit-bot: I haz the power, 2016-04-30 17:59:54]

Description was changed from

==========
Drop self from API_OWNERS, OWNERS, etc.

tkent@ takes over web-platform-tests/dom and chcunningham@ joins
wolenetz@ in mediasource/OWNERS.
==========

to

==========
Drop self from API_OWNERS, OWNERS, etc.

tkent@ takes over web-platform-tests/dom and chcunningham@ joins
wolenetz@ in mediasource/OWNERS.

Committed: https://crrev.com/c8e42b0a1c8fcd6389738414543f21ec5b2f4aed
Cr-Commit-Position: refs/heads/master@{#390604}
==========