Make WebSocket people per-file OWNERS of net/socket for files starting with "websocket"

Make WebSocket people per-file OWNERS of net/socket for files starting with "websocket"

R=mmenke@chromium.org
BUG=none

Committed: https://crrev.com/3a00cb3814f26705029307ff96db48ef036a0183
Cr-Commit-Position: refs/heads/master@{#418538}

[tyoshino (SeeGerritForStatus), 2016-09-14 07:39:01]

Description was changed from

==========
Make WebSocket people per-file OWNERS of net/socket for files starting with
"websocket"

BUG=none
==========

to

==========
Make WebSocket people per-file OWNERS of net/socket for files starting with
"websocket"

BUG=none
==========

[tyoshino (SeeGerritForStatus), 2016-09-14 07:39:02]

tyoshino@chromium.org changed reviewers:
+ mmenke@chromium.org

[mmenke, 2016-09-14 08:47:06]

LGTM, modulo comment

https://codereview.chromium.org/2335133005/diff/1/net/socket/OWNERS
File net/socket/OWNERS (right):

https://codereview.chromium.org/2335133005/diff/1/net/socket/OWNERS#newcode2
net/socket/OWNERS:2: per-file websocket.*=file://net/websockets/OWNERS
I think this should be: per-file websocket* (Or websocket_*)

Just look at other per-file regexps: 
https://cs.chromium.org/search/?q=per-file.*%5C*&sq=package:chromium&type=cs

[tyoshino (SeeGerritForStatus), 2016-09-14 11:38:30]

Description was changed from

==========
Make WebSocket people per-file OWNERS of net/socket for files starting with
"websocket"

BUG=none
==========

to

==========
Make WebSocket people per-file OWNERS of net/socket for files starting with
"websocket"

R=mmenke@chromium.org
BUG=none
==========

[tyoshino (SeeGerritForStatus), 2016-09-14 11:38:38]

https://codereview.chromium.org/2335133005/diff/1/net/socket/OWNERS
File net/socket/OWNERS (right):

https://codereview.chromium.org/2335133005/diff/1/net/socket/OWNERS#newcode2
net/socket/OWNERS:2: per-file websocket.*=file://net/websockets/OWNERS
On 2016/09/14 08:47:06, mmenke wrote:
> I think this should be: per-file websocket* (Or websocket_*)
> 
> Just look at other per-file regexps: 
> https://cs.chromium.org/search/?q=per-file.*%5C*&sq=package:chromium&type=cs

Oh, right. I copied an existing one, but looks it's just wrong and there's no
longer any file starting with websocket...

Fixed.

[tyoshino (SeeGerritForStatus), 2016-09-14 11:40:20]

The CQ bit was checked by tyoshino@chromium.org

[tyoshino (SeeGerritForStatus), 2016-09-14 11:40:20]

The patchset sent to the CQ was uploaded after l-g-t-m from mmenke@chromium.org
Link to the patchset: https://codereview.chromium.org/2335133005/#ps20001
(title: "Addressed #4")

[commit-bot: I haz the power, 2016-09-14 11:40:28]

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, 2016-09-14 12:32:05]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-14 12:34:00]

Description was changed from

==========
Make WebSocket people per-file OWNERS of net/socket for files starting with
"websocket"

R=mmenke@chromium.org
BUG=none
==========

to

==========
Make WebSocket people per-file OWNERS of net/socket for files starting with
"websocket"

R=mmenke@chromium.org
BUG=none

Committed: https://crrev.com/3a00cb3814f26705029307ff96db48ef036a0183
Cr-Commit-Position: refs/heads/master@{#418538}
==========

[commit-bot: I haz the power, 2016-09-14 12:34:01]

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