Add TEAM/COMPONENT

Add TEAM/COMPONENT

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

Review-Url: https://codereview.chromium.org/2677613005
Cr-Commit-Position: refs/heads/master@{#448681}
Committed: https://chromium.googlesource.com/chromium/src/+/61bfd27852074dac34ab7b3225fcbf92d9b0689f

[ymzhang1, 2017-02-03 22:42:44]

Description was changed from

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

BUG=679905
==========

to

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

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

[ymzhang1, 2017-02-03 22:43:30]

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

[ymzhang1, 2017-02-03 22:43:31]

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!

[vabr (Chromium), 2017-02-04 00:35:36]

Drive-by: *password_manager* LGTM

Cheers,
Vaclav

[ymzhang1, 2017-02-04 01:39:56]

On 2017/02/04 00:35:36, vabr (OOO until late Feb) wrote:
> Drive-by: *password_manager* LGTM
> 
> Cheers,
> Vaclav

Thanks for *review password_manager*!

[jochen (gone - plz use gerrit), 2017-02-04 18:45:08]

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

[jochen (gone - plz use gerrit), 2017-02-04 18:45:09]

https://codereview.chromium.org/2677613005/diff/1/components/test_runner/OWNERS
File components/test_runner/OWNERS (right):

https://codereview.chromium.org/2677613005/diff/1/components/test_runner/OWNE...
components/test_runner/OWNERS:9: # COMPONENT: Test
that'd be Label: Test>Layout

https://codereview.chromium.org/2677613005/diff/1/net/cookies/OWNERS
File net/cookies/OWNERS (right):

https://codereview.chromium.org/2677613005/diff/1/net/cookies/OWNERS#newcode4
net/cookies/OWNERS:4: # COMPONENT: Internals
Internals>Net>Cookies

https://codereview.chromium.org/2677613005/diff/1/url/OWNERS
File url/OWNERS (right):

https://codereview.chromium.org/2677613005/diff/1/url/OWNERS#newcode4
url/OWNERS:4: # COMPONENT: Blink>Forms>Url
this is incorrect. I don't think there's a label or component that applies to
this directory

[ymzhang1, 2017-02-06 17:41:47]

https://codereview.chromium.org/2677613005/diff/1/components/test_runner/OWNERS
File components/test_runner/OWNERS (right):

https://codereview.chromium.org/2677613005/diff/1/components/test_runner/OWNE...
components/test_runner/OWNERS:9: # COMPONENT: Test
On 2017/02/04 18:45:09, jochen wrote:
> that'd be Label: Test>Layout

Done.

https://codereview.chromium.org/2677613005/diff/1/net/cookies/OWNERS
File net/cookies/OWNERS (right):

https://codereview.chromium.org/2677613005/diff/1/net/cookies/OWNERS#newcode4
net/cookies/OWNERS:4: # COMPONENT: Internals
On 2017/02/04 18:45:09, jochen wrote:
> Internals>Net>Cookies

Thanks for the comments. The correct component has been added already. I'll
remove change of this file in this cl.

https://codereview.chromium.org/2677613005/diff/1/url/OWNERS
File url/OWNERS (right):

https://codereview.chromium.org/2677613005/diff/1/url/OWNERS#newcode4
url/OWNERS:4: # COMPONENT: Blink>Forms>Url
On 2017/02/04 18:45:09, jochen wrote:
> this is incorrect. I don't think there's a label or component that applies to
> this directory

Done.

[jochen (gone - plz use gerrit), 2017-02-07 15:08:53]

lgtm

[ymzhang1, 2017-02-07 17:23:46]

The CQ bit was checked by ymzhang@chromium.org

[ymzhang1, 2017-02-07 17:23:46]

The patchset sent to the CQ was uploaded after l-g-t-m from vabr@chromium.org
Link to the patchset: https://codereview.chromium.org/2677613005/#ps20001
(title: "update component")

[commit-bot: I haz the power, 2017-02-07 17:24:11]

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-07 18:55:54]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1486488226596550,
"parent_rev": "45f66398546e5f402fb9768e05c1321893dbee6b", "commit_rev":
"61bfd27852074dac34ab7b3225fcbf92d9b0689f"}

[commit-bot: I haz the power, 2017-02-07 18:56:51]

Description was changed from

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

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

to

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

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

Review-Url: https://codereview.chromium.org/2677613005
Cr-Commit-Position: refs/heads/master@{#448681}
Committed:
https://chromium.googlesource.com/chromium/src/+/61bfd27852074dac34ab7b3225fc...
==========

[commit-bot: I haz the power, 2017-02-07 18:56:52]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/61bfd27852074dac34ab7b3225fc...