[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

We are adding COMPONENT/TEAM information into OWNERS file
(Cloud Print related). 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:
List of components
https://bugs.chromium.org/p/chromium/adminComponents

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

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

[ymzhang1, 2017-02-22 21:25:07]

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:
List of components
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:
List of components
https://bugs.chromium.org/p/chromium/adminComponents

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

[ymzhang1, 2017-02-22 21:25:34]

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

[ymzhang1, 2017-02-22 21:25:34]

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!

[gene1, 2017-02-22 22:20:28]

LGTM
However, neither me nor Vitaly or Scott work on Chrome anymore. We all moved to
different projects at this point and it probably make sense to find new owners
there.

[ymzhang1, 2017-02-22 23:40:55]

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

[ymzhang1, 2017-02-22 23:40:55]

Thanks for the information!

Maybe figuring out team mailing list for these components could help. 

+ Shruthi
Any update of team mailing list for these components?
Services>CloudPrint
Internals>Printing

[ymzhang1, 2017-02-22 23:41:00]

The CQ bit was checked by ymzhang@chromium.org

[commit-bot: I haz the power, 2017-02-22 23:41:39]

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-22 23:41:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-22 23:41:40]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[ymzhang1, 2017-02-22 23:47:29]

Hi gene@

Sorry to bother again. Would you mind lgtm with your chromium account if
possible? Thanks!

[sshruthi1, 2017-02-22 23:47:46]

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:
List of components
https://bugs.chromium.org/p/chromium/adminComponents

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

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:
List of components
https://bugs.chromium.org/p/chromium/adminComponents

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

[sshruthi1, 2017-02-22 23:47:46]

sshruthi@chromium.org changed reviewers:
+ thestig@chromium.org

[sshruthi1, 2017-02-22 23:48:14]

On 2017/02/22 23:40:55, ymzhang1 wrote:
> Thanks for the information!
> 
> Maybe figuring out team mailing list for these components could help. 
> 
> + Shruthi
> Any update of team mailing list for these components?
> Services>CloudPrint
> Internals>Printing

thestig@ might have ideas on what's the right team for these components?

[ymzhang1, 2017-02-23 00:07:38]

ymzhang@chromium.org changed reviewers:
- thestig@chromium.org

[Lei Zhang, 2017-02-23 01:14:58]

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

[Lei Zhang, 2017-02-23 01:15:01]

I'm very very behind on email. Care to shoot me a separate email about team
mailing lists and what you are looking for exactly there?

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

https://codereview.chromium.org/2712643004/diff/1/chrome/browser/local_discov...
chrome/browser/local_discovery/OWNERS:5: # COMPONENT: Blink>Storage>DOMStorage
Cloud Print

https://codereview.chromium.org/2712643004/diff/1/chrome/browser/resources/lo...
File chrome/browser/resources/local_discovery/OWNERS (right):

https://codereview.chromium.org/2712643004/diff/1/chrome/browser/resources/lo...
chrome/browser/resources/local_discovery/OWNERS:5: # COMPONENT:
Blink>Storage>DOMStorage
Cloud print

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

https://codereview.chromium.org/2712643004/diff/1/chrome/browser/service_proc...
chrome/browser/service_process/OWNERS:5: # COMPONENT: Services>Chromoting
Cloud print

https://codereview.chromium.org/2712643004/diff/1/chrome/browser/ui/webui/loc...
File chrome/browser/ui/webui/local_discovery/OWNERS (right):

https://codereview.chromium.org/2712643004/diff/1/chrome/browser/ui/webui/loc...
chrome/browser/ui/webui/local_discovery/OWNERS:4: # COMPONENT: UI>Browser>WebUI
Can we do multiple components? If yes, add cloud print here too.

https://codereview.chromium.org/2712643004/diff/1/chrome/service/OWNERS
File chrome/service/OWNERS (right):

https://codereview.chromium.org/2712643004/diff/1/chrome/service/OWNERS#newcode5
chrome/service/OWNERS:5: # COMPONENT: Services>Chromoting
Cloud print

https://codereview.chromium.org/2712643004/diff/1/chrome/tools/service_discov...
File chrome/tools/service_discovery_sniffer/OWNERS (right):

https://codereview.chromium.org/2712643004/diff/1/chrome/tools/service_discov...
chrome/tools/service_discovery_sniffer/OWNERS:5: # COMPONENT:
Blink>Storage>DOMStorage
No, this is still cloud print.

[ymzhang1, 2017-02-23 22:38:30]

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:
List of components
https://bugs.chromium.org/p/chromium/adminComponents

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

to

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

We are adding COMPONENT/TEAM information into OWNERS file
(Cloud Print related). 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:
List of components
https://bugs.chromium.org/p/chromium/adminComponents

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

[ymzhang1, 2017-02-23 22:41:43]

Sure. I will send you(thestig@) a separate email about team mailing lists. 

Additional note about this cl, 
I would like to collect team mailing list for component Services>CloudPrint and
Internals>Printing in this cl.

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

https://codereview.chromium.org/2712643004/diff/1/chrome/browser/local_discov...
chrome/browser/local_discovery/OWNERS:5: # COMPONENT: Blink>Storage>DOMStorage
On 2017/02/23 01:15:00, Lei Zhang (super slow) wrote:
> Cloud Print

Done.

https://codereview.chromium.org/2712643004/diff/1/chrome/browser/resources/lo...
File chrome/browser/resources/local_discovery/OWNERS (right):

https://codereview.chromium.org/2712643004/diff/1/chrome/browser/resources/lo...
chrome/browser/resources/local_discovery/OWNERS:5: # COMPONENT:
Blink>Storage>DOMStorage
On 2017/02/23 01:15:00, Lei Zhang (super slow) wrote:
> Cloud print

Done.

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

https://codereview.chromium.org/2712643004/diff/1/chrome/browser/service_proc...
chrome/browser/service_process/OWNERS:5: # COMPONENT: Services>Chromoting
On 2017/02/23 01:15:00, Lei Zhang (super slow) wrote:
> Cloud print

Done.

https://codereview.chromium.org/2712643004/diff/1/chrome/browser/ui/webui/loc...
File chrome/browser/ui/webui/local_discovery/OWNERS (right):

https://codereview.chromium.org/2712643004/diff/1/chrome/browser/ui/webui/loc...
chrome/browser/ui/webui/local_discovery/OWNERS:4: # COMPONENT: UI>Browser>WebUI
On 2017/02/23 01:15:00, Lei Zhang (super slow) wrote:
> Can we do multiple components? If yes, add cloud print here too.

For current plan, we would like to have only one primary component for each
directory.

https://codereview.chromium.org/2712643004/diff/1/chrome/service/OWNERS
File chrome/service/OWNERS (right):

https://codereview.chromium.org/2712643004/diff/1/chrome/service/OWNERS#newcode5
chrome/service/OWNERS:5: # COMPONENT: Services>Chromoting
On 2017/02/23 01:15:00, Lei Zhang (super slow) wrote:
> Cloud print

Done.

https://codereview.chromium.org/2712643004/diff/1/chrome/tools/service_discov...
File chrome/tools/service_discovery_sniffer/OWNERS (right):

https://codereview.chromium.org/2712643004/diff/1/chrome/tools/service_discov...
chrome/tools/service_discovery_sniffer/OWNERS:5: # COMPONENT:
Blink>Storage>DOMStorage
On 2017/02/23 01:15:00, Lei Zhang (super slow) wrote:
> No, this is still cloud print.

Done.

[Lei Zhang, 2017-02-23 22:45:27]

https://codereview.chromium.org/2712643004/diff/20001/chrome/browser/ui/webui...
File chrome/browser/ui/webui/local_discovery/OWNERS (right):

https://codereview.chromium.org/2712643004/diff/20001/chrome/browser/ui/webui...
chrome/browser/ui/webui/local_discovery/OWNERS:4: # COMPONENT: UI>Browser>WebUI
If you can only have 1 COMPONENT, I'd do Cloud Print here too.

https://codereview.chromium.org/2712643004/diff/20001/chrome/service/OWNERS
File chrome/service/OWNERS (right):

https://codereview.chromium.org/2712643004/diff/20001/chrome/service/OWNERS#n...
chrome/service/OWNERS:5: # COMPONENT: Services>Chromoting
Cloud Print

[Lei Zhang, 2017-02-23 22:46:52]

On 2017/02/23 22:41:43, ymzhang1 wrote:
> Sure. I will send you(thestig@) a separate email about team mailing lists. 
> 
> Additional note about this cl, 
> I would like to collect team mailing list for component Services>CloudPrint
and
> Internals>Printing in this cl.

I looked at some of the other existing TEAM entries. We don't have a public
mailing list for printing.

[ymzhang1, 2017-02-23 22:48:38]

https://codereview.chromium.org/2712643004/diff/20001/chrome/browser/ui/webui...
File chrome/browser/ui/webui/local_discovery/OWNERS (right):

https://codereview.chromium.org/2712643004/diff/20001/chrome/browser/ui/webui...
chrome/browser/ui/webui/local_discovery/OWNERS:4: # COMPONENT: UI>Browser>WebUI
On 2017/02/23 22:45:26, Lei Zhang (super slow) wrote:
> If you can only have 1 COMPONENT, I'd do Cloud Print here too.

Done.

https://codereview.chromium.org/2712643004/diff/20001/chrome/service/OWNERS
File chrome/service/OWNERS (right):

https://codereview.chromium.org/2712643004/diff/20001/chrome/service/OWNERS#n...
chrome/service/OWNERS:5: # COMPONENT: Services>Chromoting
On 2017/02/23 22:45:26, Lei Zhang (super slow) wrote:
> Cloud Print

Done.

[Lei Zhang, 2017-02-23 22:50:39]

lgtm

[ymzhang1, 2017-02-23 22:51:05]

The CQ bit was checked by ymzhang@chromium.org

[ymzhang1, 2017-02-23 22:51:05]

The patchset sent to the CQ was uploaded after l-g-t-m from gene@google.com
Link to the patchset: https://codereview.chromium.org/2712643004/#ps40001
(title: "update component")

[commit-bot: I haz the power, 2017-02-23 22:52:04]

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-23 23:13:36]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-23 23:13:37]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Lei Zhang, 2017-02-23 23:39:28]

thestig@chromium.org changed reviewers:
+ vitalybuka@chromium.org

[Lei Zhang, 2017-02-23 23:39:29]

+vitalybuka for a quick stamp for components/cloud_devices/OWNERS.

[Vitaly Buka (NO REVIEWS), 2017-02-24 00:17:09]

lgtm

https://codereview.chromium.org/2712643004/diff/40001/components/cloud_device...
File components/cloud_devices/OWNERS (left):

https://codereview.chromium.org/2712643004/diff/40001/components/cloud_device...
components/cloud_devices/OWNERS:2: vitalybuka@chromium.org
Lei, probably you should put yourself here. It contains only cloud print stuff
now.

[Lei Zhang, 2017-02-24 00:18:30]

https://codereview.chromium.org/2712643004/diff/40001/components/cloud_device...
File components/cloud_devices/OWNERS (left):

https://codereview.chromium.org/2712643004/diff/40001/components/cloud_device...
components/cloud_devices/OWNERS:2: vitalybuka@chromium.org
On 2017/02/24 00:17:08, Vitaly Buka wrote:
> Lei, probably you should put yourself here. It contains only cloud print stuff
> now.

Ack. Will do in a follow up after this lands.

[ymzhang1, 2017-02-24 00:19:08]

https://codereview.chromium.org/2712643004/diff/40001/components/cloud_device...
File components/cloud_devices/OWNERS (left):

https://codereview.chromium.org/2712643004/diff/40001/components/cloud_device...
components/cloud_devices/OWNERS:2: vitalybuka@chromium.org
On 2017/02/24 00:17:08, Vitaly Buka wrote:
> Lei, probably you should put yourself here. It contains only cloud print stuff
> now.

@Lei,

Would you like me to make this change in this cl?

[Lei Zhang, 2017-02-24 03:18:52]

On 2017/02/24 00:19:08, ymzhang1 wrote:
>
https://codereview.chromium.org/2712643004/diff/40001/components/cloud_device...
> File components/cloud_devices/OWNERS (left):
> 
>
https://codereview.chromium.org/2712643004/diff/40001/components/cloud_device...
> components/cloud_devices/OWNERS:2: mailto:vitalybuka@chromium.org
> On 2017/02/24 00:17:08, Vitaly Buka wrote:
> > Lei, probably you should put yourself here. It contains only cloud print
stuff
> > now.
> 
> @Lei,
> 
> Would you like me to make this change in this cl?

Sure!

[ymzhang1, 2017-02-24 17:56:36]

The CQ bit was checked by ymzhang@chromium.org

[ymzhang1, 2017-02-24 17:56:36]

The patchset sent to the CQ was uploaded after l-g-t-m from
thestig@chromium.org, gene@google.com, vitalybuka@chromium.org
Link to the patchset: https://codereview.chromium.org/2712643004/#ps60001
(title: "add owner")

[commit-bot: I haz the power, 2017-02-24 17:57:08]

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-24 18:25:52]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1487958996387670,
"parent_rev": "d5f450e8a4185ba2e78f76770acc63586f80ad2a", "commit_rev":
"fd502a80c0f1aabd7c18d74b8c15c8e93dddff59"}

[commit-bot: I haz the power, 2017-02-24 18:26:30]

Description was changed from

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

We are adding COMPONENT/TEAM information into OWNERS file
(Cloud Print related). 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:
List of components
https://bugs.chromium.org/p/chromium/adminComponents

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

to

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

We are adding COMPONENT/TEAM information into OWNERS file
(Cloud Print related). 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:
List of components
https://bugs.chromium.org/p/chromium/adminComponents

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

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

[commit-bot: I haz the power, 2017-02-24 18:26:32]

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