[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

We are adding COMPONENT/TEAM information into OWNERS file
(android, etc).

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

Review-Url: https://codereview.chromium.org/2745113003
Cr-Commit-Position: refs/heads/master@{#456430}
Committed: https://chromium.googlesource.com/chromium/src/+/46237dd82d3a34ae0702a8395992f76ed6a3ad95

[ymzhang1, 2017-03-13 17:29:02]

Patchset #1 (id:1) has been deleted

[ymzhang1, 2017-03-13 17:29:36]

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!

[boliu, 2017-03-13 17:47:26]

https://codereview.chromium.org/2745113003/diff/20001/components/web_contents...
File components/web_contents_delegate_android/OWNERS (right):

https://codereview.chromium.org/2745113003/diff/20001/components/web_contents...
components/web_contents_delegate_android/OWNERS:4: # COMPONENT: Mobile>WebView
should be Internals>Core, following content, and add OS-Android

https://codereview.chromium.org/2745113003/diff/20001/device/power_save_block...
File device/power_save_blocker/OWNERS (right):

https://codereview.chromium.org/2745113003/diff/20001/device/power_save_block...
device/power_save_blocker/OWNERS:5: # COMPONENT: OS>Kernel>Power
this is not the kernel, and doesn't have its own component

generic Internals>Core again I guess

https://codereview.chromium.org/2745113003/diff/20001/ui/android/java/src/org...
File ui/android/java/src/org/chromium/ui/display/OWNERS (right):

https://codereview.chromium.org/2745113003/diff/20001/ui/android/java/src/org...
ui/android/java/src/org/chromium/ui/display/OWNERS:3: # COMPONENT:
OS>Kernel>Display
definitely not kernel

this should match ui/display/OWNERS, and since that doesn't have a component
yet, I don't really know what component it should be

Core>Internals is always a generic fallback I guess

[ymzhang1, 2017-03-13 17:57:54]

Patchset #2 (id:40001) has been deleted

[ymzhang1, 2017-03-13 17:58:36]

Thanks for the comments!

https://codereview.chromium.org/2745113003/diff/20001/components/web_contents...
File components/web_contents_delegate_android/OWNERS (right):

https://codereview.chromium.org/2745113003/diff/20001/components/web_contents...
components/web_contents_delegate_android/OWNERS:4: # COMPONENT: Mobile>WebView
On 2017/03/13 17:47:25, boliu wrote:
> should be Internals>Core, following content, and add OS-Android

Done.

https://codereview.chromium.org/2745113003/diff/20001/device/power_save_block...
File device/power_save_blocker/OWNERS (right):

https://codereview.chromium.org/2745113003/diff/20001/device/power_save_block...
device/power_save_blocker/OWNERS:5: # COMPONENT: OS>Kernel>Power
On 2017/03/13 17:47:26, boliu wrote:
> this is not the kernel, and doesn't have its own component
> 
> generic Internals>Core again I guess

Done.

https://codereview.chromium.org/2745113003/diff/20001/ui/android/java/src/org...
File ui/android/java/src/org/chromium/ui/display/OWNERS (right):

https://codereview.chromium.org/2745113003/diff/20001/ui/android/java/src/org...
ui/android/java/src/org/chromium/ui/display/OWNERS:3: # COMPONENT:
OS>Kernel>Display
On 2017/03/13 17:47:26, boliu wrote:
> definitely not kernel
> 
> this should match ui/display/OWNERS, and since that doesn't have a component
> yet, I don't really know what component it should be
> 
> Core>Internals is always a generic fallback I guess

Done.

[boliu, 2017-03-13 18:01:28]

lgtm

[ymzhang1, 2017-03-13 18:03:03]

The CQ bit was checked by ymzhang@chromium.org

[commit-bot: I haz the power, 2017-03-13 18:03:43]

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-13 18:15:39]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1489428183783450,
"parent_rev": "424607e3a61e012909b8693fd720514bfebfe641", "commit_rev":
"46237dd82d3a34ae0702a8395992f76ed6a3ad95"}

[commit-bot: I haz the power, 2017-03-13 18:16:23]

Description was changed from

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

We are adding COMPONENT/TEAM information into OWNERS file
(android, etc).

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

to

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

We are adding COMPONENT/TEAM information into OWNERS file
(android, etc).

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

Review-Url: https://codereview.chromium.org/2745113003
Cr-Commit-Position: refs/heads/master@{#456430}
Committed:
https://chromium.googlesource.com/chromium/src/+/46237dd82d3a34ae0702a8395992...
==========

[commit-bot: I haz the power, 2017-03-13 18:16:24]

Committed patchset #2 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/46237dd82d3a34ae0702a8395992...