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/2668423007
Cr-Commit-Position: refs/heads/master@{#451387}
Committed: https://chromium.googlesource.com/chromium/src/+/d15ea4791d38c7fa37969739862fd16adcdc341c

[ymzhang1, 2017-02-03 00:10:25]

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 file? Thank you very much!

[timvolodine, 2017-02-08 16:44:48]

Hi, a number of components seem to be off, I have provided some comments, but
some need to be checked still. Is there a strategy to adding this?

https://codereview.chromium.org/2668423007/diff/1/android_webview/tools/syste...
File android_webview/tools/system_webview_shell/OWNERS (right):

https://codereview.chromium.org/2668423007/diff/1/android_webview/tools/syste...
android_webview/tools/system_webview_shell/OWNERS:3: # COMPONENT:
Infra>Client>Android
I think Mobile>WebView is more appropriate here (also in other files)

https://codereview.chromium.org/2668423007/diff/1/components/safe_browsing/OW...
File components/safe_browsing/OWNERS (right):

https://codereview.chromium.org/2668423007/diff/1/components/safe_browsing/OW...
components/safe_browsing/OWNERS:9: # COMPONENT: Security
probably Services>Safebrowsing

https://codereview.chromium.org/2668423007/diff/1/content/browser/battery_sta...
File content/browser/battery_status/OWNERS (right):

https://codereview.chromium.org/2668423007/diff/1/content/browser/battery_sta...
content/browser/battery_status/OWNERS:4: # COMPONENT: UI>Browser
not sure but this probably too general

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

https://codereview.chromium.org/2668423007/diff/1/device/battery/OWNERS#newcode6
device/battery/OWNERS:6: # COMPONENT: IO>DeviceOrientation
this is about battery not orientation

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

https://codereview.chromium.org/2668423007/diff/1/device/generic_sensor/OWNER...
device/generic_sensor/OWNERS:5: # COMPONENT: IO>DeviceOrientation
not sure will have to look it up this one probably too specific though

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

https://codereview.chromium.org/2668423007/diff/1/device/vibration/OWNERS#new...
device/vibration/OWNERS:7: # COMPONENT: IO>DeviceOrientation
maybe Blink>Vibration

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

https://codereview.chromium.org/2668423007/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/battery/OWNERS:3: # COMPONENT: Blink>Media
don't think it's media

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

https://codereview.chromium.org/2668423007/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/sensor/OWNERS:6: # COMPONENT:
Blink>Media>Audio
not media/audio

[ymzhang1, 2017-02-10 18:58:53]

Thanks for the comments! I used a prediction model to get component candidate
for OWNERS files missing COMPONENT information. 

Proposal for component prediction for directory:
https://docs.google.com/document/d/1G0UD01aNXdzPsJO-gTLEQiKkDV8XU2XBbi_Gm4g9J...

https://codereview.chromium.org/2668423007/diff/1/android_webview/tools/syste...
File android_webview/tools/system_webview_shell/OWNERS (right):

https://codereview.chromium.org/2668423007/diff/1/android_webview/tools/syste...
android_webview/tools/system_webview_shell/OWNERS:3: # COMPONENT:
Infra>Client>Android
On 2017/02/08 16:44:48, timvolodine wrote:
> I think Mobile>WebView is more appropriate here (also in other files)

Done.

https://codereview.chromium.org/2668423007/diff/1/components/safe_browsing/OW...
File components/safe_browsing/OWNERS (right):

https://codereview.chromium.org/2668423007/diff/1/components/safe_browsing/OW...
components/safe_browsing/OWNERS:9: # COMPONENT: Security
On 2017/02/08 16:44:48, timvolodine wrote:
> probably Services>Safebrowsing

Done.

https://codereview.chromium.org/2668423007/diff/1/content/browser/battery_sta...
File content/browser/battery_status/OWNERS (right):

https://codereview.chromium.org/2668423007/diff/1/content/browser/battery_sta...
content/browser/battery_status/OWNERS:4: # COMPONENT: UI>Browser
On 2017/02/08 16:44:48, timvolodine wrote:
> not sure but this probably too general

Shall I remove component from this OWNERS file?

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

https://codereview.chromium.org/2668423007/diff/1/device/battery/OWNERS#newcode6
device/battery/OWNERS:6: # COMPONENT: IO>DeviceOrientation
On 2017/02/08 16:44:48, timvolodine wrote:
> this is about battery not orientation

It seems there's no component related to battery. Maybe I remove #COMPONENT
information from this file?

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

https://codereview.chromium.org/2668423007/diff/1/device/generic_sensor/OWNER...
device/generic_sensor/OWNERS:5: # COMPONENT: IO>DeviceOrientation
On 2017/02/08 16:44:48, timvolodine wrote:
> not sure will have to look it up this one probably too specific though

Currently not find any good suggestion from my side. Shall I remove this line?

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

https://codereview.chromium.org/2668423007/diff/1/device/vibration/OWNERS#new...
device/vibration/OWNERS:7: # COMPONENT: IO>DeviceOrientation
On 2017/02/08 16:44:48, timvolodine wrote:
> maybe Blink>Vibration

Done.

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

https://codereview.chromium.org/2668423007/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/battery/OWNERS:3: # COMPONENT: Blink>Media
On 2017/02/08 16:44:48, timvolodine wrote:
> don't think it's media

I'll remove this for now.

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

https://codereview.chromium.org/2668423007/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/sensor/OWNERS:6: # COMPONENT:
Blink>Media>Audio
On 2017/02/08 16:44:48, timvolodine wrote:
> not media/audio

I'll remove for now

[ymzhang1, 2017-02-13 21:54:29]

Description was changed from

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

BUG=679905
R=timvolodine
==========

to

==========
BUG=679905
R=timvolodine
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:58]

Description was changed from

==========
BUG=679905
R=timvolodine
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
==========

[timvolodine, 2017-02-16 17:14:57]

https://codereview.chromium.org/2668423007/diff/1/content/browser/battery_sta...
File content/browser/battery_status/OWNERS (right):

https://codereview.chromium.org/2668423007/diff/1/content/browser/battery_sta...
content/browser/battery_status/OWNERS:4: # COMPONENT: UI>Browser
On 2017/02/10 18:58:53, ymzhang1 wrote:
> On 2017/02/08 16:44:48, timvolodine wrote:
> > not sure but this probably too general
> 
> Shall I remove component from this OWNERS file?

I think you could remove the unknown / unsure components from this patch in
order to not block other components on this.

[ymzhang1, 2017-02-16 22:21:13]

On 2017/02/16 17:14:57, timvolodine wrote:
>
https://codereview.chromium.org/2668423007/diff/1/content/browser/battery_sta...
> File content/browser/battery_status/OWNERS (right):
> 
>
https://codereview.chromium.org/2668423007/diff/1/content/browser/battery_sta...
> content/browser/battery_status/OWNERS:4: # COMPONENT: UI>Browser
> On 2017/02/10 18:58:53, ymzhang1 wrote:
> > On 2017/02/08 16:44:48, timvolodine wrote:
> > > not sure but this probably too general
> > 
> > Shall I remove component from this OWNERS file?
> 
> I think you could remove the unknown / unsure components from this patch in
> order to not block other components on this.

Sure. I'll do this. Thank you!

[ymzhang1, 2017-02-16 23:15:19]

Patchset #2 (id:20001) has been deleted

[ymzhang1, 2017-02-16 23:17:52]

https://codereview.chromium.org/2668423007/diff/1/content/browser/battery_sta...
File content/browser/battery_status/OWNERS (right):

https://codereview.chromium.org/2668423007/diff/1/content/browser/battery_sta...
content/browser/battery_status/OWNERS:4: # COMPONENT: UI>Browser
On 2017/02/16 17:14:57, timvolodine wrote:
> On 2017/02/10 18:58:53, ymzhang1 wrote:
> > On 2017/02/08 16:44:48, timvolodine wrote:
> > > not sure but this probably too general
> > 
> > Shall I remove component from this OWNERS file?
> 
> I think you could remove the unknown / unsure components from this patch in
> order to not block other components on this.

Done.

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

https://codereview.chromium.org/2668423007/diff/1/device/battery/OWNERS#newcode6
device/battery/OWNERS:6: # COMPONENT: IO>DeviceOrientation
On 2017/02/10 18:58:53, ymzhang1 wrote:
> On 2017/02/08 16:44:48, timvolodine wrote:
> > this is about battery not orientation
> 
> It seems there's no component related to battery. Maybe I remove #COMPONENT
> information from this file?

Done.

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

https://codereview.chromium.org/2668423007/diff/1/device/generic_sensor/OWNER...
device/generic_sensor/OWNERS:5: # COMPONENT: IO>DeviceOrientation
On 2017/02/10 18:58:53, ymzhang1 wrote:
> On 2017/02/08 16:44:48, timvolodine wrote:
> > not sure will have to look it up this one probably too specific though
> 
> Currently not find any good suggestion from my side. Shall I remove this line?

Done.

[timvolodine, 2017-02-17 14:13:20]

few more fixes, otherwise looks fine

https://codereview.chromium.org/2668423007/diff/40001/android_webview/tools/s...
File android_webview/tools/system_webview_shell/apk/OWNERS (right):

https://codereview.chromium.org/2668423007/diff/40001/android_webview/tools/s...
android_webview/tools/system_webview_shell/apk/OWNERS:3: # COMPONENT:
Infra>Client>Android
Mobile>WebView

https://codereview.chromium.org/2668423007/diff/40001/android_webview/tools/s...
File android_webview/tools/system_webview_shell/layout_tests/OWNERS (right):

https://codereview.chromium.org/2668423007/diff/40001/android_webview/tools/s...
android_webview/tools/system_webview_shell/layout_tests/OWNERS:3: # COMPONENT:
Infra>Client>Android
Mobile>WebView

[ymzhang1, 2017-02-17 17:53:53]

Updated. Thanks!

https://codereview.chromium.org/2668423007/diff/40001/android_webview/tools/s...
File android_webview/tools/system_webview_shell/apk/OWNERS (right):

https://codereview.chromium.org/2668423007/diff/40001/android_webview/tools/s...
android_webview/tools/system_webview_shell/apk/OWNERS:3: # COMPONENT:
Infra>Client>Android
On 2017/02/17 14:13:20, timvolodine wrote:
> Mobile>WebView

Done.

https://codereview.chromium.org/2668423007/diff/40001/android_webview/tools/s...
File android_webview/tools/system_webview_shell/layout_tests/OWNERS (right):

https://codereview.chromium.org/2668423007/diff/40001/android_webview/tools/s...
android_webview/tools/system_webview_shell/layout_tests/OWNERS:3: # COMPONENT:
Infra>Client>Android
On 2017/02/17 14:13:20, timvolodine wrote:
> Mobile>WebView

Done.

[timvolodine, 2017-02-17 19:51:21]

On 2017/02/17 17:53:53, ymzhang1 wrote:
> Updated. Thanks!
> 
>
https://codereview.chromium.org/2668423007/diff/40001/android_webview/tools/s...
> File android_webview/tools/system_webview_shell/apk/OWNERS (right):
> 
>
https://codereview.chromium.org/2668423007/diff/40001/android_webview/tools/s...
> android_webview/tools/system_webview_shell/apk/OWNERS:3: # COMPONENT:
> Infra>Client>Android
> On 2017/02/17 14:13:20, timvolodine wrote:
> > Mobile>WebView
> 
> Done.
> 
>
https://codereview.chromium.org/2668423007/diff/40001/android_webview/tools/s...
> File android_webview/tools/system_webview_shell/layout_tests/OWNERS (right):
> 
>
https://codereview.chromium.org/2668423007/diff/40001/android_webview/tools/s...
> android_webview/tools/system_webview_shell/layout_tests/OWNERS:3: # COMPONENT:
> Infra>Client>Android
> On 2017/02/17 14:13:20, timvolodine wrote:
> > Mobile>WebView
> 
> Done.

lgtm, thanks!

[ymzhang1, 2017-02-17 20:30:26]

The CQ bit was checked by ymzhang@chromium.org

[commit-bot: I haz the power, 2017-02-17 20:32:47]

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-17 22:27:51]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1487363426997310,
"parent_rev": "5047017bdca8641764919bcbb25b60b8d89c5684", "commit_rev":
"d15ea4791d38c7fa37969739862fd16adcdc341c"}

[commit-bot: I haz the power, 2017-02-17 22:28:40]

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/2668423007
Cr-Commit-Position: refs/heads/master@{#451387}
Committed:
https://chromium.googlesource.com/chromium/src/+/d15ea4791d38c7fa37969739862f...
==========

[commit-bot: I haz the power, 2017-02-17 22:28:41]

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