Update src/device OWNERS files with TEAM / COMPONENTS and removing emeritus.

Update src/device OWNERS files with TEAM / COMPONENTS and removing emeritus.

device-dev@chromium.org now exists and is taking on triage of most of
src/device. This change associates the TEAM in OWNERS files and updates
component labels as well.

The root device/OWNERS sets the TEAM that is used in most subdirectories.

Bluetooth has a specific team list, and has components added. 
Redundant OWNERS files are removed.

mvanouwerkerk@chromium.org is removed, as they now work in other areas.

geolocation does not need a redundant COMPONENT in the interfaces directory.

media_transfer_protocol given Media component.

Review-Url: https://codereview.chromium.org/2789433002
Cr-Commit-Position: refs/heads/master@{#461263}
Committed: https://chromium.googlesource.com/chromium/src/+/82e698b4a53d8592222d7e641275f14057665f4e

[scheib, 2017-03-30 04:26:54]

Description was changed from

==========
Update src/device OWNERS files with TEAM / COMPONENTS and removing emeritus.

device-dev@chromium.org now exists and is taking on triage of most of
src/device. This change associates the TEAM in OWNERS files and updates
component labels as well.

mvanouwerkerk@chromium.org is removed, as they now work in other areas.

Redundant OWNERS files are removed.
==========

to

==========
Update src/device OWNERS files with TEAM / COMPONENTS and removing emeritus.

device-dev@chromium.org now exists and is taking on triage of most of
src/device. This change associates the TEAM in OWNERS files and updates
component labels as well.

The root device/OWNERS sets the TEAM that is used in most subdirectories.

Bluetooth has a specific team list, and has components added. Redundant OWNERS
files are removed.

mvanouwerkerk@chromium.org is removed, as they now work in other areas.

geolocation does not need a redundant COMPONENT in the interfaces directory.

media_transfer_protocol and screen_orientation are given Media components.
==========

[scheib, 2017-03-30 04:28:37]

scheib@chromium.org changed reviewers:
+ mlamouri@chromium.org, mvanouwerkerk@google.com

[scheib, 2017-03-30 04:28:38]

mvanouwerkerk@google.com
Please verify your removal.

mlamouri@chromium.org
Please verify or suggest COMPONENT to use for media_transfer_protocol and
screen_orientation. Does an appropriate team group exist?

[Michael van Ouwerkerk, 2017-03-30 09:31:15]

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

[Michael van Ouwerkerk, 2017-03-30 09:31:15]

lgtm

[timvolodine, 2017-03-30 10:36:28]

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

[timvolodine, 2017-03-30 10:36:29]

https://codereview.chromium.org/2789433002/diff/1/device/sensors/OWNERS
File device/sensors/OWNERS (left):

https://codereview.chromium.org/2789433002/diff/1/device/sensors/OWNERS#oldcode3
device/sensors/OWNERS:3: # COMPONENT: IO>DeviceOrientation
this OWNERS file is actually being discussed as part of
https://codereview.chromium.org/2774433002/.
perhaps best to remove from this patch to avoid clashes.

[mlamouri (slow - plz ping), 2017-03-30 11:00:18]

lgtm with the comment applied.

https://codereview.chromium.org/2789433002/diff/1/device/screen_orientation/O...
File device/screen_orientation/OWNERS (right):

https://codereview.chromium.org/2789433002/diff/1/device/screen_orientation/O...
device/screen_orientation/OWNERS:4: # COMPONENT: Internals>Media
Not really the right component. Can you not add one for now. I will ask for one
to be created.

[scheib, 2017-03-30 18:02:47]

The CQ bit was checked by scheib@chromium.org to run a CQ dry run

[scheib, 2017-03-30 18:03:15]

scheib@chromium.org changed reviewers:
+ reillyg@google.com

[scheib, 2017-03-30 18:03:15]

reillyg PTAL

https://codereview.chromium.org/2789433002/diff/1/device/screen_orientation/O...
File device/screen_orientation/OWNERS (right):

https://codereview.chromium.org/2789433002/diff/1/device/screen_orientation/O...
device/screen_orientation/OWNERS:4: # COMPONENT: Internals>Media
On 2017/03/30 11:00:18, mlamouri wrote:
> Not really the right component. Can you not add one for now. I will ask for
one
> to be created.

Done.

https://codereview.chromium.org/2789433002/diff/1/device/sensors/OWNERS
File device/sensors/OWNERS (left):

https://codereview.chromium.org/2789433002/diff/1/device/sensors/OWNERS#oldcode3
device/sensors/OWNERS:3: # COMPONENT: IO>DeviceOrientation
On 2017/03/30 10:36:29, timvolodine wrote:
> this OWNERS file is actually being discussed as part of
> https://codereview.chromium.org/2774433002/.
> perhaps best to remove from this patch to avoid clashes.

Done.

[commit-bot: I haz the power, 2017-03-30 18:03:36]

Dry run: 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-30 19:16:41]

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

[commit-bot: I haz the power, 2017-03-30 19:16:42]

Dry run: This issue passed the CQ dry run.

[scheib, 2017-03-30 22:26:16]

Description was changed from

==========
Update src/device OWNERS files with TEAM / COMPONENTS and removing emeritus.

device-dev@chromium.org now exists and is taking on triage of most of
src/device. This change associates the TEAM in OWNERS files and updates
component labels as well.

The root device/OWNERS sets the TEAM that is used in most subdirectories.

Bluetooth has a specific team list, and has components added. Redundant OWNERS
files are removed.

mvanouwerkerk@chromium.org is removed, as they now work in other areas.

geolocation does not need a redundant COMPONENT in the interfaces directory.

media_transfer_protocol and screen_orientation are given Media components.
==========

to

==========
Update src/device OWNERS files with TEAM / COMPONENTS and removing emeritus.

device-dev@chromium.org now exists and is taking on triage of most of
src/device. This change associates the TEAM in OWNERS files and updates
component labels as well.

The root device/OWNERS sets the TEAM that is used in most subdirectories.

Bluetooth has a specific team list, and has components added. Redundant OWNERS
files are removed.

mvanouwerkerk@chromium.org is removed, as they now work in other areas.

geolocation does not need a redundant COMPONENT in the interfaces directory.

media_transfer_protocol given Media component.
==========

[scheib, 2017-03-30 22:26:36]

Description was changed from

==========
Update src/device OWNERS files with TEAM / COMPONENTS and removing emeritus.

device-dev@chromium.org now exists and is taking on triage of most of
src/device. This change associates the TEAM in OWNERS files and updates
component labels as well.

The root device/OWNERS sets the TEAM that is used in most subdirectories.

Bluetooth has a specific team list, and has components added. Redundant OWNERS
files are removed.

mvanouwerkerk@chromium.org is removed, as they now work in other areas.

geolocation does not need a redundant COMPONENT in the interfaces directory.

media_transfer_protocol given Media component.
==========

to

==========
Update src/device OWNERS files with TEAM / COMPONENTS and removing emeritus.

device-dev@chromium.org now exists and is taking on triage of most of
src/device. This change associates the TEAM in OWNERS files and updates
component labels as well.

The root device/OWNERS sets the TEAM that is used in most subdirectories.

Bluetooth has a specific team list, and has components added. 
Redundant OWNERS files are removed.

mvanouwerkerk@chromium.org is removed, as they now work in other areas.

geolocation does not need a redundant COMPONENT in the interfaces directory.

media_transfer_protocol given Media component.
==========

[scheib, 2017-03-31 21:09:52]

scheib@chromium.org changed reviewers:
+ reillyg@chromium.org
- reillyg@google.com

[Reilly Grant (use Gerrit), 2017-03-31 21:15:21]

lgtm

[scheib, 2017-03-31 21:37:14]

The CQ bit was checked by scheib@chromium.org

[scheib, 2017-03-31 21:37:14]

The patchset sent to the CQ was uploaded after l-g-t-m from
mlamouri@chromium.org, mvanouwerkerk@chromium.org
Link to the patchset: https://codereview.chromium.org/2789433002/#ps20001
(title: "Address comments, removing changes to
device/screen_orientation/OWNERS")

[commit-bot: I haz the power, 2017-03-31 21:38:13]

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

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490996234311640,
"parent_rev": "887c78b39f27eccd42ec308d1b899efc2a8f7a49", "commit_rev":
"82e698b4a53d8592222d7e641275f14057665f4e"}

[commit-bot: I haz the power, 2017-03-31 22:57:55]

Description was changed from

==========
Update src/device OWNERS files with TEAM / COMPONENTS and removing emeritus.

device-dev@chromium.org now exists and is taking on triage of most of
src/device. This change associates the TEAM in OWNERS files and updates
component labels as well.

The root device/OWNERS sets the TEAM that is used in most subdirectories.

Bluetooth has a specific team list, and has components added. 
Redundant OWNERS files are removed.

mvanouwerkerk@chromium.org is removed, as they now work in other areas.

geolocation does not need a redundant COMPONENT in the interfaces directory.

media_transfer_protocol given Media component.
==========

to

==========
Update src/device OWNERS files with TEAM / COMPONENTS and removing emeritus.

device-dev@chromium.org now exists and is taking on triage of most of
src/device. This change associates the TEAM in OWNERS files and updates
component labels as well.

The root device/OWNERS sets the TEAM that is used in most subdirectories.

Bluetooth has a specific team list, and has components added. 
Redundant OWNERS files are removed.

mvanouwerkerk@chromium.org is removed, as they now work in other areas.

geolocation does not need a redundant COMPONENT in the interfaces directory.

media_transfer_protocol given Media component.

Review-Url: https://codereview.chromium.org/2789433002
Cr-Commit-Position: refs/heads/master@{#461263}
Committed:
https://chromium.googlesource.com/chromium/src/+/82e698b4a53d8592222d7e641275...
==========

[commit-bot: I haz the power, 2017-03-31 22:57:56]

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

[Lei Zhang, 2017-04-18 02:09:34]

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

[Lei Zhang, 2017-04-18 02:09:34]

https://codereview.chromium.org/2789433002/diff/20001/device/media_transfer_p...
File device/media_transfer_protocol/OWNERS (right):

https://codereview.chromium.org/2789433002/diff/20001/device/media_transfer_p...
device/media_transfer_protocol/OWNERS:4: # COMPONENT: Internals>Media
Not sure why you didn't ask the folks above. There used to be a MediaGalleries
component that was the only user of this code. However, that project went away.
In the meanwhile, ChromeOS started using this code in their FileManager and is
now basically the only user.

A recent MTP bug is https://crbug.com/676968 - maybe Platform>Apps>FileManager
is the better component for this? This isn't directly audio/video related.

[scheib, 2017-04-18 18:19:27]

https://codereview.chromium.org/2789433002/diff/20001/device/media_transfer_p...
File device/media_transfer_protocol/OWNERS (right):

https://codereview.chromium.org/2789433002/diff/20001/device/media_transfer_p...
device/media_transfer_protocol/OWNERS:4: # COMPONENT: Internals>Media
On 2017/04/18 02:09:34, Lei Zhang wrote:
> Not sure why you didn't ask the folks above. There used to be a MediaGalleries
> component that was the only user of this code. However, that project went
away.
> In the meanwhile, ChromeOS started using this code in their FileManager and is
> now basically the only user.
> 
> A recent MTP bug is https://crbug.com/676968 - maybe Platform>Apps>FileManager
> is the better component for this? This isn't directly audio/video related.

Platform>Apps>FileManager SGTM, I'll push a new patch.

[Lei Zhang, 2017-04-18 18:45:08]

On 2017/04/18 18:19:27, scheib wrote:
> Platform>Apps>FileManager SGTM, I'll push a new patch.

Thanks!