[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

We are adding COMPONENT/TEAM information into OWNERS file.

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=eranm@chromium.org

Review-Url: https://codereview.chromium.org/2740133004
Cr-Commit-Position: refs/heads/master@{#456143}
Committed: https://chromium.googlesource.com/chromium/src/+/04ab58bf5dc99328860c1f3f3b46862bb628b0df

[ymzhang1, 2017-03-10 19:12:11]

Description was changed from

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

We are adding COMPONENT/TEAM information into OWNERS file.

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=eranm@chromium.org
==========

to

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

We are adding COMPONENT/TEAM information into OWNERS file.

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=eranm@chromium.org
==========

[ymzhang1, 2017-03-10 19:12:12]

ymzhang@chromium.org changed reviewers:
+ rsleevi@chromium.org

[ymzhang1, 2017-03-10 19:12:30]

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!

[Ryan Sleevi, 2017-03-10 19:17:33]

Suggestions below, LGTM % fixing :)

https://codereview.chromium.org/2740133004/diff/1/components/certificate_tran...
File components/certificate_transparency/OWNERS (right):

https://codereview.chromium.org/2740133004/diff/1/components/certificate_tran...
components/certificate_transparency/OWNERS:5: # COMPONENT: Internals>Network
Internals>Network>CertTrans

https://codereview.chromium.org/2740133004/diff/1/components/packed_ct_ev_whi...
File components/packed_ct_ev_whitelist/OWNERS (right):

https://codereview.chromium.org/2740133004/diff/1/components/packed_ct_ev_whi...
components/packed_ct_ev_whitelist/OWNERS:5: # COMPONENT: Internals>Network
Internals>Network>CertTrans

[ymzhang1, 2017-03-10 19:22:56]

On 2017/03/10 19:17:33, Ryan Sleevi wrote:
> Suggestions below, LGTM % fixing :)
> 
>
https://codereview.chromium.org/2740133004/diff/1/components/certificate_tran...
> File components/certificate_transparency/OWNERS (right):
> 
>
https://codereview.chromium.org/2740133004/diff/1/components/certificate_tran...
> components/certificate_transparency/OWNERS:5: # COMPONENT: Internals>Network
> Internals>Network>CertTrans
> 
>
https://codereview.chromium.org/2740133004/diff/1/components/packed_ct_ev_whi...
> File components/packed_ct_ev_whitelist/OWNERS (right):
> 
>
https://codereview.chromium.org/2740133004/diff/1/components/packed_ct_ev_whi...
> components/packed_ct_ev_whitelist/OWNERS:5: # COMPONENT: Internals>Network
> Internals>Network>CertTrans

Thanks for the quick reply! One question about team. Shall I take
net-dev@chromium.org as the responsible team to triage component
Internals>Network>CertTrans?

[Ryan Sleevi, 2017-03-10 19:23:44]

On 2017/03/10 19:22:56, ymzhang1 wrote:
> Thanks for the quick reply! One question about team. Shall I take
> mailto:net-dev@chromium.org as the responsible team to triage component
> Internals>Network>CertTrans?

Yes

[ymzhang1, 2017-03-10 19:27:19]

https://codereview.chromium.org/2740133004/diff/1/components/certificate_tran...
File components/certificate_transparency/OWNERS (right):

https://codereview.chromium.org/2740133004/diff/1/components/certificate_tran...
components/certificate_transparency/OWNERS:5: # COMPONENT: Internals>Network
On 2017/03/10 19:17:33, Ryan Sleevi wrote:
> Internals>Network>CertTrans

Done.

https://codereview.chromium.org/2740133004/diff/1/components/packed_ct_ev_whi...
File components/packed_ct_ev_whitelist/OWNERS (right):

https://codereview.chromium.org/2740133004/diff/1/components/packed_ct_ev_whi...
components/packed_ct_ev_whitelist/OWNERS:5: # COMPONENT: Internals>Network
On 2017/03/10 19:17:33, Ryan Sleevi wrote:
> Internals>Network>CertTrans

Done.

[ymzhang1, 2017-03-10 19:27:23]

The CQ bit was checked by ymzhang@chromium.org

[ymzhang1, 2017-03-10 19:27:24]

The patchset sent to the CQ was uploaded after l-g-t-m from rsleevi@chromium.org
Link to the patchset: https://codereview.chromium.org/2740133004/#ps20001
(title: "update component")

[commit-bot: I haz the power, 2017-03-10 19:28:29]

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 20:04:05]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489174043874740,
"parent_rev": "1e87636c63028418b779dadeaadbe4e5f487c76e", "commit_rev":
"04ab58bf5dc99328860c1f3f3b46862bb628b0df"}

[commit-bot: I haz the power, 2017-03-10 20:05:17]

Description was changed from

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

We are adding COMPONENT/TEAM information into OWNERS file.

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=eranm@chromium.org
==========

to

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

We are adding COMPONENT/TEAM information into OWNERS file.

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=eranm@chromium.org

Review-Url: https://codereview.chromium.org/2740133004
Cr-Commit-Position: refs/heads/master@{#456143}
Committed:
https://chromium.googlesource.com/chromium/src/+/04ab58bf5dc99328860c1f3f3b46...
==========

[commit-bot: I haz the power, 2017-03-10 20:05:20]

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