Add TEAM/COMPONENT to OWNERS

add TEAM/COMPONENT information to OWNERS file

BUG=679905
Review-Url: https://codereview.chromium.org/2642373003
Cr-Commit-Position: refs/heads/master@{#446387}
Committed: https://chromium.googlesource.com/chromium/src/+/3e8c32bb3006e746c373356020923da18fbce6cc

[ymzhang1, 2017-01-21 01:43:35]

Description was changed from

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

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

to

==========
add TEAM/COMPONENT information to OWNERS file

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

[ymzhang1, 2017-01-21 01:43:36]

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

[ymzhang1, 2017-01-21 01:44:51]

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!

[Justin Novosad, 2017-01-23 00:49:05]

https://codereview.chromium.org/2642373003/diff/1/content/browser/renderer_ho...
File content/browser/renderer_host/OWNERS (right):

https://codereview.chromium.org/2642373003/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/OWNERS:37: # COMPONENT: Content>WebApps
This directory is very diverse and does not fit into a single component, but I
think the common thread is Internals>Sandbox>SiteIsolation

https://codereview.chromium.org/2642373003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/imagebitmap/OWNERS (right):

https://codereview.chromium.org/2642373003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/imagebitmap/OWNERS:5: # COMPONENT: Blink>Image
This interface is closely related to canvas and it is maintained by the canvas
team, so it would be more effective to put Blink>Canvas here.

[ymzhang1, 2017-01-24 17:26:23]

Thanks for the comments! Updated as suggested.

https://codereview.chromium.org/2642373003/diff/1/content/browser/renderer_ho...
File content/browser/renderer_host/OWNERS (right):

https://codereview.chromium.org/2642373003/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/OWNERS:37: # COMPONENT: Content>WebApps
On 2017/01/23 00:49:05, Justin Novosad wrote:
> This directory is very diverse and does not fit into a single component, but I
> think the common thread is Internals>Sandbox>SiteIsolation

Done.

https://codereview.chromium.org/2642373003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/imagebitmap/OWNERS (right):

https://codereview.chromium.org/2642373003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/imagebitmap/OWNERS:5: # COMPONENT: Blink>Image
On 2017/01/23 00:49:05, Justin Novosad wrote:
> This interface is closely related to canvas and it is maintained by the canvas
> team, so it would be more effective to put Blink>Canvas here.

Done.

[Justin Novosad, 2017-01-24 21:08:20]

lgtm

[ymzhang1, 2017-01-24 21:11:53]

The CQ bit was checked by ymzhang@chromium.org

[commit-bot: I haz the power, 2017-01-24 21:12:33]

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-01-24 21:23:15]

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

[commit-bot: I haz the power, 2017-01-24 21:23:16]

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

[ymzhang1, 2017-01-25 21:59:30]

The CQ bit was checked by ymzhang@chromium.org

[ymzhang1, 2017-01-25 21:59:31]

The patchset sent to the CQ was uploaded after l-g-t-m from junov@chromium.org
Link to the patchset: https://codereview.chromium.org/2642373003/#ps40001
(title: "Merge branch 'master' ")

[commit-bot: I haz the power, 2017-01-25 22:00: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-01-25 22:11:14]

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

[commit-bot: I haz the power, 2017-01-25 22:11:14]

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

[ymzhang1, 2017-01-25 22:19:20]

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

[ymzhang1, 2017-01-25 22:19:20]

asvitkine@chromium.org: Please review changes in 

# COMPONENT in content/browser/renderer_host/OWNER. We need a LGTM to add
COMPONENT information. Thanks!

[Alexei Svitkine (slow), 2017-01-26 16:29:19]

I don't think SiteIsolation is the right component for this.

I looked at the list under Internals> and I think Internals>Core is the best fit
for it - since renderer_host is basically part of the core infrastructure for
spinning up renderer processes.

[ymzhang1, 2017-01-26 17:10:28]

On 2017/01/26 16:29:19, Alexei Svitkine (slow) wrote:
> I don't think SiteIsolation is the right component for this.
> 
> I looked at the list under Internals> and I think Internals>Core is the best
fit
> for it - since renderer_host is basically part of the core infrastructure for
> spinning up renderer processes.

Good to know that! Updated as suggested.

[Alexei Svitkine (slow), 2017-01-26 17:15:38]

lgtm

[ymzhang1, 2017-01-26 17:16:37]

The CQ bit was checked by ymzhang@chromium.org

[ymzhang1, 2017-01-26 17:16:37]

The patchset sent to the CQ was uploaded after l-g-t-m from junov@chromium.org
Link to the patchset: https://codereview.chromium.org/2642373003/#ps60001
(title: "update compo")

[commit-bot: I haz the power, 2017-01-26 17:17:05]

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-01-26 19:25:06]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1485450997179700,
"parent_rev": "9e084e668b738137e23cbc7f19bd21d1f40c5fe9", "commit_rev":
"3e8c32bb3006e746c373356020923da18fbce6cc"}

[commit-bot: I haz the power, 2017-01-26 19:25:39]

Description was changed from

==========
add TEAM/COMPONENT information to OWNERS file

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

to

==========
add TEAM/COMPONENT information to OWNERS file

BUG=679905

Review-Url: https://codereview.chromium.org/2642373003
Cr-Commit-Position: refs/heads/master@{#446387}
Committed:
https://chromium.googlesource.com/chromium/src/+/3e8c32bb3006e746c37335602092...
==========

[commit-bot: I haz the power, 2017-01-26 19:25:43]

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