Update translate related OWNERS

Update translate related OWNERS

Add new owner, and change to have only one real OWNERS and
others just refers it by file://components/translate/OWNERS.

Committed: https://crrev.com/0d7d7de910c9c54b25458d439b81a029398a0e6e
Cr-Commit-Position: refs/heads/master@{#392574}

[Takashi Toyoshima, 2016-05-09 10:38:30]

Description was changed from

==========
Update translate related OWNERS
==========

to

==========
Update translate related OWNERS
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[Takashi Toyoshima, 2016-05-09 10:41:18]

Description was changed from

==========
Update translate related OWNERS
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Update translate related OWNERS
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[Takashi Toyoshima, 2016-05-09 10:41:18]

toyoshim@chromium.org changed reviewers:
+ andrewhayden@chromium.org, droger@chromium.org, hajimehoshi@chromium.org,
mad@chromium.org

[Takashi Toyoshima, 2016-05-09 10:56:09]

Hi translate owners.
I would like to add groby@ to translate related owners files who works to revamp
translate bubble UI actively.

Also, I noticed that some files still contain mad@ but if I remember correctly,
mad@ asked to remove himself from translate owner files. So I'd remove these
lines. mad@: please correct me if my memory bits were flipped.

One idea to revise this patch set:
There are so many owner files for translate, and these are hard to maintain. How
about having one real OWNRES file in component/translate, and just write
"file://component/translate/OWNERS" for other OWNERS files?

pros: easy to maintain
cons: we can not have slightly modified owners on minor directories.

Even in the case, we may want to have a separate OWNERS file for cld and cld_2
since these are isolate modules from translate and many of us may not be
familiar with.

Any other suggestion is welcomed.

[droger, 2016-05-09 11:46:38]

lgtm

I'm fine using "file://component/translate/OWNERS" too, I don't have a strong
opinion.

[hajimehoshi, 2016-05-09 11:47:01]

lgtm

On 2016/05/09 10:56:09, Takashi Toyoshima (still slow) wrote:
> Hi translate owners.
> I would like to add groby@ to translate related owners files who works to
revamp
> translate bubble UI actively.
> 
> Also, I noticed that some files still contain mad@ but if I remember
correctly,
> mad@ asked to remove himself from translate owner files. So I'd remove these
> lines. mad@: please correct me if my memory bits were flipped.
> 
> One idea to revise this patch set:
> There are so many owner files for translate, and these are hard to maintain.
How
> about having one real OWNRES file in component/translate, and just write
> "file://component/translate/OWNERS" for other OWNERS files?

+1

> 
> pros: easy to maintain
> cons: we can not have slightly modified owners on minor directories.
> 
> Even in the case, we may want to have a separate OWNERS file for cld and cld_2
> since these are isolate modules from translate and many of us may not be
> familiar with.
> 
> Any other suggestion is welcomed.

[MAD, 2016-05-09 21:10:44]

lgtm

[Takashi Toyoshima, 2016-05-10 06:02:49]

Description was changed from

==========
Update translate related OWNERS
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Update translate related OWNERS

Add new owner, and change to have only one real OWNERS and
others just refers it by file://components/translate/OWNERS.
==========

[Takashi Toyoshima, 2016-05-10 06:04:37]

CL and description were updated to use file://components/translate/OWNERS now.
CC groby just in case.

I'd submit this once Andrew agrees.

[Andrew Hayden (chromium.org), 2016-05-10 08:32:00]

LGTM. Sorry, I've been out ill.

[Takashi Toyoshima, 2016-05-10 08:35:29]

The CQ bit was checked by toyoshim@chromium.org

[Takashi Toyoshima, 2016-05-10 08:35:30]

The patchset sent to the CQ was uploaded after l-g-t-m from droger@chromium.org,
hajimehoshi@chromium.org, mad@chromium.org
Link to the patchset: https://codereview.chromium.org/1963563003/#ps20001
(title: "use file:// redirect")

[commit-bot: I haz the power, 2016-05-10 08:35:46]

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

[commit-bot: I haz the power, 2016-05-10 10:35:05]

Description was changed from

==========
Update translate related OWNERS

Add new owner, and change to have only one real OWNERS and
others just refers it by file://components/translate/OWNERS.
==========

to

==========
Update translate related OWNERS

Add new owner, and change to have only one real OWNERS and
others just refers it by file://components/translate/OWNERS.
==========

[commit-bot: I haz the power, 2016-05-10 10:35:07]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-10 10:36:41]

Description was changed from

==========
Update translate related OWNERS

Add new owner, and change to have only one real OWNERS and
others just refers it by file://components/translate/OWNERS.
==========

to

==========
Update translate related OWNERS

Add new owner, and change to have only one real OWNERS and
others just refers it by file://components/translate/OWNERS.

Committed: https://crrev.com/0d7d7de910c9c54b25458d439b81a029398a0e6e
Cr-Commit-Position: refs/heads/master@{#392574}
==========

[commit-bot: I haz the power, 2016-05-10 10:36:42]

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

[groby-ooo-7-16, 2016-05-10 14:38:57]

groby@chromium.org changed reviewers:
+ groby@chromium.org

[groby-ooo-7-16, 2016-05-10 14:38:58]

Thank you for adding me.