Add TEAM/COMPONENT

[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

We are adding COMPONENT/TEAM information into OWNERS file. Please help us to 
verify the added TEAM/COMPONENT or suggest the correct TEAM/COMPONENT in your 
OWNERS files. Thanks. 

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
Review-Url: https://codereview.chromium.org/2686253004
Cr-Commit-Position: refs/heads/master@{#450457}
Committed: https://chromium.googlesource.com/chromium/src/+/8f2127bb83d8c0f9cc421b7c485ea0848d500d3b

[ymzhang1, 2017-02-10 22:13:52]

Description was changed from

==========
Add TEAM/COMPONENT

BUG=679905
==========

to

==========
Add TEAM/COMPONENT

BUG=679905
==========

[ymzhang1, 2017-02-10 22:13:52]

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

[ymzhang1, 2017-02-10 22:14:12]

Patchset #1 (id:1) has been deleted

[ymzhang1, 2017-02-10 22:14:59]

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!

[Martin Barbella, 2017-02-10 22:28:55]

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

[Martin Barbella, 2017-02-10 22:28:59]

Drive-by comments since I saw this on a watchlist and was curious. Tom will
still know better than me for most of these.

https://codereview.chromium.org/2686253004/diff/20001/base/numerics/OWNERS
File base/numerics/OWNERS (right):

https://codereview.chromium.org/2686253004/diff/20001/base/numerics/OWNERS#ne...
base/numerics/OWNERS:5: # COMPONENT: Blink>Forms>Number
Security is the component generally used for this code.

https://codereview.chromium.org/2686253004/diff/20001/ipc/OWNERS
File ipc/OWNERS (right):

https://codereview.chromium.org/2686253004/diff/20001/ipc/OWNERS#newcode16
ipc/OWNERS:16: # COMPONENT: Manifest
Guessing this should be Internals>Core or just Internals, tsepez will know
better here.

https://codereview.chromium.org/2686253004/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/weborigin/OWNERS (right):

https://codereview.chromium.org/2686253004/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/weborigin/OWNERS:5: # COMPONENT:
Infra>Platform
Anything under Infra seems incorrect, but I'm not sure what the correct
component is. Will leave this to tsepez.

https://codereview.chromium.org/2686253004/diff/20001/tools/ipc_fuzzer/OWNERS
File tools/ipc_fuzzer/OWNERS (right):

https://codereview.chromium.org/2686253004/diff/20001/tools/ipc_fuzzer/OWNERS...
tools/ipc_fuzzer/OWNERS:5: # COMPONENT: Tools
Seems good. Maybe we should make a subcomponent for it, but there's a very low
bug volume here.

https://codereview.chromium.org/2686253004/diff/20001/tools/security/OWNERS
File tools/security/OWNERS (right):

https://codereview.chromium.org/2686253004/diff/20001/tools/security/OWNERS#n...
tools/security/OWNERS:4: # COMPONENT: Tools
I suspect Security will be preferred here, but will let tsepez comment.

[ymzhang1, 2017-02-10 22:53:53]

On 2017/02/10 22:28:59, Martin Barbella wrote:
> Drive-by comments since I saw this on a watchlist and was curious. Tom will
> still know better than me for most of these.
> 
> https://codereview.chromium.org/2686253004/diff/20001/base/numerics/OWNERS
> File base/numerics/OWNERS (right):
> 
>
https://codereview.chromium.org/2686253004/diff/20001/base/numerics/OWNERS#ne...
> base/numerics/OWNERS:5: # COMPONENT: Blink>Forms>Number
> Security is the component generally used for this code.
> 
> https://codereview.chromium.org/2686253004/diff/20001/ipc/OWNERS
> File ipc/OWNERS (right):
> 
> https://codereview.chromium.org/2686253004/diff/20001/ipc/OWNERS#newcode16
> ipc/OWNERS:16: # COMPONENT: Manifest
> Guessing this should be Internals>Core or just Internals, tsepez will know
> better here.
> 
>
https://codereview.chromium.org/2686253004/diff/20001/third_party/WebKit/Sour...
> File third_party/WebKit/Source/platform/weborigin/OWNERS (right):
> 
>
https://codereview.chromium.org/2686253004/diff/20001/third_party/WebKit/Sour...
> third_party/WebKit/Source/platform/weborigin/OWNERS:5: # COMPONENT:
> Infra>Platform
> Anything under Infra seems incorrect, but I'm not sure what the correct
> component is. Will leave this to tsepez.
> 
> https://codereview.chromium.org/2686253004/diff/20001/tools/ipc_fuzzer/OWNERS
> File tools/ipc_fuzzer/OWNERS (right):
> 
>
https://codereview.chromium.org/2686253004/diff/20001/tools/ipc_fuzzer/OWNERS...
> tools/ipc_fuzzer/OWNERS:5: # COMPONENT: Tools
> Seems good. Maybe we should make a subcomponent for it, but there's a very low
> bug volume here.
> 
> https://codereview.chromium.org/2686253004/diff/20001/tools/security/OWNERS
> File tools/security/OWNERS (right):
> 
>
https://codereview.chromium.org/2686253004/diff/20001/tools/security/OWNERS#n...
> tools/security/OWNERS:4: # COMPONENT: Tools
> I suspect Security will be preferred here, but will let tsepez comment.

Thanks for driving-by comments! They are helpful. I'll update after receiving
more comments.

[Tom Sepez, 2017-02-10 23:09:19]

tsepez@chromium.org changed reviewers:
+ jschuh@chromium.org

[Tom Sepez, 2017-02-10 23:17:21]

I added jschuh for input as well.

Is there a better way to get a list of compontents other than trying to match
string fragments against suggestions in the bug tracker query fields ???

https://codereview.chromium.org/2686253004/diff/20001/base/numerics/OWNERS
File base/numerics/OWNERS (right):

https://codereview.chromium.org/2686253004/diff/20001/base/numerics/OWNERS#ne...
base/numerics/OWNERS:5: # COMPONENT: Blink>Forms>Number
On 2017/02/10 22:28:58, Martin Barbella wrote:
> Security is the component generally used for this code.

Internals.

https://codereview.chromium.org/2686253004/diff/20001/ipc/OWNERS
File ipc/OWNERS (right):

https://codereview.chromium.org/2686253004/diff/20001/ipc/OWNERS#newcode16
ipc/OWNERS:16: # COMPONENT: Manifest
On 2017/02/10 22:28:58, Martin Barbella wrote:
> Guessing this should be Internals>Core or just Internals, tsepez will know
> better here.

Internals>Core.

https://codereview.chromium.org/2686253004/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/weborigin/OWNERS (right):

https://codereview.chromium.org/2686253004/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/weborigin/OWNERS:5: # COMPONENT:
Infra>Platform
On 2017/02/10 22:28:59, Martin Barbella wrote:
> Anything under Infra seems incorrect, but I'm not sure what the correct
> component is. Will leave this to tsepez.

This would neeed to be blink>something, maybe blink>internals

https://codereview.chromium.org/2686253004/diff/20001/tools/security/OWNERS
File tools/security/OWNERS (right):

https://codereview.chromium.org/2686253004/diff/20001/tools/security/OWNERS#n...
tools/security/OWNERS:4: # COMPONENT: Tools
On 2017/02/10 22:28:59, Martin Barbella wrote:
> I suspect Security will be preferred here, but will let tsepez comment.

security

[ymzhang1, 2017-02-10 23:45:58]

On 2017/02/10 23:17:21, Tom Sepez wrote:
> I added jschuh for input as well.
> 
> Is there a better way to get a list of compontents other than trying to match
> string fragments against suggestions in the bug tracker query fields ???
> 
> https://codereview.chromium.org/2686253004/diff/20001/base/numerics/OWNERS
> File base/numerics/OWNERS (right):
> 
>
https://codereview.chromium.org/2686253004/diff/20001/base/numerics/OWNERS#ne...
> base/numerics/OWNERS:5: # COMPONENT: Blink>Forms>Number
> On 2017/02/10 22:28:58, Martin Barbella wrote:
> > Security is the component generally used for this code.
> 
> Internals.
> 
> https://codereview.chromium.org/2686253004/diff/20001/ipc/OWNERS
> File ipc/OWNERS (right):
> 
> https://codereview.chromium.org/2686253004/diff/20001/ipc/OWNERS#newcode16
> ipc/OWNERS:16: # COMPONENT: Manifest
> On 2017/02/10 22:28:58, Martin Barbella wrote:
> > Guessing this should be Internals>Core or just Internals, tsepez will know
> > better here.
> 
> Internals>Core.
> 
>
https://codereview.chromium.org/2686253004/diff/20001/third_party/WebKit/Sour...
> File third_party/WebKit/Source/platform/weborigin/OWNERS (right):
> 
>
https://codereview.chromium.org/2686253004/diff/20001/third_party/WebKit/Sour...
> third_party/WebKit/Source/platform/weborigin/OWNERS:5: # COMPONENT:
> Infra>Platform
> On 2017/02/10 22:28:59, Martin Barbella wrote:
> > Anything under Infra seems incorrect, but I'm not sure what the correct
> > component is. Will leave this to tsepez.
> 
> This would neeed to be blink>something, maybe blink>internals
> 
> https://codereview.chromium.org/2686253004/diff/20001/tools/security/OWNERS
> File tools/security/OWNERS (right):
> 
>
https://codereview.chromium.org/2686253004/diff/20001/tools/security/OWNERS#n...
> tools/security/OWNERS:4: # COMPONENT: Tools
> On 2017/02/10 22:28:59, Martin Barbella wrote:
> > I suspect Security will be preferred here, but will let tsepez comment.
> 
> security

Regarding getting component list, maybe this is helpful?
https://bugs.chromium.org/p/chromium/adminComponents

[ymzhang1, 2017-02-13 21:35:03]

Description was changed from

==========
Add TEAM/COMPONENT

BUG=679905
==========

to

==========
BUG=679905
We are adding COMPONENT/TEAM information into OWNERS file. Please help us to
verify the added TEAM/COMPONENT or suggest the correct
TEAM/COMPONENT in your OWNERS files. Thanks. 

Proposal to add TEAM/COMPONENT information into OWNERS files
https://docs.google.com/document/d/1jty6UsFMW9-SYgpQC-ztEc3lltziOQBBArMkROCST...

Proposal about how to get suggested component for directory. 
https://docs.google.com/document/d/1G0UD01aNXdzPsJO-gTLEQiKkDV8XU2XBbi_Gm4g9J...

TEAM-COMPONENT mapping
https://docs.google.com/spreadsheets/d/19JEFMvsxD3eThyGiJRqAjcpx362LHUDdVzICA...
==========

[ymzhang1, 2017-02-13 23:14:31]

Description was changed from

==========
BUG=679905
We are adding COMPONENT/TEAM information into OWNERS file. Please help us to
verify the added TEAM/COMPONENT or suggest the correct
TEAM/COMPONENT in your OWNERS files. Thanks. 

Proposal to add TEAM/COMPONENT information into OWNERS files
https://docs.google.com/document/d/1jty6UsFMW9-SYgpQC-ztEc3lltziOQBBArMkROCST...

Proposal about how to get suggested component for directory. 
https://docs.google.com/document/d/1G0UD01aNXdzPsJO-gTLEQiKkDV8XU2XBbi_Gm4g9J...

TEAM-COMPONENT mapping
https://docs.google.com/spreadsheets/d/19JEFMvsxD3eThyGiJRqAjcpx362LHUDdVzICA...
==========

to

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

We are adding COMPONENT/TEAM information into OWNERS file. Please help us to 
verify the added TEAM/COMPONENT or suggest the correct TEAM/COMPONENT in your 
OWNERS files. Thanks. 

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
==========

[ymzhang1, 2017-02-14 19:14:29]

https://codereview.chromium.org/2686253004/diff/20001/base/numerics/OWNERS
File base/numerics/OWNERS (right):

https://codereview.chromium.org/2686253004/diff/20001/base/numerics/OWNERS#ne...
base/numerics/OWNERS:5: # COMPONENT: Blink>Forms>Number
On 2017/02/10 23:17:21, Tom Sepez wrote:
> On 2017/02/10 22:28:58, Martin Barbella wrote:
> > Security is the component generally used for this code.
> 
> Internals.

Done.

https://codereview.chromium.org/2686253004/diff/20001/ipc/OWNERS
File ipc/OWNERS (right):

https://codereview.chromium.org/2686253004/diff/20001/ipc/OWNERS#newcode16
ipc/OWNERS:16: # COMPONENT: Manifest
On 2017/02/10 23:17:21, Tom Sepez wrote:
> On 2017/02/10 22:28:58, Martin Barbella wrote:
> > Guessing this should be Internals>Core or just Internals, tsepez will know
> > better here.
> 
> Internals>Core.

Done.

https://codereview.chromium.org/2686253004/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/weborigin/OWNERS (right):

https://codereview.chromium.org/2686253004/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/weborigin/OWNERS:5: # COMPONENT:
Infra>Platform
On 2017/02/10 23:17:21, Tom Sepez wrote:
> On 2017/02/10 22:28:59, Martin Barbella wrote:
> > Anything under Infra seems incorrect, but I'm not sure what the correct
> > component is. Will leave this to tsepez.
> 
> This would neeed to be blink>something, maybe blink>internals

Done.

https://codereview.chromium.org/2686253004/diff/20001/tools/ipc_fuzzer/OWNERS
File tools/ipc_fuzzer/OWNERS (right):

https://codereview.chromium.org/2686253004/diff/20001/tools/ipc_fuzzer/OWNERS...
tools/ipc_fuzzer/OWNERS:5: # COMPONENT: Tools
On 2017/02/10 22:28:59, Martin Barbella wrote:
> Seems good. Maybe we should make a subcomponent for it, but there's a very low
> bug volume here.

Sure. We could update if a new component is created for this in the future.

https://codereview.chromium.org/2686253004/diff/20001/tools/security/OWNERS
File tools/security/OWNERS (right):

https://codereview.chromium.org/2686253004/diff/20001/tools/security/OWNERS#n...
tools/security/OWNERS:4: # COMPONENT: Tools
On 2017/02/10 23:17:21, Tom Sepez wrote:
> On 2017/02/10 22:28:59, Martin Barbella wrote:
> > I suspect Security will be preferred here, but will let tsepez comment.
> 
> security

Done.

[Tom Sepez, 2017-02-14 19:20:40]

lgtm

[ymzhang1, 2017-02-14 19:21:01]

The CQ bit was checked by ymzhang@chromium.org

[commit-bot: I haz the power, 2017-02-14 19:21:58]

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-02-14 20:48:12]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1487100061748940,
"parent_rev": "d42d0e2c837f374221b8c655cb58b6e9a702d8b8", "commit_rev":
"8f2127bb83d8c0f9cc421b7c485ea0848d500d3b"}

[commit-bot: I haz the power, 2017-02-14 20:49:14]

Description was changed from

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

We are adding COMPONENT/TEAM information into OWNERS file. Please help us to 
verify the added TEAM/COMPONENT or suggest the correct TEAM/COMPONENT in your 
OWNERS files. Thanks. 

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
==========

to

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

We are adding COMPONENT/TEAM information into OWNERS file. Please help us to 
verify the added TEAM/COMPONENT or suggest the correct TEAM/COMPONENT in your 
OWNERS files. Thanks. 

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

Review-Url: https://codereview.chromium.org/2686253004
Cr-Commit-Position: refs/heads/master@{#450457}
Committed:
https://chromium.googlesource.com/chromium/src/+/8f2127bb83d8c0f9cc421b7c485e...
==========

[commit-bot: I haz the power, 2017-02-14 20:49:15]

Committed patchset #2 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/8f2127bb83d8c0f9cc421b7c485e...