[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

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

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

[ymzhang1, 2017-03-10 21:20:37]

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!

[Finnur, 2017-03-14 10:31:15]

Sorry for the late response, missed this yesterday.

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

https://codereview.chromium.org/2745833002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/preferences/OWNERS:4: #
COMPONENT: UI>Browser>Preferences>Protector
Browser>Preferences makes sense. Protector? No idea. Maybe twellington@ knows.

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

https://codereview.chromium.org/2745833002/diff/1/chrome/android/javatests/sr...
chrome/android/javatests/src/org/chromium/chrome/browser/preferences/OWNERS:4: #
COMPONENT: UI>Browser>Preferences>Protector
Ditto.

https://codereview.chromium.org/2745833002/diff/1/chrome/browser/extensions/a...
File chrome/browser/extensions/api/commands/OWNERS (right):

https://codereview.chromium.org/2745833002/diff/1/chrome/browser/extensions/a...
chrome/browser/extensions/api/commands/OWNERS:5: # COMPONENT:
Blink>Editing>Command
Some of the other APIs have 
# COMPONENT: Platform>Extensions>API

... which seems more appropriate here.

[ymzhang1, 2017-03-14 23:39:22]

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

[ymzhang1, 2017-03-14 23:39:23]

+twellington@

Could you help to review the component added in the following files?
chrome/android/java/src/org/chromium/chrome/browser/preferences/OWNERS
chrome/android/javatests/src/org/chromium/chrome/browser/preferences/OWNERS

Thanks!

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

https://codereview.chromium.org/2745833002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/preferences/OWNERS:4: #
COMPONENT: UI>Browser>Preferences>Protector
On 2017/03/14 10:31:15, Finnur wrote:
> Browser>Preferences makes sense. Protector? No idea. Maybe twellington@ knows.

Done.

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

https://codereview.chromium.org/2745833002/diff/1/chrome/android/javatests/sr...
chrome/android/javatests/src/org/chromium/chrome/browser/preferences/OWNERS:4: #
COMPONENT: UI>Browser>Preferences>Protector
On 2017/03/14 10:31:15, Finnur wrote:
> Ditto.

Done.

https://codereview.chromium.org/2745833002/diff/1/chrome/browser/extensions/a...
File chrome/browser/extensions/api/commands/OWNERS (right):

https://codereview.chromium.org/2745833002/diff/1/chrome/browser/extensions/a...
chrome/browser/extensions/api/commands/OWNERS:5: # COMPONENT:
Blink>Editing>Command
On 2017/03/14 10:31:15, Finnur wrote:
> Some of the other APIs have 
> # COMPONENT: Platform>Extensions>API
> 
> ... which seems more appropriate here.

Done.

[Theresa, 2017-03-14 23:43:45]

https://codereview.chromium.org/2745833002/diff/20001/chrome/android/java/src...
File chrome/android/java/src/org/chromium/chrome/browser/preferences/OWNERS
(right):

https://codereview.chromium.org/2745833002/diff/20001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/preferences/OWNERS:4: #
COMPONENT: UI>Browser>Preferences
We typically use UI>Settings for bugs related to the Java classes in this
package.

UI>Browser>Preferences doesn't appear to exist in crbug.

[ymzhang1, 2017-03-14 23:53:07]

https://codereview.chromium.org/2745833002/diff/20001/chrome/android/java/src...
File chrome/android/java/src/org/chromium/chrome/browser/preferences/OWNERS
(right):

https://codereview.chromium.org/2745833002/diff/20001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/preferences/OWNERS:4: #
COMPONENT: UI>Browser>Preferences
On 2017/03/14 23:43:45, Theresa wrote:
> We typically use UI>Settings for bugs related to the Java classes in this
> package.
> 
> UI>Browser>Preferences doesn't appear to exist in crbug.

That's true. My bad. Thanks for pointing out and the suggestion!

[Theresa, 2017-03-15 15:01:12]

lgtm

[Finnur, 2017-03-16 10:28:56]

The CQ bit was checked by finnur@chromium.org

[Finnur, 2017-03-16 10:29:00]

The CQ bit was unchecked by finnur@chromium.org

[ymzhang1, 2017-03-21 16:45:06]

The CQ bit was checked by ymzhang@chromium.org

[commit-bot: I haz the power, 2017-03-21 16:45:55]

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-21 16:58:48]

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

[commit-bot: I haz the power, 2017-03-21 16:58:49]

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...)

[Finnur, 2017-03-31 09:51:29]

LGTM

[ymzhang1, 2017-03-31 13:50:04]

The CQ bit was checked by ymzhang@chromium.org

[commit-bot: I haz the power, 2017-03-31 13:50:32]

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-31 14:01:26]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1490968204788690,
"parent_rev": "c1eda31a97825bb7504ca74b5b14a6cfc73de03d", "commit_rev":
"eedc8af55aa7e6fea5b8430c2534fe0ce0a3be3a"}

[commit-bot: I haz the power, 2017-03-31 14:02:10]

Description was changed from

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

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

to

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

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

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

[commit-bot: I haz the power, 2017-03-31 14:02:12]

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