add TEAM/COMPONENT information into OWNERS

add TEAM/COMPONENT information into OWNERS

Please help us to verify the TEAM/COMPONENT information. If the current information is incorrect, please suggest the correct team or component or None(if not applicable). 

BUG=679905
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2624303002
Cr-Commit-Position: refs/heads/master@{#444210}
Committed: https://chromium.googlesource.com/chromium/src/+/aa99b0c20fc1a58fa7231c0f0a9d69dd4dbc3596

[ymzhang1, 2017-01-11 22:40:19]

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

[ymzhang1, 2017-01-11 22:40:20]

Hello, 

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

[ymzhang1, 2017-01-11 22:42:34]

Description was changed from

==========
update Blink team info


add TEAM/COMPONENT information into OWNERS

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

to

==========
add TEAM/COMPONENT information into OWNERS

Please help us to verify the TEAM/COMPONENT information. If the current
information is incorrect, please suggest the correct team or component or
None(if not applicable). 

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

[Reilly Grant (use Gerrit), 2017-01-11 22:59:20]

https://codereview.chromium.org/2624303002/diff/1/chrome/browser/OWNERS
File chrome/browser/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/chrome/browser/OWNERS#newco...
chrome/browser/OWNERS:92: # COMPONENT: Internals>Network
This directory is so high level I don't think you could assign it a component.

https://codereview.chromium.org/2624303002/diff/1/chrome/browser/chooser_cont...
File chrome/browser/chooser_controller/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/chrome/browser/chooser_cont...
chrome/browser/chooser_controller/OWNERS:5: # COMPONENT: Internals>Network
Internals>Permissions

https://codereview.chromium.org/2624303002/diff/1/chrome/browser/media_galler...
File chrome/browser/media_galleries/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/chrome/browser/media_galler...
chrome/browser/media_galleries/OWNERS:5: # COMPONENT: Internals>Network
Platform>Extensions>API

https://codereview.chromium.org/2624303002/diff/1/chrome/browser/usb/OWNERS
File chrome/browser/usb/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/chrome/browser/usb/OWNERS#n...
chrome/browser/usb/OWNERS:5: # COMPONENT: Internals>Network
Blink>USB

https://codereview.chromium.org/2624303002/diff/1/components/device_event_log...
File components/device_event_log/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/components/device_event_log...
components/device_event_log/OWNERS:4: # TEAM: reillyg@chromium.org
I am not a mailing list.

https://codereview.chromium.org/2624303002/diff/1/components/device_event_log...
components/device_event_log/OWNERS:5: # COMPONENT: Blink>USB
Internals>Logging

https://codereview.chromium.org/2624303002/diff/1/device/OWNERS
File device/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/device/OWNERS#newcode7
device/OWNERS:7: # TEAM: reillyg@chromium.org
I am not a team. I see other files here just do without the TEAM line.

https://codereview.chromium.org/2624303002/diff/1/device/OWNERS#newcode8
device/OWNERS:8: # COMPONENT: Blink>USB
//device is more than just USB. There is no single component for this top-level
directory.

https://codereview.chromium.org/2624303002/diff/1/device/usb/OWNERS
File device/usb/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/device/usb/OWNERS#newcode5
device/usb/OWNERS:5: # TEAM: reillyg@chromium.org
Same here.

https://codereview.chromium.org/2624303002/diff/1/device/usb/OWNERS#newcode6
device/usb/OWNERS:6: # COMPONENT: Blink>USB
IO>USB

https://codereview.chromium.org/2624303002/diff/1/extensions/browser/api/hid/...
File extensions/browser/api/hid/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/extensions/browser/api/hid/...
extensions/browser/api/hid/OWNERS:4: # COMPONENT: Platform>Extensions
Platform>Extensions>API

https://codereview.chromium.org/2624303002/diff/1/extensions/browser/api/seri...
File extensions/browser/api/serial/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/extensions/browser/api/seri...
extensions/browser/api/serial/OWNERS:4: # COMPONENT: Platform>Extensions
Platform>Extensions>API

https://codereview.chromium.org/2624303002/diff/1/extensions/browser/api/sock...
File extensions/browser/api/socket/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/extensions/browser/api/sock...
extensions/browser/api/socket/OWNERS:5: # COMPONENT: Platform>Extensions
Platform>Extensions>API

https://codereview.chromium.org/2624303002/diff/1/extensions/browser/api/usb/...
File extensions/browser/api/usb/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/extensions/browser/api/usb/...
extensions/browser/api/usb/OWNERS:6: # COMPONENT: Platform>Extensions
Platform>Extensions>API

https://codereview.chromium.org/2624303002/diff/1/services/device/OWNERS
File services/device/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/services/device/OWNERS#newc...
services/device/OWNERS:5: # TEAM: reillyg@chromium.org
I am not a team.

https://codereview.chromium.org/2624303002/diff/1/services/device/OWNERS#newc...
services/device/OWNERS:6: # COMPONENT: Blink>USB
IO>USB

https://codereview.chromium.org/2624303002/diff/1/third_party/WebKit/Source/m...
File third_party/WebKit/Source/modules/webusb/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/webusb/OWNERS:4: # TEAM: reillyg@chromium.org
I am not a mailing list.

https://codereview.chromium.org/2624303002/diff/1/third_party/libudev/OWNERS
File third_party/libudev/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/third_party/libudev/OWNERS#...
third_party/libudev/OWNERS:4: # TEAM: crblink-rotation@chromium.org
This doesn't seem appropriate.

https://codereview.chromium.org/2624303002/diff/1/third_party/libudev/OWNERS#...
third_party/libudev/OWNERS:5: # COMPONENT: Blink
IO>USB

https://codereview.chromium.org/2624303002/diff/1/third_party/usb_ids/OWNERS
File third_party/usb_ids/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/third_party/usb_ids/OWNERS#...
third_party/usb_ids/OWNERS:3: # TEAM: reillyg@chromium.org
I am not a mailing list.

https://codereview.chromium.org/2624303002/diff/1/third_party/usb_ids/OWNERS#...
third_party/usb_ids/OWNERS:4: # COMPONENT: Blink>USB
IO>USB

https://codereview.chromium.org/2624303002/diff/1/tools/usb_gadget/OWNERS
File tools/usb_gadget/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/tools/usb_gadget/OWNERS#new...
tools/usb_gadget/OWNERS:3: # TEAM: reillyg@chromium.org
I am not a mailing list.

https://codereview.chromium.org/2624303002/diff/1/tools/usb_gadget/OWNERS#new...
tools/usb_gadget/OWNERS:4: # COMPONENT: Blink>USB
IO>USB

[stgao, 2017-01-12 02:28:15]

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

[stgao, 2017-01-12 02:28:15]

BTW, once a new patch is uploaded and needs reviewing again, you will have to
publish it again.

https://codereview.chromium.org/2624303002/diff/20001/extensions/OWNERS
File extensions/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/20001/extensions/OWNERS#newco...
extensions/OWNERS:11: #   chrome/utility/extensions/OWNERS
It seems the above directories are related to extensions. Could we also add the
component there?

[Reilly Grant (use Gerrit), 2017-01-12 03:05:08]

https://codereview.chromium.org/2624303002/diff/20001/services/device/OWNERS
File services/device/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/20001/services/device/OWNERS#...
services/device/OWNERS:5: # COMPONENT: IO>USB
My mistake, this is another top-level thing that doesn't have a good component
unless there's a generic "Mojo service infrastructure" component.

[ymzhang1, 2017-01-12 18:12:44]

https://codereview.chromium.org/2624303002/diff/1/chrome/browser/OWNERS
File chrome/browser/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/chrome/browser/OWNERS#newco...
chrome/browser/OWNERS:92: # COMPONENT: Internals>Network
On 2017/01/11 22:59:18, Reilly Grant wrote:
> This directory is so high level I don't think you could assign it a component.

Done.

https://codereview.chromium.org/2624303002/diff/1/chrome/browser/chooser_cont...
File chrome/browser/chooser_controller/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/chrome/browser/chooser_cont...
chrome/browser/chooser_controller/OWNERS:5: # COMPONENT: Internals>Network
On 2017/01/11 22:59:18, Reilly Grant wrote:
> Internals>Permissions

Done.

https://codereview.chromium.org/2624303002/diff/1/chrome/browser/media_galler...
File chrome/browser/media_galleries/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/chrome/browser/media_galler...
chrome/browser/media_galleries/OWNERS:5: # COMPONENT: Internals>Network
On 2017/01/11 22:59:19, Reilly Grant wrote:
> Platform>Extensions>API

Done.

https://codereview.chromium.org/2624303002/diff/1/chrome/browser/usb/OWNERS
File chrome/browser/usb/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/chrome/browser/usb/OWNERS#n...
chrome/browser/usb/OWNERS:5: # COMPONENT: Internals>Network
On 2017/01/11 22:59:19, Reilly Grant wrote:
> Blink>USB

Done.

https://codereview.chromium.org/2624303002/diff/1/components/device_event_log...
File components/device_event_log/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/components/device_event_log...
components/device_event_log/OWNERS:4: # TEAM: reillyg@chromium.org
On 2017/01/11 22:59:19, Reilly Grant wrote:
> I am not a mailing list.

Done.

https://codereview.chromium.org/2624303002/diff/1/components/device_event_log...
components/device_event_log/OWNERS:5: # COMPONENT: Blink>USB
On 2017/01/11 22:59:19, Reilly Grant wrote:
> Internals>Logging

Done.

https://codereview.chromium.org/2624303002/diff/1/device/OWNERS
File device/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/device/OWNERS#newcode7
device/OWNERS:7: # TEAM: reillyg@chromium.org
On 2017/01/11 22:59:19, Reilly Grant wrote:
> I am not a team. I see other files here just do without the TEAM line.

Done.

https://codereview.chromium.org/2624303002/diff/1/device/OWNERS#newcode8
device/OWNERS:8: # COMPONENT: Blink>USB
On 2017/01/11 22:59:19, Reilly Grant wrote:
> //device is more than just USB. There is no single component for this
top-level
> directory.

Done.

https://codereview.chromium.org/2624303002/diff/1/device/usb/OWNERS
File device/usb/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/device/usb/OWNERS#newcode5
device/usb/OWNERS:5: # TEAM: reillyg@chromium.org
On 2017/01/11 22:59:19, Reilly Grant wrote:
> Same here.

Done.

https://codereview.chromium.org/2624303002/diff/1/device/usb/OWNERS#newcode6
device/usb/OWNERS:6: # COMPONENT: Blink>USB
On 2017/01/11 22:59:19, Reilly Grant wrote:
> IO>USB

Done.

https://codereview.chromium.org/2624303002/diff/1/extensions/browser/api/hid/...
File extensions/browser/api/hid/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/extensions/browser/api/hid/...
extensions/browser/api/hid/OWNERS:4: # COMPONENT: Platform>Extensions
On 2017/01/11 22:59:19, Reilly Grant wrote:
> Platform>Extensions>API

Done.

https://codereview.chromium.org/2624303002/diff/1/extensions/browser/api/seri...
File extensions/browser/api/serial/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/extensions/browser/api/seri...
extensions/browser/api/serial/OWNERS:4: # COMPONENT: Platform>Extensions
On 2017/01/11 22:59:19, Reilly Grant wrote:
> Platform>Extensions>API

Done.

https://codereview.chromium.org/2624303002/diff/1/extensions/browser/api/sock...
File extensions/browser/api/socket/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/extensions/browser/api/sock...
extensions/browser/api/socket/OWNERS:5: # COMPONENT: Platform>Extensions
On 2017/01/11 22:59:19, Reilly Grant wrote:
> Platform>Extensions>API

Done.

https://codereview.chromium.org/2624303002/diff/1/services/device/OWNERS
File services/device/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/services/device/OWNERS#newc...
services/device/OWNERS:5: # TEAM: reillyg@chromium.org
On 2017/01/11 22:59:19, Reilly Grant wrote:
> I am not a team.

Done.

https://codereview.chromium.org/2624303002/diff/1/services/device/OWNERS#newc...
services/device/OWNERS:6: # COMPONENT: Blink>USB
On 2017/01/11 22:59:19, Reilly Grant wrote:
> IO>USB

Done.

https://codereview.chromium.org/2624303002/diff/1/third_party/WebKit/Source/m...
File third_party/WebKit/Source/modules/webusb/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/webusb/OWNERS:4: # TEAM: reillyg@chromium.org
On 2017/01/11 22:59:19, Reilly Grant wrote:
> I am not a mailing list.

Done.

https://codereview.chromium.org/2624303002/diff/1/third_party/libudev/OWNERS
File third_party/libudev/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/third_party/libudev/OWNERS#...
third_party/libudev/OWNERS:4: # TEAM: crblink-rotation@chromium.org
On 2017/01/11 22:59:19, Reilly Grant wrote:
> This doesn't seem appropriate.

Done.

https://codereview.chromium.org/2624303002/diff/1/third_party/libudev/OWNERS#...
third_party/libudev/OWNERS:5: # COMPONENT: Blink
On 2017/01/11 22:59:19, Reilly Grant wrote:
> IO>USB

Done.

https://codereview.chromium.org/2624303002/diff/1/third_party/usb_ids/OWNERS
File third_party/usb_ids/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/1/third_party/usb_ids/OWNERS#...
third_party/usb_ids/OWNERS:3: # TEAM: reillyg@chromium.org
On 2017/01/11 22:59:19, Reilly Grant wrote:
> I am not a mailing list.

Done.

https://codereview.chromium.org/2624303002/diff/1/third_party/usb_ids/OWNERS#...
third_party/usb_ids/OWNERS:4: # COMPONENT: Blink>USB
On 2017/01/11 22:59:19, Reilly Grant wrote:
> IO>USB

Done.

https://codereview.chromium.org/2624303002/diff/20001/extensions/OWNERS
File extensions/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/20001/extensions/OWNERS#newco...
extensions/OWNERS:11: #   chrome/utility/extensions/OWNERS
On 2017/01/12 02:28:15, stgao (slow on Monday) wrote:
> It seems the above directories are related to extensions. Could we also add
the
> component there?

Oh, my mistake. Thanks for pointing out! I will add these files in the new
patch.

https://codereview.chromium.org/2624303002/diff/20001/services/device/OWNERS
File services/device/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/20001/services/device/OWNERS#...
services/device/OWNERS:5: # COMPONENT: IO>USB
On 2017/01/12 03:05:07, Reilly Grant (OOO Jan 12-16th) wrote:
> My mistake, this is another top-level thing that doesn't have a good component
> unless there's a generic "Mojo service infrastructure" component.

Thanks for the comments! We have following Mojo related components:
Infra>Client>Mojo,Internals>Mojo, Internals>Media>Mojo, Internals>Mojo>Bindings.

If none of above is a good component for this directory, I will remove this
line.

[ymzhang1, 2017-01-12 18:47:51]

Description was changed from

==========
add TEAM/COMPONENT information into OWNERS

Please help us to verify the TEAM/COMPONENT information. If the current
information is incorrect, please suggest the correct team or component or
None(if not applicable). 

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

to

==========
add TEAM/COMPONENT information into OWNERS

Please help us to verify the TEAM/COMPONENT information. If the current
information is incorrect, please suggest the correct team or component or
None(if not applicable). 

BUG=679905
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[ymzhang1, 2017-01-12 18:50:51]

Add 8 OWNERS file. 

TODO: update services/device/OWNERS.

[sshruthi1, 2017-01-13 23:19:08]

On 2017/01/12 18:50:51, ymzhang1 wrote:
> Add 8 OWNERS file. 
> 
> TODO: update services/device/OWNERS.

Just to reiterate Reilly's comments, let's not add anything to TEAM that is not
a mailing list (no sites page, no individual emails), its fine to add the line
but leave it blank if there is no clear team owner "yet".

[Reilly Grant (use Gerrit), 2017-01-17 19:35:29]

https://codereview.chromium.org/2624303002/diff/40001/chrome/browser/renderer...
File chrome/browser/renderer_host/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/40001/chrome/browser/renderer...
chrome/browser/renderer_host/OWNERS:21: # COMPONENT: UI>Accessibility
This directory is rather high level but perhaps the Content>Core component is
appropriate.

https://codereview.chromium.org/2624303002/diff/40001/chrome/browser/resource...
File chrome/browser/resources/webstore_app/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/40001/chrome/browser/resource...
chrome/browser/resources/webstore_app/OWNERS:3: # COMPONENT: Internals
Webstore

[ymzhang1, 2017-01-17 23:26:55]

https://codereview.chromium.org/2624303002/diff/40001/chrome/browser/renderer...
File chrome/browser/renderer_host/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/40001/chrome/browser/renderer...
chrome/browser/renderer_host/OWNERS:21: # COMPONENT: UI>Accessibility
On 2017/01/17 19:35:29, Reilly Grant wrote:
> This directory is rather high level but perhaps the Content>Core component is
> appropriate.

Done.

https://codereview.chromium.org/2624303002/diff/40001/chrome/browser/resource...
File chrome/browser/resources/webstore_app/OWNERS (right):

https://codereview.chromium.org/2624303002/diff/40001/chrome/browser/resource...
chrome/browser/resources/webstore_app/OWNERS:3: # COMPONENT: Internals
On 2017/01/17 19:35:29, Reilly Grant wrote:
> Webstore

Done.

[Reilly Grant (use Gerrit), 2017-01-17 23:30:10]

lgtm

[ymzhang1, 2017-01-17 23:30:57]

The CQ bit was checked by ymzhang@chromium.org

[commit-bot: I haz the power, 2017-01-17 23:32:18]

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-01-18 00:46:13]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1484695857964220,
"parent_rev": "d224a9dddebc82b819515e4d9c35737c7a92e9d0", "commit_rev":
"aa99b0c20fc1a58fa7231c0f0a9d69dd4dbc3596"}

[commit-bot: I haz the power, 2017-01-18 00:46:49]

Description was changed from

==========
add TEAM/COMPONENT information into OWNERS

Please help us to verify the TEAM/COMPONENT information. If the current
information is incorrect, please suggest the correct team or component or
None(if not applicable). 

BUG=679905
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
add TEAM/COMPONENT information into OWNERS

Please help us to verify the TEAM/COMPONENT information. If the current
information is incorrect, please suggest the correct team or component or
None(if not applicable). 

BUG=679905
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2624303002
Cr-Commit-Position: refs/heads/master@{#444210}
Committed:
https://chromium.googlesource.com/chromium/src/+/aa99b0c20fc1a58fa7231c0f0a9d...
==========

[commit-bot: I haz the power, 2017-01-18 00:46:50]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/aa99b0c20fc1a58fa7231c0f0a9d...

[Wez, 2017-02-07 22:30:12]

wez@chromium.org changed reviewers:
+ thakis@chromium.org

[Wez, 2017-02-07 22:30:13]

Hallo thakis@chromium.org!
Due to a depot_tools patch which mistakenly removed the OWNERS check for
non-source files (see crbug.com/684270), the following files landed in this CL
and need a retrospective rubber-stamp review from you:
	chrome/browser/renderer_host/OWNERS
Thanks,
Wez