Cleanup: move WebRTC related files from chrome/browser/media to chrome/browser/media/webrtc/

Cleanup: move WebRTC related files from chrome/browser/media to chrome/browser/media/webrtc/

The WebRTC related files under chrome/browser/media are
isolated from "the rest" of the folder, i.e. do not include
other files and "the rest" don't include webrtc-related
files; there's 130 files in media/ folder, of which 85
are webrtc-specific, so this CL extracts these WebRTC files
(i.e. having to do with Media Stream, WebRTC, Desktop/Tab
capture and capture permissions) into its own folder.

No new code whatsoever. Splitted OWNERS and DEPS.

TBR= stevenjb@chromium.org for
for chrome/browser/ui/webui/options/media_devices_selection_handler.h
that has a noparent in the OWNERS file, and
anyway says : "# This UI is deprecated."

Committed: https://crrev.com/a2023abe2a2932c1bc9ae39106f983f292980242
Cr-Commit-Position: refs/heads/master@{#417136}

[mcasas, 2016-09-02 16:43:19]

Patchset #1 (id:1) has been deleted

[mcasas, 2016-09-02 16:47:02]

Description was changed from

==========
Cleanup: move WebRTC related files from chrome/browser/media to
chrome/browser/webrtc/

BUG=
==========

to

==========
Cleanup: move WebRTC related files from chrome/browser/media to
chrome/browser/webrtc/

The WebRTC related files under chrome/browser/media are
isolated from "the rest" of the folder, i.e. do not include
other files and "the rest" don't include webrtc-related
files, and there's 133 files in that folder, so this CL
extracts the WebRTC (Media Stream, WebRTC, Desktop/Tab
capture and capture permissions) into its own folder.


No new code whatsoever. Splitted OWNERS and DEPS.
==========

[mcasas, 2016-09-02 16:48:25]

Description was changed from

==========
Cleanup: move WebRTC related files from chrome/browser/media to
chrome/browser/webrtc/

The WebRTC related files under chrome/browser/media are
isolated from "the rest" of the folder, i.e. do not include
other files and "the rest" don't include webrtc-related
files, and there's 133 files in that folder, so this CL
extracts the WebRTC (Media Stream, WebRTC, Desktop/Tab
capture and capture permissions) into its own folder.


No new code whatsoever. Splitted OWNERS and DEPS.
==========

to

==========
Cleanup: move WebRTC related files from chrome/browser/media to
chrome/browser/webrtc/

The WebRTC related files under chrome/browser/media are
isolated from "the rest" of the folder, i.e. do not include
other files and "the rest" don't include webrtc-related
files; there's 130 files in media/ folder, of which 81
are webrtc-specific, so this CL extracts these WebRTC files
(i.e. having to do with Media Stream, WebRTC, Desktop/Tab
capture and capture permissions) into its own folder.


No new code whatsoever. Splitted OWNERS and DEPS.
==========

[mcasas, 2016-09-02 16:49:07]

mcasas@chromium.org changed reviewers:
+ dalecurtis@chromium.org, tommi@chromium.org

[mcasas, 2016-09-02 16:49:09]

tommi@, dalecurtis@ Owners RS/PTAL

[mcasas, 2016-09-02 21:55:17]

Patchset #1 (id:20001) has been deleted

[mcasas, 2016-09-02 21:55:40]

Patchset #1 (id:40001) has been deleted

[mcasas, 2016-09-02 21:55:47]

Description was changed from

==========
Cleanup: move WebRTC related files from chrome/browser/media to
chrome/browser/webrtc/

The WebRTC related files under chrome/browser/media are
isolated from "the rest" of the folder, i.e. do not include
other files and "the rest" don't include webrtc-related
files; there's 130 files in media/ folder, of which 81
are webrtc-specific, so this CL extracts these WebRTC files
(i.e. having to do with Media Stream, WebRTC, Desktop/Tab
capture and capture permissions) into its own folder.


No new code whatsoever. Splitted OWNERS and DEPS.
==========

to

==========
Cleanup: move WebRTC related files from chrome/browser/media to
chrome/browser/webrtc/

The WebRTC related files under chrome/browser/media are
isolated from "the rest" of the folder, i.e. do not include
other files and "the rest" don't include webrtc-related
files; there's 130 files in media/ folder, of which 83
are webrtc-specific, so this CL extracts these WebRTC files
(i.e. having to do with Media Stream, WebRTC, Desktop/Tab
capture and capture permissions) into its own folder.


No new code whatsoever. Splitted OWNERS and DEPS.
==========

[tommi (sloooow) - chröme, 2016-09-05 13:19:02]

content/renderer/media has a 'webrtc' folder.  Would
content/browser/media/webrtc then make sense or do you prefer
content/browser/webrtc?

[mcasas, 2016-09-06 15:06:00]

Description was changed from

==========
Cleanup: move WebRTC related files from chrome/browser/media to
chrome/browser/webrtc/

The WebRTC related files under chrome/browser/media are
isolated from "the rest" of the folder, i.e. do not include
other files and "the rest" don't include webrtc-related
files; there's 130 files in media/ folder, of which 83
are webrtc-specific, so this CL extracts these WebRTC files
(i.e. having to do with Media Stream, WebRTC, Desktop/Tab
capture and capture permissions) into its own folder.


No new code whatsoever. Splitted OWNERS and DEPS.
==========

to

==========
Cleanup: move WebRTC related files from chrome/browser/media to
chrome/browser/media/webrtc/

The WebRTC related files under chrome/browser/media are
isolated from "the rest" of the folder, i.e. do not include
other files and "the rest" don't include webrtc-related
files; there's 130 files in media/ folder, of which 83
are webrtc-specific, so this CL extracts these WebRTC files
(i.e. having to do with Media Stream, WebRTC, Desktop/Tab
capture and capture permissions) into its own folder.


No new code whatsoever. Splitted OWNERS and DEPS.
==========

[mcasas, 2016-09-06 18:45:25]

Patchset #3 (id:100001) has been deleted

[mcasas, 2016-09-06 18:47:13]

Patchset #3 (id:110001) has been deleted

[mcasas, 2016-09-06 23:13:59]

Patchset #3 (id:120001) has been deleted

[mcasas, 2016-09-06 23:14:31]

Patchset #3 (id:130001) has been deleted

[mcasas, 2016-09-06 23:15:07]

Description was changed from

==========
Cleanup: move WebRTC related files from chrome/browser/media to
chrome/browser/media/webrtc/

The WebRTC related files under chrome/browser/media are
isolated from "the rest" of the folder, i.e. do not include
other files and "the rest" don't include webrtc-related
files; there's 130 files in media/ folder, of which 83
are webrtc-specific, so this CL extracts these WebRTC files
(i.e. having to do with Media Stream, WebRTC, Desktop/Tab
capture and capture permissions) into its own folder.


No new code whatsoever. Splitted OWNERS and DEPS.
==========

to

==========
Cleanup: move WebRTC related files from chrome/browser/media to
chrome/browser/media/webrtc/

The WebRTC related files under chrome/browser/media are
isolated from "the rest" of the folder, i.e. do not include
other files and "the rest" don't include webrtc-related
files; there's 130 files in media/ folder, of which 85
are webrtc-specific, so this CL extracts these WebRTC files
(i.e. having to do with Media Stream, WebRTC, Desktop/Tab
capture and capture permissions) into its own folder.


No new code whatsoever. Splitted OWNERS and DEPS.
==========

[mcasas, 2016-09-06 23:16:57]

On 2016/09/05 13:19:02, tommi (chrömium) wrote:
> content/renderer/media has a 'webrtc' folder.  Would
> content/browser/media/webrtc then make sense or do you prefer
> content/browser/webrtc?

Insofar as media/ OWNERS tend to be good fallbacks for
webrtc reviews/rubberstamping, I think your first option,
chrome/browser/media/webrtc it's better; done in PS2 and PS3, 
PTAL.

[mcasas, 2016-09-06 23:29:57]

Patchset #3 (id:140001) has been deleted

[mcasas, 2016-09-07 17:22:19]

On 2016/09/06 23:16:57, mcasas wrote:
> On 2016/09/05 13:19:02, tommi (chrömium) wrote:
> > content/renderer/media has a 'webrtc' folder.  Would
> > content/browser/media/webrtc then make sense or do you prefer
> > content/browser/webrtc?
> 
> Insofar as media/ OWNERS tend to be good fallbacks for
> webrtc reviews/rubberstamping, I think your first option,
> chrome/browser/media/webrtc it's better; done in PS2 and PS3, 
> PTAL.

ping guys.

[tommi (sloooow) - chröme, 2016-09-07 17:41:47]

On 2016/09/07 17:22:19, mcasas wrote:
> On 2016/09/06 23:16:57, mcasas wrote:
> > On 2016/09/05 13:19:02, tommi (chrömium) wrote:
> > > content/renderer/media has a 'webrtc' folder.  Would
> > > content/browser/media/webrtc then make sense or do you prefer
> > > content/browser/webrtc?
> > 
> > Insofar as media/ OWNERS tend to be good fallbacks for
> > webrtc reviews/rubberstamping, I think your first option,
> > chrome/browser/media/webrtc it's better; done in PS2 and PS3, 
> > PTAL.
> 
> ping guys.

lgtm

[mcasas, 2016-09-07 17:47:17]

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

[mcasas, 2016-09-07 17:47:18]

sky@ RS plz  

(moving files in a subfolder for better
browsing and mental tidiness)

[sky, 2016-09-07 19:33:55]

https://codereview.chromium.org/2307083002/diff/150001/chrome/chrome_tests_un...
File chrome/chrome_tests_unit.gypi (right):

https://codereview.chromium.org/2307083002/diff/150001/chrome/chrome_tests_un...
chrome/chrome_tests_unit.gypi:1: # Copyright 2012 The Chromium Authors. All
rights reserved.
Why are you adding this file?

[mcasas, 2016-09-07 21:15:36]

On 2016/09/07 19:33:55, sky wrote:
>
https://codereview.chromium.org/2307083002/diff/150001/chrome/chrome_tests_un...
> File chrome/chrome_tests_unit.gypi (right):
> 
>
https://codereview.chromium.org/2307083002/diff/150001/chrome/chrome_tests_un...
> chrome/chrome_tests_unit.gypi:1: # Copyright 2012 The Chromium Authors. All
> rights reserved.
> Why are you adding this file?

Argh, it got back from the dead during the rebases, 
removed it now.

[sky, 2016-09-07 21:54:15]

RS LGTM

[mcasas, 2016-09-07 22:04:41]

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

[commit-bot: I haz the power, 2016-09-07 22:05:08]

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, 2016-09-07 23:24:40]

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

[commit-bot: I haz the power, 2016-09-07 23:24:42]

Dry run: This issue passed the CQ dry run.

[mcasas, 2016-09-08 00:07:26]

The CQ bit was checked by mcasas@chromium.org

[mcasas, 2016-09-08 00:07:27]

The patchset sent to the CQ was uploaded after l-g-t-m from tommi@chromium.org
Link to the patchset: https://codereview.chromium.org/2307083002/#ps160001
(title: "Removed file wrongly resuscitated during rebase")

[commit-bot: I haz the power, 2016-09-08 00:07:43]

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, 2016-09-08 00:14:13]

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

[commit-bot: I haz the power, 2016-09-08 00:14: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...)

[mcasas, 2016-09-08 00:37:15]

The CQ bit was checked by mcasas@chromium.org

[commit-bot: I haz the power, 2016-09-08 00:37:41]

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, 2016-09-08 00:43:29]

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

[commit-bot: I haz the power, 2016-09-08 00:43:32]

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

[mcasas, 2016-09-08 00:55:38]

Description was changed from

==========
Cleanup: move WebRTC related files from chrome/browser/media to
chrome/browser/media/webrtc/

The WebRTC related files under chrome/browser/media are
isolated from "the rest" of the folder, i.e. do not include
other files and "the rest" don't include webrtc-related
files; there's 130 files in media/ folder, of which 85
are webrtc-specific, so this CL extracts these WebRTC files
(i.e. having to do with Media Stream, WebRTC, Desktop/Tab
capture and capture permissions) into its own folder.


No new code whatsoever. Splitted OWNERS and DEPS.
==========

to

==========
Cleanup: move WebRTC related files from chrome/browser/media to
chrome/browser/media/webrtc/

The WebRTC related files under chrome/browser/media are
isolated from "the rest" of the folder, i.e. do not include
other files and "the rest" don't include webrtc-related
files; there's 130 files in media/ folder, of which 85
are webrtc-specific, so this CL extracts these WebRTC files
(i.e. having to do with Media Stream, WebRTC, Desktop/Tab
capture and capture permissions) into its own folder.

No new code whatsoever. Splitted OWNERS and DEPS.


TBR= stevenjb@chromium.org for
for chrome/browser/ui/webui/options/media_devices_selection_handler.h
that has a noparent in the OWNERS file, and
anyway says : "# This UI is deprecated."
==========

[mcasas, 2016-09-08 00:57:47]

The CQ bit was checked by mcasas@chromium.org

[commit-bot: I haz the power, 2016-09-08 00:58:15]

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, 2016-09-08 01:07:15]

Description was changed from

==========
Cleanup: move WebRTC related files from chrome/browser/media to
chrome/browser/media/webrtc/

The WebRTC related files under chrome/browser/media are
isolated from "the rest" of the folder, i.e. do not include
other files and "the rest" don't include webrtc-related
files; there's 130 files in media/ folder, of which 85
are webrtc-specific, so this CL extracts these WebRTC files
(i.e. having to do with Media Stream, WebRTC, Desktop/Tab
capture and capture permissions) into its own folder.

No new code whatsoever. Splitted OWNERS and DEPS.


TBR= stevenjb@chromium.org for
for chrome/browser/ui/webui/options/media_devices_selection_handler.h
that has a noparent in the OWNERS file, and
anyway says : "# This UI is deprecated."
==========

to

==========
Cleanup: move WebRTC related files from chrome/browser/media to
chrome/browser/media/webrtc/

The WebRTC related files under chrome/browser/media are
isolated from "the rest" of the folder, i.e. do not include
other files and "the rest" don't include webrtc-related
files; there's 130 files in media/ folder, of which 85
are webrtc-specific, so this CL extracts these WebRTC files
(i.e. having to do with Media Stream, WebRTC, Desktop/Tab
capture and capture permissions) into its own folder.

No new code whatsoever. Splitted OWNERS and DEPS.


TBR= stevenjb@chromium.org for
for chrome/browser/ui/webui/options/media_devices_selection_handler.h
that has a noparent in the OWNERS file, and
anyway says : "# This UI is deprecated."
==========

[commit-bot: I haz the power, 2016-09-08 01:07:18]

Committed patchset #4 (id:160001)

[commit-bot: I haz the power, 2016-09-08 01:08:45]

Description was changed from

==========
Cleanup: move WebRTC related files from chrome/browser/media to
chrome/browser/media/webrtc/

The WebRTC related files under chrome/browser/media are
isolated from "the rest" of the folder, i.e. do not include
other files and "the rest" don't include webrtc-related
files; there's 130 files in media/ folder, of which 85
are webrtc-specific, so this CL extracts these WebRTC files
(i.e. having to do with Media Stream, WebRTC, Desktop/Tab
capture and capture permissions) into its own folder.

No new code whatsoever. Splitted OWNERS and DEPS.


TBR= stevenjb@chromium.org for
for chrome/browser/ui/webui/options/media_devices_selection_handler.h
that has a noparent in the OWNERS file, and
anyway says : "# This UI is deprecated."
==========

to

==========
Cleanup: move WebRTC related files from chrome/browser/media to
chrome/browser/media/webrtc/

The WebRTC related files under chrome/browser/media are
isolated from "the rest" of the folder, i.e. do not include
other files and "the rest" don't include webrtc-related
files; there's 130 files in media/ folder, of which 85
are webrtc-specific, so this CL extracts these WebRTC files
(i.e. having to do with Media Stream, WebRTC, Desktop/Tab
capture and capture permissions) into its own folder.

No new code whatsoever. Splitted OWNERS and DEPS.

TBR= stevenjb@chromium.org for
for chrome/browser/ui/webui/options/media_devices_selection_handler.h
that has a noparent in the OWNERS file, and
anyway says : "# This UI is deprecated."

Committed: https://crrev.com/a2023abe2a2932c1bc9ae39106f983f292980242
Cr-Commit-Position: refs/heads/master@{#417136}
==========

[commit-bot: I haz the power, 2016-09-08 01:08:48]

Patchset 4 (id:??) landed as
https://crrev.com/a2023abe2a2932c1bc9ae39106f983f292980242
Cr-Commit-Position: refs/heads/master@{#417136}