[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

[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/2708283002
Cr-Commit-Position: refs/heads/master@{#453095}
Committed: https://chromium.googlesource.com/chromium/src/+/74c09cc3128a36e8a705731f16716b820ecdd754

[ymzhang1, 2017-02-22 00:48:10]

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

[ymzhang1, 2017-02-22 00:48:11]

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!

[David Trainor- moved to gerrit, 2017-02-23 03:35:33]

On 2017/02/22 00:48:11, 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!

Sorry for the delayed review.  I'm planning on setting aside some time tomorrow
to go through this.  Sadly a lot of the specified components won't work, and the
ones that we usually set might be somewhat generic without OS info.  I'll do my
best to get you the right components though!

[ymzhang1, 2017-02-23 17:37:21]

On 2017/02/23 03:35:33, David Trainor wrote:
> On 2017/02/22 00:48:11, 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!
> 
> Sorry for the delayed review.  I'm planning on setting aside some time
tomorrow
> to go through this.  Sadly a lot of the specified components won't work, and
the
> ones that we usually set might be somewhat generic without OS info.  I'll do
my
> best to get you the right components though!

Sure. No problem. If some directories don't have proper component, we could also
remove component information for now.

In case you would like to create some new component, here is the link.  
https://bugs.chromium.org/p/chromium/issues/entry?template=Component%20Request

Thank you very much!

[David Trainor- moved to gerrit, 2017-02-24 18:16:05]

Sorry for the delays.  Updated the components.  There's a bug (695941) tracking
the creation of the new component we probably need.

Thanks!

https://codereview.chromium.org/2708283002/diff/1/chrome/android/OWNERS
File chrome/android/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/chrome/android/OWNERS#newco...
chrome/android/OWNERS:13: # COMPONENT: Mobile>WebView
This is basically the same as //chrome but with OS=Android.  Do we have a
generic component for all of //chrome?

https://codereview.chromium.org/2708283002/diff/1/chrome/android/java/res/OWNERS
File chrome/android/java/res/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/chrome/android/java/res/OWN...
chrome/android/java/res/OWNERS:14: # COMPONENT: Mobile>WebView
UI>Browser>Mobile

https://codereview.chromium.org/2708283002/diff/1/chrome/android/java/src/org...
File chrome/android/java/src/org/chromium/chrome/browser/compositor/OWNERS
(right):

https://codereview.chromium.org/2708283002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/compositor/OWNERS:5: #
COMPONENT: Internals>Compositing
UI>Browser>Mobile>CompositedUI (filed 695941 to add).

https://codereview.chromium.org/2708283002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/compositor/overlays/strip/OWNERS
(right):

https://codereview.chromium.org/2708283002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/compositor/overlays/strip/OWNERS:5:
# COMPONENT: Internals>Compositing
UI>Browser>Mobile>CompositedUI (filed 695941 to add).

https://codereview.chromium.org/2708283002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/compositor/scene_layer/OWNERS
(right):

https://codereview.chromium.org/2708283002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/compositor/scene_layer/OWNERS:7:
# COMPONENT: Internals>Compositing>Images
UI>Browser>Mobile>CompositedUI (filed 695941 to add).

https://codereview.chromium.org/2708283002/diff/1/chrome/android/javatests/OW...
File chrome/android/javatests/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/chrome/android/javatests/OW...
chrome/android/javatests/OWNERS:9: # COMPONENT: Mobile>WebView
Same as //chrome/android.  Is there a top level component for //chrome/test?

https://codereview.chromium.org/2708283002/diff/1/chrome/android/javatests/sr...
File chrome/android/javatests/src/org/chromium/chrome/browser/compositor/OWNERS
(right):

https://codereview.chromium.org/2708283002/diff/1/chrome/android/javatests/sr...
chrome/android/javatests/src/org/chromium/chrome/browser/compositor/OWNERS:4: #
COMPONENT: Internals>Compositing
UI>Browser>Mobile>CompositedUI (filed 695941 to add).

https://codereview.chromium.org/2708283002/diff/1/chrome/android/junit/src/or...
File chrome/android/junit/src/org/chromium/chrome/browser/compositor/OWNERS
(right):

https://codereview.chromium.org/2708283002/diff/1/chrome/android/junit/src/or...
chrome/android/junit/src/org/chromium/chrome/browser/compositor/OWNERS:4: #
COMPONENT: Internals>Compositing>Software
UI>Browser>Mobile>CompositedUI (filed 695941 to add).

https://codereview.chromium.org/2708283002/diff/1/chrome/app/android/OWNERS
File chrome/app/android/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/chrome/app/android/OWNERS#n...
chrome/app/android/OWNERS:6: # COMPONENT: Mobile>WebView
Internals>PlatformIntegration

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

https://codereview.chromium.org/2708283002/diff/1/chrome/browser/android/OWNE...
chrome/browser/android/OWNERS:16: # COMPONENT: Mobile>WebView
Same as //chrome/browser.  Do we have one for that yet?  UI>Browser>Mobile?

https://codereview.chromium.org/2708283002/diff/1/chrome/browser/android/comp...
File chrome/browser/android/compositor/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/chrome/browser/android/comp...
chrome/browser/android/compositor/OWNERS:9: # COMPONENT: Mobile>WebView
UI>Browser>Mobile

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

https://codereview.chromium.org/2708283002/diff/1/chrome/browser/download/OWN...
chrome/browser/download/OWNERS:8: # COMPONENT: Internals>Network
UI>Browser>Downloads

https://codereview.chromium.org/2708283002/diff/1/chrome/browser/ui/android/t...
File chrome/browser/ui/android/tab_contents/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/chrome/browser/ui/android/t...
chrome/browser/ui/android/tab_contents/OWNERS:4: # COMPONENT: Mobile>WebView
UI>Browser>Mobile

https://codereview.chromium.org/2708283002/diff/1/chrome/browser/ui/android/t...
File chrome/browser/ui/android/tab_model/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/chrome/browser/ui/android/t...
chrome/browser/ui/android/tab_model/OWNERS:4: # COMPONENT: Mobile>WebView
UI>Browser>Mobile

https://codereview.chromium.org/2708283002/diff/1/content/browser/download/OW...
File content/browser/download/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/content/browser/download/OW...
content/browser/download/OWNERS:5: # COMPONENT: Internals>Network
UI>Browser>Downloads

https://codereview.chromium.org/2708283002/diff/1/content/public/android/java...
File content/public/android/java/res/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/content/public/android/java...
content/public/android/java/res/OWNERS:7: # COMPONENT: Infra>Client>Android
Content>Core

https://codereview.chromium.org/2708283002/diff/1/third_party/android_opengl/...
File third_party/android_opengl/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/third_party/android_opengl/...
third_party/android_opengl/OWNERS:3: # COMPONENT: Infra>Client>Android
Mobile

https://codereview.chromium.org/2708283002/diff/1/ui/android/OWNERS
File ui/android/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/ui/android/OWNERS#newcode12
ui/android/OWNERS:12: # COMPONENT: Infra>Client>Android
UI

[ymzhang1, 2017-02-24 18:53:48]

Patchset #2 (id:20001) has been deleted

[ymzhang1, 2017-02-24 18:56:26]

Thank you very much for the helpful comments and creating the new component!

https://codereview.chromium.org/2708283002/diff/1/chrome/android/OWNERS
File chrome/android/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/chrome/android/OWNERS#newco...
chrome/android/OWNERS:13: # COMPONENT: Mobile>WebView
On 2017/02/24 18:16:04, David Trainor wrote:
> This is basically the same as //chrome but with OS=Android.  Do we have a
> generic component for all of //chrome?

We don't have one for //chrome yet. In this case, I would like to leave it
empty. By default it would have same component as its parent directory.

https://codereview.chromium.org/2708283002/diff/1/chrome/android/java/res/OWNERS
File chrome/android/java/res/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/chrome/android/java/res/OWN...
chrome/android/java/res/OWNERS:14: # COMPONENT: Mobile>WebView
On 2017/02/24 18:16:04, David Trainor wrote:
> UI>Browser>Mobile

Done.

https://codereview.chromium.org/2708283002/diff/1/chrome/android/java/src/org...
File chrome/android/java/src/org/chromium/chrome/browser/compositor/OWNERS
(right):

https://codereview.chromium.org/2708283002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/compositor/OWNERS:5: #
COMPONENT: Internals>Compositing
On 2017/02/24 18:16:04, David Trainor wrote:
> UI>Browser>Mobile>CompositedUI (filed 695941 to add).

Grt. Thanks!

https://codereview.chromium.org/2708283002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/compositor/overlays/strip/OWNERS
(right):

https://codereview.chromium.org/2708283002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/compositor/overlays/strip/OWNERS:5:
# COMPONENT: Internals>Compositing
On 2017/02/24 18:16:04, David Trainor wrote:
> UI>Browser>Mobile>CompositedUI (filed 695941 to add).

Done.

https://codereview.chromium.org/2708283002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/compositor/scene_layer/OWNERS
(right):

https://codereview.chromium.org/2708283002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/compositor/scene_layer/OWNERS:7:
# COMPONENT: Internals>Compositing>Images
On 2017/02/24 18:16:04, David Trainor wrote:
> UI>Browser>Mobile>CompositedUI (filed 695941 to add).

Done.

https://codereview.chromium.org/2708283002/diff/1/chrome/android/javatests/OW...
File chrome/android/javatests/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/chrome/android/javatests/OW...
chrome/android/javatests/OWNERS:9: # COMPONENT: Mobile>WebView
On 2017/02/24 18:16:04, David Trainor wrote:
> Same as //chrome/android.  Is there a top level component for //chrome/test?

Some have proposed to use Test>Android. Please let me know if it's not a proper
one. Thanks!

https://codereview.chromium.org/2708283002/diff/1/chrome/android/javatests/sr...
File chrome/android/javatests/src/org/chromium/chrome/browser/compositor/OWNERS
(right):

https://codereview.chromium.org/2708283002/diff/1/chrome/android/javatests/sr...
chrome/android/javatests/src/org/chromium/chrome/browser/compositor/OWNERS:4: #
COMPONENT: Internals>Compositing
On 2017/02/24 18:16:04, David Trainor wrote:
> UI>Browser>Mobile>CompositedUI (filed 695941 to add).

Done.

https://codereview.chromium.org/2708283002/diff/1/chrome/android/junit/src/or...
File chrome/android/junit/src/org/chromium/chrome/browser/compositor/OWNERS
(right):

https://codereview.chromium.org/2708283002/diff/1/chrome/android/junit/src/or...
chrome/android/junit/src/org/chromium/chrome/browser/compositor/OWNERS:4: #
COMPONENT: Internals>Compositing>Software
On 2017/02/24 18:16:04, David Trainor wrote:
> UI>Browser>Mobile>CompositedUI (filed 695941 to add).

Done.

https://codereview.chromium.org/2708283002/diff/1/chrome/app/android/OWNERS
File chrome/app/android/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/chrome/app/android/OWNERS#n...
chrome/app/android/OWNERS:6: # COMPONENT: Mobile>WebView
On 2017/02/24 18:16:04, David Trainor wrote:
> Internals>PlatformIntegration

Done.

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

https://codereview.chromium.org/2708283002/diff/1/chrome/browser/android/OWNE...
chrome/browser/android/OWNERS:16: # COMPONENT: Mobile>WebView
On 2017/02/24 18:16:04, David Trainor wrote:
> Same as //chrome/browser.  Do we have one for that yet?  UI>Browser>Mobile?

We don't have one for //chrome/browser. I add UI>Browser>Mobile for now. Feel
free to let me know if not adding is better. Thanks!

https://codereview.chromium.org/2708283002/diff/1/chrome/browser/android/comp...
File chrome/browser/android/compositor/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/chrome/browser/android/comp...
chrome/browser/android/compositor/OWNERS:9: # COMPONENT: Mobile>WebView
On 2017/02/24 18:16:04, David Trainor wrote:
> UI>Browser>Mobile

Done.

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

https://codereview.chromium.org/2708283002/diff/1/chrome/browser/download/OWN...
chrome/browser/download/OWNERS:8: # COMPONENT: Internals>Network
On 2017/02/24 18:16:04, David Trainor wrote:
> UI>Browser>Downloads

Done.

https://codereview.chromium.org/2708283002/diff/1/chrome/browser/ui/android/t...
File chrome/browser/ui/android/tab_contents/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/chrome/browser/ui/android/t...
chrome/browser/ui/android/tab_contents/OWNERS:4: # COMPONENT: Mobile>WebView
On 2017/02/24 18:16:04, David Trainor wrote:
> UI>Browser>Mobile

Done.

https://codereview.chromium.org/2708283002/diff/1/chrome/browser/ui/android/t...
File chrome/browser/ui/android/tab_model/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/chrome/browser/ui/android/t...
chrome/browser/ui/android/tab_model/OWNERS:4: # COMPONENT: Mobile>WebView
On 2017/02/24 18:16:04, David Trainor wrote:
> UI>Browser>Mobile

Done.

https://codereview.chromium.org/2708283002/diff/1/content/browser/download/OW...
File content/browser/download/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/content/browser/download/OW...
content/browser/download/OWNERS:5: # COMPONENT: Internals>Network
On 2017/02/24 18:16:04, David Trainor wrote:
> UI>Browser>Downloads

Done.

https://codereview.chromium.org/2708283002/diff/1/content/public/android/java...
File content/public/android/java/res/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/content/public/android/java...
content/public/android/java/res/OWNERS:7: # COMPONENT: Infra>Client>Android
On 2017/02/24 18:16:05, David Trainor wrote:
> Content>Core

Done.

https://codereview.chromium.org/2708283002/diff/1/third_party/android_opengl/...
File third_party/android_opengl/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/third_party/android_opengl/...
third_party/android_opengl/OWNERS:3: # COMPONENT: Infra>Client>Android
On 2017/02/24 18:16:05, David Trainor wrote:
> Mobile

Done.

https://codereview.chromium.org/2708283002/diff/1/ui/android/OWNERS
File ui/android/OWNERS (right):

https://codereview.chromium.org/2708283002/diff/1/ui/android/OWNERS#newcode12
ui/android/OWNERS:12: # COMPONENT: Infra>Client>Android
On 2017/02/24 18:16:05, David Trainor wrote:
> UI

Done.

[David Trainor- moved to gerrit, 2017-02-25 05:13:55]

lgtm

[ymzhang1, 2017-02-25 15:33:33]

The CQ bit was checked by ymzhang@chromium.org

[commit-bot: I haz the power, 2017-02-25 15:33:40]

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-25 16:14:25]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1488036813078160,
"parent_rev": "a621fc7d334253502b1ad9d3233b6bbc637980b5", "commit_rev":
"74c09cc3128a36e8a705731f16716b820ecdd754"}

[commit-bot: I haz the power, 2017-02-25 16:14:59]

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

[commit-bot: I haz the power, 2017-02-25 16:15:00]

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