[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

We are adding COMPONENT/TEAM information into OWNERS file
(offline_pages, 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=dewittj@chromium.org

Review-Url: https://codereview.chromium.org/2744723002
Cr-Commit-Position: refs/heads/master@{#455953}
Committed: https://chromium.googlesource.com/chromium/src/+/589c696ecd13c722eb3b7eeef6eb43791978a687

[ymzhang1, 2017-03-09 22:00:26]

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!

[Peter Beverloo, 2017-03-09 22:09:14]

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

[Peter Beverloo, 2017-03-09 22:09:14]

(Random drive-by since I'm on the message_center watch list, sorry. Thanks for
doing this once more :>)

https://codereview.chromium.org/2744723002/diff/20001/chrome/browser/extensio...
File chrome/browser/extensions/api/notifications/OWNERS (right):

https://codereview.chromium.org/2744723002/diff/20001/chrome/browser/extensio...
chrome/browser/extensions/api/notifications/OWNERS:3: # TEAM:
platform-capabilities@chromium.org
extensions-dev@chromium.org

https://codereview.chromium.org/2744723002/diff/20001/ui/message_center/OWNERS
File ui/message_center/OWNERS (right):

https://codereview.chromium.org/2744723002/diff/20001/ui/message_center/OWNER...
ui/message_center/OWNERS:7: # COMPONENT: Blink>Messaging
UI>Notifications

[ymzhang1, 2017-03-09 22:35:07]

On 2017/03/09 22:09:14, Peter Beverloo wrote:
> (Random drive-by since I'm on the message_center watch list, sorry. Thanks for
> doing this once more :>)
> 
>
https://codereview.chromium.org/2744723002/diff/20001/chrome/browser/extensio...
> File chrome/browser/extensions/api/notifications/OWNERS (right):
> 
>
https://codereview.chromium.org/2744723002/diff/20001/chrome/browser/extensio...
> chrome/browser/extensions/api/notifications/OWNERS:3: # TEAM:
> mailto:platform-capabilities@chromium.org
> mailto:extensions-dev@chromium.org
> 
> https://codereview.chromium.org/2744723002/diff/20001/ui/message_center/OWNERS
> File ui/message_center/OWNERS (right):
> 
>
https://codereview.chromium.org/2744723002/diff/20001/ui/message_center/OWNER...
> ui/message_center/OWNERS:7: # COMPONENT: Blink>Messaging
> UI>Notifications


Thanks for the comments! 

Could you help to confirm which is the primary team mailing list for component
UI>Notifications? We would like to have one team mailing list for each component
and the team is responsible to triage bugs of this component.

Current we have 
UI>Notifications: platform-capabilities@
Platform>Extensions: extensions-dev@

Thanks!

[dewittj, 2017-03-09 22:38:14]

https://codereview.chromium.org/2744723002/diff/20001/chrome/browser/extensio...
File chrome/browser/extensions/api/system_indicator/OWNERS (right):

https://codereview.chromium.org/2744723002/diff/20001/chrome/browser/extensio...
chrome/browser/extensions/api/system_indicator/OWNERS:3: # COMPONENT:
UI>Browser>Omnibox>SecurityIndicators
Platform > Apps

https://codereview.chromium.org/2744723002/diff/20001/chrome/browser/ui/views...
File chrome/browser/ui/views/message_center/OWNERS (right):

https://codereview.chromium.org/2744723002/diff/20001/chrome/browser/ui/views...
chrome/browser/ui/views/message_center/OWNERS:4: # TEAM:
platform-capabilities@chromium.org
I think that this is not correct.  But I don't think there is an existing
mailing list for this component.

https://codereview.chromium.org/2744723002/diff/20001/ui/message_center/OWNERS
File ui/message_center/OWNERS (right):

https://codereview.chromium.org/2744723002/diff/20001/ui/message_center/OWNER...
ui/message_center/OWNERS:6: # TEAM: platform-architecture-dev@chromium.org
This team may not be right, I don't know the mission of
platform-architecture-dev.  Don't think there's really a mailing list for this
code at this point.

[ymzhang1, 2017-03-10 01:04:59]

https://codereview.chromium.org/2744723002/diff/20001/chrome/browser/extensio...
File chrome/browser/extensions/api/notifications/OWNERS (right):

https://codereview.chromium.org/2744723002/diff/20001/chrome/browser/extensio...
chrome/browser/extensions/api/notifications/OWNERS:3: # TEAM:
platform-capabilities@chromium.org
On 2017/03/09 22:09:14, Peter Beverloo wrote:
> mailto:extensions-dev@chromium.org

I'll use component Platform>Extensions since this mailing list is responsible to
triage Platform>Extensions. Please correct me if it's not proper here. Thanks!

https://codereview.chromium.org/2744723002/diff/20001/chrome/browser/extensio...
File chrome/browser/extensions/api/system_indicator/OWNERS (right):

https://codereview.chromium.org/2744723002/diff/20001/chrome/browser/extensio...
chrome/browser/extensions/api/system_indicator/OWNERS:3: # COMPONENT:
UI>Browser>Omnibox>SecurityIndicators
On 2017/03/09 22:38:14, dewittj wrote:
> Platform > Apps

Done.

https://codereview.chromium.org/2744723002/diff/20001/chrome/browser/ui/views...
File chrome/browser/ui/views/message_center/OWNERS (right):

https://codereview.chromium.org/2744723002/diff/20001/chrome/browser/ui/views...
chrome/browser/ui/views/message_center/OWNERS:4: # TEAM:
platform-capabilities@chromium.org
On 2017/03/09 22:38:14, dewittj wrote:
> I think that this is not correct.  But I don't think there is an existing
> mailing list for this component.

I'll remove this team information

https://codereview.chromium.org/2744723002/diff/20001/ui/message_center/OWNERS
File ui/message_center/OWNERS (right):

https://codereview.chromium.org/2744723002/diff/20001/ui/message_center/OWNER...
ui/message_center/OWNERS:6: # TEAM: platform-architecture-dev@chromium.org
On 2017/03/09 22:38:14, dewittj wrote:
> This team may not be right, I don't know the mission of
> platform-architecture-dev.  Don't think there's really a mailing list for this
> code at this point.

Done.

https://codereview.chromium.org/2744723002/diff/20001/ui/message_center/OWNER...
ui/message_center/OWNERS:7: # COMPONENT: Blink>Messaging
On 2017/03/09 22:09:14, Peter Beverloo wrote:
> UI>Notifications

Done.

[dewittj, 2017-03-10 01:08:51]

lgtm

[ymzhang1, 2017-03-10 01:10:32]

The CQ bit was checked by ymzhang@chromium.org

[commit-bot: I haz the power, 2017-03-10 01:11:09]

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-10 01:38:48]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1489108232451820,
"parent_rev": "0eb06aa0290f91b653252d790635f311759ed7ef", "commit_rev":
"589c696ecd13c722eb3b7eeef6eb43791978a687"}

[commit-bot: I haz the power, 2017-03-10 01:39:48]

Description was changed from

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

We are adding COMPONENT/TEAM information into OWNERS file
(offline_pages, 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=dewittj@chromium.org
==========

to

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

We are adding COMPONENT/TEAM information into OWNERS file
(offline_pages, 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=dewittj@chromium.org

Review-Url: https://codereview.chromium.org/2744723002
Cr-Commit-Position: refs/heads/master@{#455953}
Committed:
https://chromium.googlesource.com/chromium/src/+/589c696ecd13c722eb3b7eeef6eb...
==========

[commit-bot: I haz the power, 2017-03-10 01:39:49]

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