Add TEAM/COMPONENT

Add TEAM/COMPONENT

BUG=679905
Review-Url: https://codereview.chromium.org/2681013002
Cr-Commit-Position: refs/heads/master@{#449680}
Committed: https://chromium.googlesource.com/chromium/src/+/922b98ba5c004ba8b4273c2530aedf9281f6bdff

[ymzhang1, 2017-02-07 23:57:18]

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

[ymzhang1, 2017-02-07 23:57:19]

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!

[lpy, 2017-02-08 00:06:57]

On 2017/02/07 23:57:19, ymzhang1 wrote:
> 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!

For third_party/WebKit/Source/platform/instrumentation/OWNERS, I think it should
be tracing@ rather than paint-dev@. We currently don't have a component under
Blink linked to it so I guess it should be Speed>Tracing, at least for now.

[ymzhang1, 2017-02-08 00:16:34]

On 2017/02/08 00:06:57, lpy wrote:
> On 2017/02/07 23:57:19, ymzhang1 wrote:
> > 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!
> 
> For third_party/WebKit/Source/platform/instrumentation/OWNERS, I think it
should
> be tracing@ rather than paint-dev@. We currently don't have a component under
> Blink linked to it so I guess it should be Speed>Tracing, at least for now.

Updated. Thanks!

[Primiano Tucci (use gerrit), 2017-02-08 16:33:00]

LGTM with comment addressed.

https://codereview.chromium.org/2681013002/diff/20001/base/allocator/OWNERS
File base/allocator/OWNERS (right):

https://codereview.chromium.org/2681013002/diff/20001/base/allocator/OWNERS#n...
base/allocator/OWNERS:7: # COMPONENT: Manifest
What is manifest? if this is about the crbug component, the answer here is
"Internals"

instead within base/allocator/partition_allocator/OWNERS that should be
Blink>MemoryAllocator>Partition

https://codereview.chromium.org/2681013002/diff/20001/base/trace_event/OWNERS
File base/trace_event/OWNERS (right):

https://codereview.chromium.org/2681013002/diff/20001/base/trace_event/OWNERS...
base/trace_event/OWNERS:8: # COMPONENT: Manifest
Speed>Tracing

[ymzhang1, 2017-02-10 00:33:16]

https://codereview.chromium.org/2681013002/diff/20001/base/allocator/OWNERS
File base/allocator/OWNERS (right):

https://codereview.chromium.org/2681013002/diff/20001/base/allocator/OWNERS#n...
base/allocator/OWNERS:7: # COMPONENT: Manifest
On 2017/02/08 16:33:00, Primiano Tucci wrote:
> What is manifest? if this is about the crbug component, the answer here is
> "Internals"
> 
> instead within base/allocator/partition_allocator/OWNERS that should be
> Blink>MemoryAllocator>Partition

Done.

https://codereview.chromium.org/2681013002/diff/20001/base/trace_event/OWNERS
File base/trace_event/OWNERS (right):

https://codereview.chromium.org/2681013002/diff/20001/base/trace_event/OWNERS...
base/trace_event/OWNERS:8: # COMPONENT: Manifest
On 2017/02/08 16:33:00, Primiano Tucci wrote:
> Speed>Tracing

Done.

[ymzhang1, 2017-02-10 18:04:41]

The CQ bit was checked by ymzhang@chromium.org

[ymzhang1, 2017-02-10 18:04:41]

The patchset sent to the CQ was uploaded after l-g-t-m from
primiano@chromium.org
Link to the patchset: https://codereview.chromium.org/2681013002/#ps40001
(title: "update team/compo")

[commit-bot: I haz the power, 2017-02-10 18:05: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-02-10 19:05:39]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1486749881268570,
"parent_rev": "f54eb60f2783c67c77808d7e0aa3e9d39f8c600a", "commit_rev":
"922b98ba5c004ba8b4273c2530aedf9281f6bdff"}

[commit-bot: I haz the power, 2017-02-10 19:06:16]

Description was changed from

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

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

to

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

BUG=679905

Review-Url: https://codereview.chromium.org/2681013002
Cr-Commit-Position: refs/heads/master@{#449680}
Committed:
https://chromium.googlesource.com/chromium/src/+/922b98ba5c004ba8b4273c2530ae...
==========

[commit-bot: I haz the power, 2017-02-10 19:06:17]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/922b98ba5c004ba8b4273c2530ae...