[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

We are adding COMPONENT/TEAM information into OWNERS file(
signin, etc).

Proposal to add TEAM/COMPONENT information into OWNERS files
http://bit.ly/add-team-component-proposal
Proposal about how to get suggested component for directory.
http://bit.ly/directory-mapping-proposal
TEAM-COMPONENT mapping
http://bit.ly/component-team-mapping

Additional Information:
Component lists
https://bugs.chromium.org/p/chromium/adminComponents

BUG=679905
R=rogerta@chromium.org

Review-Url: https://codereview.chromium.org/2740953005
Cr-Commit-Position: refs/heads/master@{#456191}
Committed: https://chromium.googlesource.com/chromium/src/+/094b5e38f83ceebb408911139f194dd3705e54be

[ymzhang1, 2017-03-10 22:00:54]

Hello,

We are adding COMPONENT/TEAM information into OWNERS file. Would you mind
helping us to verify the added TEAM/COMPONENT or suggest the correct
TEAM/COMPONENT in your OWNERS files? Thank you very much!

[Roger Tawa OOO till Jul 10th, 2017-03-10 22:06:33]

https://codereview.chromium.org/2740953005/diff/1/chrome/browser/extensions/s...
File chrome/browser/extensions/signin/OWNERS (right):

https://codereview.chromium.org/2740953005/diff/1/chrome/browser/extensions/s...
chrome/browser/extensions/signin/OWNERS:2: guohui@chromium.org
Since you are here, could you remove guohui@ ? She no longer works at google or
on chrome.

https://codereview.chromium.org/2740953005/diff/1/google_apis/OWNERS
File google_apis/OWNERS (right):

https://codereview.chromium.org/2740953005/diff/1/google_apis/OWNERS#newcode5
google_apis/OWNERS:5: # COMPONENT: Services>SignIn
I don't believe there should be a component at this level.  The components
should be in the subdirs.  //google_apis is just a home for all of chrome's
integrations with google web apis.

https://codereview.chromium.org/2740953005/diff/1/rlz/OWNERS
File rlz/OWNERS (right):

https://codereview.chromium.org/2740953005/diff/1/rlz/OWNERS#newcode4
rlz/OWNERS:4: # COMPONENT: Services>SignIn
This should be the same as c/b/rlz, which is Internals>Core

[ymzhang1, 2017-03-10 22:13:44]

https://codereview.chromium.org/2740953005/diff/1/chrome/browser/extensions/s...
File chrome/browser/extensions/signin/OWNERS (right):

https://codereview.chromium.org/2740953005/diff/1/chrome/browser/extensions/s...
chrome/browser/extensions/signin/OWNERS:2: guohui@chromium.org
On 2017/03/10 22:06:33, Roger Tawa wrote:
> Since you are here, could you remove guohui@ ? She no longer works at google
or
> on chrome.

sure.

https://codereview.chromium.org/2740953005/diff/1/google_apis/OWNERS
File google_apis/OWNERS (right):

https://codereview.chromium.org/2740953005/diff/1/google_apis/OWNERS#newcode5
google_apis/OWNERS:5: # COMPONENT: Services>SignIn
On 2017/03/10 22:06:33, Roger Tawa wrote:
> I don't believe there should be a component at this level.  The components
> should be in the subdirs.  //google_apis is just a home for all of chrome's
> integrations with google web apis.

Thanks for the comments!

https://codereview.chromium.org/2740953005/diff/1/rlz/OWNERS
File rlz/OWNERS (right):

https://codereview.chromium.org/2740953005/diff/1/rlz/OWNERS#newcode4
rlz/OWNERS:4: # COMPONENT: Services>SignIn
On 2017/03/10 22:06:33, Roger Tawa wrote:
> This should be the same as c/b/rlz, which is Internals>Core

Done.

[Roger Tawa OOO till Jul 10th, 2017-03-10 22:20:16]

Thanks!

lgtm

[ymzhang1, 2017-03-10 22:20:59]

The CQ bit was checked by ymzhang@chromium.org

[commit-bot: I haz the power, 2017-03-10 22:21:31]

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-03-10 22:34:28]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489184459920950,
"parent_rev": "c0f6017abb9aeb5ae1c8e137b6a3671305298b40", "commit_rev":
"094b5e38f83ceebb408911139f194dd3705e54be"}

[commit-bot: I haz the power, 2017-03-10 22:35:02]

Description was changed from

==========
[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

We are adding COMPONENT/TEAM information into OWNERS file(
signin, etc).

Proposal to add TEAM/COMPONENT information into OWNERS files
http://bit.ly/add-team-component-proposal
Proposal about how to get suggested component for directory.
http://bit.ly/directory-mapping-proposal
TEAM-COMPONENT mapping
http://bit.ly/component-team-mapping

Additional Information:
Component lists
https://bugs.chromium.org/p/chromium/adminComponents

BUG=679905
R=rogerta@chromium.org
==========

to

==========
[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

We are adding COMPONENT/TEAM information into OWNERS file(
signin, etc).

Proposal to add TEAM/COMPONENT information into OWNERS files
http://bit.ly/add-team-component-proposal
Proposal about how to get suggested component for directory.
http://bit.ly/directory-mapping-proposal
TEAM-COMPONENT mapping
http://bit.ly/component-team-mapping

Additional Information:
Component lists
https://bugs.chromium.org/p/chromium/adminComponents

BUG=679905
R=rogerta@chromium.org

Review-Url: https://codereview.chromium.org/2740953005
Cr-Commit-Position: refs/heads/master@{#456191}
Committed:
https://chromium.googlesource.com/chromium/src/+/094b5e38f83ceebb408911139f19...
==========

[commit-bot: I haz the power, 2017-03-10 22:35:04]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/094b5e38f83ceebb408911139f19...