Add TEAM/COMPONENT

Add TEAM/COMPONENT

BUG=679905
Review-Url: https://codereview.chromium.org/2673673003
Cr-Commit-Position: refs/heads/master@{#448108}
Committed: https://chromium.googlesource.com/chromium/src/+/05bd488e584433dabf586586c5d79ee63228ceca

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

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

[ymzhang1, 2017-02-02 22:43:04]

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-02 22:44:39]

Patchset #1 (id:1) has been deleted

[nhiroki, 2017-02-03 13:48:23]

nhiroki@chromium.org changed reviewers:
+ jsbell@chromium.org

[nhiroki, 2017-02-03 13:48:24]

+jsbell@, can you check my comment in ontent/browser/fileapi/OWNERS?

https://codereview.chromium.org/2673673003/diff/20001/content/browser/fileapi...
File content/browser/fileapi/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/20001/content/browser/fileapi...
content/browser/fileapi/OWNERS:7: # COMPONENT: Blink>FileAPI
This and others under **/fileapi/ could be Blink>Storage>FileSystem?

jsbell@, any thoughts?

https://codereview.chromium.org/2673673003/diff/20001/content/browser/resourc...
File content/browser/resources/service_worker/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/20001/content/browser/resourc...
content/browser/resources/service_worker/OWNERS:9: # COMPONENT: Blink>Workers
Blink>ServiceWorker

https://codereview.chromium.org/2673673003/diff/20001/content/browser/service...
File content/browser/service_worker/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/20001/content/browser/service...
content/browser/service_worker/OWNERS:9: # COMPONENT: Blink>Workers
Blink>ServiceWorker

https://codereview.chromium.org/2673673003/diff/20001/content/child/service_w...
File content/child/service_worker/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/20001/content/child/service_w...
content/child/service_worker/OWNERS:9: # COMPONENT: Blink>Workers
Blink>ServiceWorker

https://codereview.chromium.org/2673673003/diff/20001/content/common/service_...
File content/common/service_worker/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/20001/content/common/service_...
content/common/service_worker/OWNERS:15: # COMPONENT: Blink>Workers
Blink>ServiceWorker

https://codereview.chromium.org/2673673003/diff/20001/content/renderer/servic...
File content/renderer/service_worker/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/20001/content/renderer/servic...
content/renderer/service_worker/OWNERS:9: # COMPONENT: Blink>Workers
Blink>ServiceWorker

[jsbell, 2017-02-03 17:57:39]

I agree with nhiroki@

(I'm hoping we can tease apart fileapi and filesystem implementation when we
mojoify everything.)

https://codereview.chromium.org/2673673003/diff/20001/content/browser/fileapi...
File content/browser/fileapi/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/20001/content/browser/fileapi...
content/browser/fileapi/OWNERS:7: # COMPONENT: Blink>FileAPI
On 2017/02/03 13:48:24, nhiroki (slow - traveling) wrote:
> This and others under **/fileapi/ could be Blink>Storage>FileSystem?
> 
> jsbell@, any thoughts?

Agreed. Both components end up being implemented here but there's more code
required for FileSystem so we should use that component.

[ymzhang1, 2017-02-03 19:05:21]

https://codereview.chromium.org/2673673003/diff/20001/content/browser/fileapi...
File content/browser/fileapi/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/20001/content/browser/fileapi...
content/browser/fileapi/OWNERS:7: # COMPONENT: Blink>FileAPI
On 2017/02/03 17:57:39, jsbell (at-BlinkOn-slow) wrote:
> On 2017/02/03 13:48:24, nhiroki (slow - traveling) wrote:
> > This and others under **/fileapi/ could be Blink>Storage>FileSystem?
> > 
> > jsbell@, any thoughts?
> 
> Agreed. Both components end up being implemented here but there's more code
> required for FileSystem so we should use that component.

Done.

https://codereview.chromium.org/2673673003/diff/20001/content/browser/resourc...
File content/browser/resources/service_worker/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/20001/content/browser/resourc...
content/browser/resources/service_worker/OWNERS:9: # COMPONENT: Blink>Workers
On 2017/02/03 13:48:24, nhiroki (slow - traveling) wrote:
> Blink>ServiceWorker

Done.

https://codereview.chromium.org/2673673003/diff/20001/content/browser/service...
File content/browser/service_worker/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/20001/content/browser/service...
content/browser/service_worker/OWNERS:9: # COMPONENT: Blink>Workers
On 2017/02/03 13:48:24, nhiroki (slow - traveling) wrote:
> Blink>ServiceWorker

Done.

https://codereview.chromium.org/2673673003/diff/20001/content/child/service_w...
File content/child/service_worker/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/20001/content/child/service_w...
content/child/service_worker/OWNERS:9: # COMPONENT: Blink>Workers
On 2017/02/03 13:48:24, nhiroki (slow - traveling) wrote:
> Blink>ServiceWorker

Done.

https://codereview.chromium.org/2673673003/diff/20001/content/common/service_...
File content/common/service_worker/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/20001/content/common/service_...
content/common/service_worker/OWNERS:15: # COMPONENT: Blink>Workers
On 2017/02/03 13:48:24, nhiroki (slow - traveling) wrote:
> Blink>ServiceWorker

Done.

https://codereview.chromium.org/2673673003/diff/20001/content/renderer/servic...
File content/renderer/service_worker/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/20001/content/renderer/servic...
content/renderer/service_worker/OWNERS:9: # COMPONENT: Blink>Workers
On 2017/02/03 13:48:24, nhiroki (slow - traveling) wrote:
> Blink>ServiceWorker

Done.

[nhiroki, 2017-02-03 21:18:34]

https://codereview.chromium.org/2673673003/diff/20001/content/browser/fileapi...
File content/browser/fileapi/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/20001/content/browser/fileapi...
content/browser/fileapi/OWNERS:7: # COMPONENT: Blink>FileAPI
On 2017/02/03 17:57:39, jsbell wrote:
> On 2017/02/03 13:48:24, nhiroki (slow - traveling) wrote:
> > This and others under **/fileapi/ could be Blink>Storage>FileSystem?
> > 
> > jsbell@, any thoughts?
> 
> Agreed. Both components end up being implemented here but there's more code
> required for FileSystem so we should use that component.

Thank you.

https://codereview.chromium.org/2673673003/diff/40001/content/child/fileapi/O...
File content/child/fileapi/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/40001/content/child/fileapi/O...
content/child/fileapi/OWNERS:7: # COMPONENT: Blink>FileAPI
Blink>Storage>FileSystem?

https://codereview.chromium.org/2673673003/diff/40001/content/common/fileapi/...
File content/common/fileapi/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/40001/content/common/fileapi/...
content/common/fileapi/OWNERS:10: # COMPONENT: Blink>FileAPI
Blink>Storage>FileSystem

https://codereview.chromium.org/2673673003/diff/40001/storage/browser/fileapi...
File storage/browser/fileapi/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/40001/storage/browser/fileapi...
storage/browser/fileapi/OWNERS:5: # COMPONENT: Blink>FileAPI
Blink>Storage>FileSystem

https://codereview.chromium.org/2673673003/diff/40001/storage/common/fileapi/...
File storage/common/fileapi/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/40001/storage/common/fileapi/...
storage/common/fileapi/OWNERS:5: # COMPONENT: Blink>FileAPI
Blink>Storage>FileSystem

[ymzhang1, 2017-02-03 21:42:20]

Sorry. I should have made these changes in the previous patch. Thanks for your
patience!

https://codereview.chromium.org/2673673003/diff/40001/content/child/fileapi/O...
File content/child/fileapi/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/40001/content/child/fileapi/O...
content/child/fileapi/OWNERS:7: # COMPONENT: Blink>FileAPI
On 2017/02/03 21:18:34, nhiroki (slow - traveling) wrote:
> Blink>Storage>FileSystem?

Done.

https://codereview.chromium.org/2673673003/diff/40001/content/common/fileapi/...
File content/common/fileapi/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/40001/content/common/fileapi/...
content/common/fileapi/OWNERS:10: # COMPONENT: Blink>FileAPI
On 2017/02/03 21:18:34, nhiroki (slow - traveling) wrote:
> Blink>Storage>FileSystem

Done.

https://codereview.chromium.org/2673673003/diff/40001/storage/browser/fileapi...
File storage/browser/fileapi/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/40001/storage/browser/fileapi...
storage/browser/fileapi/OWNERS:5: # COMPONENT: Blink>FileAPI
On 2017/02/03 21:18:34, nhiroki (slow - traveling) wrote:
> Blink>Storage>FileSystem

Done.

https://codereview.chromium.org/2673673003/diff/40001/storage/common/fileapi/...
File storage/common/fileapi/OWNERS (right):

https://codereview.chromium.org/2673673003/diff/40001/storage/common/fileapi/...
storage/common/fileapi/OWNERS:5: # COMPONENT: Blink>FileAPI
On 2017/02/03 21:18:34, nhiroki (slow - traveling) wrote:
> Blink>Storage>FileSystem

Done.

[nhiroki, 2017-02-03 21:47:54]

LGTM!

On 2017/02/03 21:42:20, ymzhang1 wrote:
> Sorry. I should have made these changes in the previous patch. Thanks for your
> patience!

No problem! :)

[ymzhang1, 2017-02-03 21:49:16]

The CQ bit was checked by ymzhang@chromium.org

[commit-bot: I haz the power, 2017-02-03 21:49:42]

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

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1486158556043690,
"parent_rev": "68e89f9a47050d3fb41818be880f3c6fcf6cf533", "commit_rev":
"05bd488e584433dabf586586c5d79ee63228ceca"}

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

Description was changed from

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

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

to

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

BUG=679905

Review-Url: https://codereview.chromium.org/2673673003
Cr-Commit-Position: refs/heads/master@{#448108}
Committed:
https://chromium.googlesource.com/chromium/src/+/05bd488e584433dabf586586c5d7...
==========

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

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