[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/2702423003
Cr-Commit-Position: refs/heads/master@{#456938}
Committed: https://chromium.googlesource.com/chromium/src/+/304bef4f5c6502b7a863bb7be2c956ff1fd1420f

[ymzhang1, 2017-02-21 21:46:48]

Patchset #1 (id:1) has been deleted

[ymzhang1, 2017-02-21 21:47:23]

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

[ymzhang1, 2017-02-21 21:47:24]

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!

[ymzhang1, 2017-02-21 21:50:03]

Patchset #1 (id:20001) has been deleted

[palmer, 2017-02-21 22:11:06]

https://codereview.chromium.org/2702423003/diff/40001/components/security_sta...
File components/security_state/OWNERS (right):

https://codereview.chromium.org/2702423003/diff/40001/components/security_sta...
components/security_state/OWNERS:4: # COMPONENT: Internals>CertAnalysis
I think this one should be  COMPONENT: Internals>Network>SSL.

https://codereview.chromium.org/2702423003/diff/40001/content/public/android/...
File content/public/android/java/src/org/chromium/content/browser/crypto/OWNERS
(right):

https://codereview.chromium.org/2702423003/diff/40001/content/public/android/...
content/public/android/java/src/org/chromium/content/browser/crypto/OWNERS:5: #
COMPONENT: Blink>WebCrypto
This isn't for WebCrypto. I'd say Internals>Core is a better fit.

https://codereview.chromium.org/2702423003/diff/40001/content/public/android/...
File
content/public/android/javatests/src/org/chromium/content/browser/crypto/OWNERS
(right):

https://codereview.chromium.org/2702423003/diff/40001/content/public/android/...
content/public/android/javatests/src/org/chromium/content/browser/crypto/OWNERS:5:
# COMPONENT: Blink>WebCrypto
Internals>Core.

[ymzhang1, 2017-02-21 22:16:18]

https://codereview.chromium.org/2702423003/diff/40001/components/security_sta...
File components/security_state/OWNERS (right):

https://codereview.chromium.org/2702423003/diff/40001/components/security_sta...
components/security_state/OWNERS:4: # COMPONENT: Internals>CertAnalysis
On 2017/02/21 22:11:05, palmer wrote:
> I think this one should be  COMPONENT: Internals>Network>SSL.

Done.

https://codereview.chromium.org/2702423003/diff/40001/content/public/android/...
File content/public/android/java/src/org/chromium/content/browser/crypto/OWNERS
(right):

https://codereview.chromium.org/2702423003/diff/40001/content/public/android/...
content/public/android/java/src/org/chromium/content/browser/crypto/OWNERS:5: #
COMPONENT: Blink>WebCrypto
On 2017/02/21 22:11:05, palmer wrote:
> This isn't for WebCrypto. I'd say Internals>Core is a better fit.

Done.

https://codereview.chromium.org/2702423003/diff/40001/content/public/android/...
File
content/public/android/javatests/src/org/chromium/content/browser/crypto/OWNERS
(right):

https://codereview.chromium.org/2702423003/diff/40001/content/public/android/...
content/public/android/javatests/src/org/chromium/content/browser/crypto/OWNERS:5:
# COMPONENT: Blink>WebCrypto
On 2017/02/21 22:11:05, palmer wrote:
> Internals>Core.

Done.

[ymzhang1, 2017-03-13 20:20:51]

Patchset #3 (id:80001) has been deleted

[ymzhang1, 2017-03-13 20:31:51]

Patchset #4 (id:120001) has been deleted

[ymzhang1, 2017-03-13 20:31:57]

Patchset #3 (id:100001) has been deleted

[ymzhang1, 2017-03-13 20:33:51]

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

[ymzhang1, 2017-03-13 20:33:52]

moved ssl related OWNERS files to 
https://codereview.chromium.org/2746523002/.

[gone, 2017-03-14 18:16:26]

I don't know what I'm supposed to be reviewing here; palmer@ already started.

[palmer, 2017-03-14 18:22:22]

lgtm

[ymzhang1, 2017-03-14 23:54:51]

The CQ bit was checked by ymzhang@chromium.org

[commit-bot: I haz the power, 2017-03-14 23:55:34]

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-15 01:26:09]

CQ is committing da patch.

Bot data: {"patchset_id": 140001, "attempt_start_ts": 1489535691888190,
"parent_rev": "3801f1a0449592bdb25e7155ac1eabf50610f207", "commit_rev":
"304bef4f5c6502b7a863bb7be2c956ff1fd1420f"}

[commit-bot: I haz the power, 2017-03-15 01:26:53]

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/2702423003
Cr-Commit-Position: refs/heads/master@{#456938}
Committed:
https://chromium.googlesource.com/chromium/src/+/304bef4f5c6502b7a863bb7be2c9...
==========

[commit-bot: I haz the power, 2017-03-15 01:26:55]

Committed patchset #3 (id:140001) as
https://chromium.googlesource.com/chromium/src/+/304bef4f5c6502b7a863bb7be2c9...