[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

[DirectoryOwnership] Add TEAM/COMPONENT into OWNERS files.

Adding COMPONENT/TEAM information into OWNERS file.

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:
List of components
https://bugs.chromium.org/p/chromium/adminComponents

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

[ymzhang1, 2017-01-25 19:39:54]

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

[ymzhang1, 2017-01-25 19:39:55]

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!

[mcasas, 2017-01-25 20:48:50]

https://codereview.chromium.org/2650943010/diff/1/content/browser/media/captu...
File content/browser/media/capture/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/1/content/browser/media/captu...
content/browser/media/capture/OWNERS:7: # COMPONENT: Internals>Media>Capture
Yeah, this component is confusing and should be
removed (fled https://crbug.com/685345).

Blink>GetUserMedia should be better.

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

https://codereview.chromium.org/2650943010/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/media/OWNERS:11: # COMPONENT: Internals>Media
Phew! This folder has a bunch of stuff but
ultimately it's all under Blink>GetUserMedia

https://codereview.chromium.org/2650943010/diff/1/content/renderer/media/OWNERS
File content/renderer/media/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/1/content/renderer/media/OWNE...
content/renderer/media/OWNERS:14: # COMPONENT: Internals>Media
This folder is by far the most messed up in real-time
audio/video land.  So probably Internals>Media is good
for the moment.

https://codereview.chromium.org/2650943010/diff/1/device/geolocation/OWNERS
File device/geolocation/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/1/device/geolocation/OWNERS#n...
device/geolocation/OWNERS:5: # TEAM: fizz-team-lon@chromium.org
Is this email even existing :?  If anything,
it could be fizz-team@, but I don't think
that team name is public (bc is a project,
not a team).  Just leave it out.

https://codereview.chromium.org/2650943010/diff/1/media/capture/video/OWNERS
File media/capture/video/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/1/media/capture/video/OWNERS#...
media/capture/video/OWNERS:6: # COMPONENT: Internals>Media>Video
Blink>GetUserMedia>WebCam

https://codereview.chromium.org/2650943010/diff/1/media/capture/video/android...
File media/capture/video/android/java/src/org/chromium/media/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/1/media/capture/video/android...
media/capture/video/android/java/src/org/chromium/media/OWNERS:4: # COMPONENT:
Internals>Media>Video
Blink>GetUserMedia>WebCam

https://codereview.chromium.org/2650943010/diff/1/media/muxers/OWNERS
File media/muxers/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/1/media/muxers/OWNERS#newcode4
media/muxers/OWNERS:4: # COMPONENT: Internals>Media
Blink>MediaRecording since it's the only user.

https://codereview.chromium.org/2650943010/diff/1/services/video_capture/OWNERS
File services/video_capture/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/1/services/video_capture/OWNE...
services/video_capture/OWNERS:5: # COMPONENT: Services
Hmm I'd rather make this
Blink>GetUserMedia

https://codereview.chromium.org/2650943010/diff/1/third_party/WebKit/Source/m...
File third_party/WebKit/Source/modules/mediacapturefromelement/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/mediacapturefromelement/OWNERS:4: # COMPONENT:
Internals>Media>Capture
Blink>MediaStream>CaptureFromElement

https://codereview.chromium.org/2650943010/diff/1/third_party/decklink/OWNERS
File third_party/decklink/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/1/third_party/decklink/OWNERS...
third_party/decklink/OWNERS:5: # COMPONENT: ChromeForWork
Blink>GetUserMedia, decklink 
is only used for video capture in Mac.

https://codereview.chromium.org/2650943010/diff/1/third_party/libwebm/OWNERS
File third_party/libwebm/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/1/third_party/libwebm/OWNERS#...
third_party/libwebm/OWNERS:9: # COMPONENT: Internals>Network>Library
Blink>MediaRecording  (since it's the only user)

[ymzhang1, 2017-01-25 23:33:37]

Thanks for the suggestions! Update the COMPONENT information, and add
media-capture-and-streams@grotations.appspotmail.com as team mailing list for
Blink>GetUserMedia.

https://codereview.chromium.org/2650943010/diff/1/content/browser/media/captu...
File content/browser/media/capture/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/1/content/browser/media/captu...
content/browser/media/capture/OWNERS:7: # COMPONENT: Internals>Media>Capture
On 2017/01/25 20:48:49, mcasas wrote:
> Yeah, this component is confusing and should be
> removed (fled https://crbug.com/685345).
> 
> Blink>GetUserMedia should be better.

Done.

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

https://codereview.chromium.org/2650943010/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/media/OWNERS:11: # COMPONENT: Internals>Media
On 2017/01/25 20:48:50, mcasas wrote:
> Phew! This folder has a bunch of stuff but
> ultimately it's all under Blink>GetUserMedia

Done.

https://codereview.chromium.org/2650943010/diff/1/content/renderer/media/OWNERS
File content/renderer/media/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/1/content/renderer/media/OWNE...
content/renderer/media/OWNERS:14: # COMPONENT: Internals>Media
On 2017/01/25 20:48:50, mcasas wrote:
> This folder is by far the most messed up in real-time
> audio/video land.  So probably Internals>Media is good
> for the moment.

Thanks for the comments!

https://codereview.chromium.org/2650943010/diff/1/device/geolocation/OWNERS
File device/geolocation/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/1/device/geolocation/OWNERS#n...
device/geolocation/OWNERS:5: # TEAM: fizz-team-lon@chromium.org
On 2017/01/25 20:48:50, mcasas wrote:
> Is this email even existing :?  If anything,
> it could be fizz-team@, but I don't think
> that team name is public (bc is a project,
> not a team).  Just leave it out.

Done.

https://codereview.chromium.org/2650943010/diff/1/media/capture/video/OWNERS
File media/capture/video/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/1/media/capture/video/OWNERS#...
media/capture/video/OWNERS:6: # COMPONENT: Internals>Media>Video
On 2017/01/25 20:48:50, mcasas wrote:
> Blink>GetUserMedia>WebCam

Done.

https://codereview.chromium.org/2650943010/diff/1/media/capture/video/android...
File media/capture/video/android/java/src/org/chromium/media/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/1/media/capture/video/android...
media/capture/video/android/java/src/org/chromium/media/OWNERS:4: # COMPONENT:
Internals>Media>Video
On 2017/01/25 20:48:50, mcasas wrote:
> Blink>GetUserMedia>WebCam

Done.

https://codereview.chromium.org/2650943010/diff/1/media/muxers/OWNERS
File media/muxers/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/1/media/muxers/OWNERS#newcode4
media/muxers/OWNERS:4: # COMPONENT: Internals>Media
On 2017/01/25 20:48:50, mcasas wrote:
> Blink>MediaRecording since it's the only user.

Done. Thanks for the explanation!

https://codereview.chromium.org/2650943010/diff/1/services/video_capture/OWNERS
File services/video_capture/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/1/services/video_capture/OWNE...
services/video_capture/OWNERS:5: # COMPONENT: Services
On 2017/01/25 20:48:50, mcasas wrote:
> Hmm I'd rather make this
> Blink>GetUserMedia

Done.

https://codereview.chromium.org/2650943010/diff/1/third_party/WebKit/Source/m...
File third_party/WebKit/Source/modules/mediacapturefromelement/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/mediacapturefromelement/OWNERS:4: # COMPONENT:
Internals>Media>Capture
On 2017/01/25 20:48:50, mcasas wrote:
> Blink>MediaStream>CaptureFromElement 

Done.

https://codereview.chromium.org/2650943010/diff/1/third_party/decklink/OWNERS
File third_party/decklink/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/1/third_party/decklink/OWNERS...
third_party/decklink/OWNERS:5: # COMPONENT: ChromeForWork
On 2017/01/25 20:48:50, mcasas wrote:
> Blink>GetUserMedia, decklink 
> is only used for video capture in Mac.

Done.

[mcasas, 2017-01-26 00:02:50]

lgtm

https://codereview.chromium.org/2650943010/diff/20001/media/capture/video/and...
File media/capture/video/android/java/src/org/chromium/media/OWNERS (right):

https://codereview.chromium.org/2650943010/diff/20001/media/capture/video/and...
media/capture/video/android/java/src/org/chromium/media/OWNERS:4: #
media-capture-and-streams@grotations.appspotmail.com
nit: prefix with TEAM.

[ymzhang1, 2017-01-26 00:03:58]

The CQ bit was checked by ymzhang@chromium.org

[commit-bot: I haz the power, 2017-01-26 00:05: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-26 00:18:22]

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

[commit-bot: I haz the power, 2017-01-26 00:18:23]

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

[ncarter (slow), 2017-01-26 00:35:41]

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

[ncarter (slow), 2017-01-26 00:35:42]

lgtm

[ymzhang1, 2017-01-26 01:33:18]

The CQ bit was checked by ymzhang@chromium.org

[commit-bot: I haz the power, 2017-01-26 01:34: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-01-26 03:02:00]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1485394398593230,
"parent_rev": "a980b812f2f213270551f986fc7c4fdf0cc2d3a8", "commit_rev":
"d1fd836e3f69578eb72e52d2da6f2f3106aa2d4b"}

[commit-bot: I haz the power, 2017-01-26 03:04:35]

Description was changed from

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

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

to

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

BUG=679905

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

[commit-bot: I haz the power, 2017-01-26 03:04:37]

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

[ymzhang1, 2017-03-02 21:18:23]

Description was changed from

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

BUG=679905

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

to

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

Adding COMPONENT/TEAM information into OWNERS file.

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:
List of components
https://bugs.chromium.org/p/chromium/adminComponents

BUG=679905

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