Moved gpu factories to media/renderers/

Moved gpu factories to media/renderers/

Since DefaultRendererFactory uses GPU factories, code that instantiates
the default factory need to include GPU factories headers and need to be
able to instantiate them as well, so moving the GPU factories (both the
mock and the real one) to media/renderers/ to make deps easier.

BUG=457959
Committed: https://crrev.com/8b3b39bc38e111503353d5c5afa37b4b9d091d2e
Cr-Commit-Position: refs/heads/master@{#318922}

[servolk, 2015-03-03 01:19:47]

servolk@chromium.org changed reviewers:
+ avi@chromium.org, dalecurtis@chromium.org, darin@chromium.org,
gunsch@chromium.org, jam@chromium.org, jochen@chromium.org, lcwu@chromium.org,
piman@chromium.org, scherkus@chromium.org, wolenetz@chromium.org,
xhwang@chromium.org

[DaleCurtis, 2015-03-03 01:24:12]

You definitely don't need all these OWNERS, please pare it down to only those
required. Otherwise, media/ lgtm; though please let xhwang@ weigh in.

[servolk, 2015-03-03 01:29:28]

On 2015/03/03 01:24:12, DaleCurtis wrote:
> You definitely don't need all these OWNERS, please pare it down to only those
> required. Otherwise, media/ lgtm; though please let xhwang@ weigh in.

Any suggestion on who's the best person to review this? Most of the changes are
under content/renderer/, but content/renderer/OWNERS contains only per-file
entries, so I added content/OWNERS instead.
Last related CL (moving most of the renderer code to media/renderers/) was
approved by jochen@, but he is traveling this week.

[DaleCurtis, 2015-03-03 01:33:38]

You only need a single content/ and media/ OWNER; though in this case I'd like
xhwang@ to review as well; there's no need to add any more than that.

[DaleCurtis, 2015-03-03 01:34:33]

(If there's only per-file OWNERS in a given file, just walk up to the next
directory until you find a general OWNER).

[servolk, 2015-03-03 01:42:44]

servolk@chromium.org changed reviewers:
- darin@chromium.org, gunsch@chromium.org, lcwu@chromium.org,
piman@chromium.org, scherkus@chromium.org, wolenetz@chromium.org

[servolk, 2015-03-03 01:43:07]

servolk@chromium.org changed reviewers:
- avi@chromium.org, jochen@chromium.org

[servolk, 2015-03-03 02:08:03]

servolk@chromium.org changed reviewers:
+ aa@chromium.org

[servolk, 2015-03-03 02:09:06]

On 2015/03/03 01:34:33, DaleCurtis wrote:
> (If there's only per-file OWNERS in a given file, just walk up to the next
> directory until you find a general OWNER).

Ok, I've left only jam@ for content/ for now and added aa@ to take a quick look
at  mojo/services/html_viewer/webmediaplayer_factory.cc

[jam, 2015-03-03 04:33:05]

lgtm

[xhwang, 2015-03-03 17:21:46]

lgtm

[servolk, 2015-03-03 17:52:25]

servolk@chromium.org changed reviewers:
+ jamesr@chromium.org
- aa@chromium.org

[servolk, 2015-03-03 17:52:58]

On 2015/03/03 17:21:46, xhwang wrote:
> lgtm

+jamesr@, could you please take super quick look at mojo/ ?

[jamesr, 2015-03-03 19:02:34]

lgtm

[servolk, 2015-03-03 19:03:44]

The CQ bit was checked by servolk@chromium.org

[servolk, 2015-03-03 19:03:45]

The patchset sent to the CQ was uploaded after l-g-t-m from
dalecurtis@chromium.org
Link to the patchset: https://codereview.chromium.org/970023002/#ps40001 (title:
"Fixed mojo webmediaplayer_factory.cc")

[commit-bot: I haz the power, 2015-03-03 19:04:22]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/970023002/40001

[commit-bot: I haz the power, 2015-03-03 19:08:34]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-03-03 19:09:07]

Patchset 3 (id:??) landed as
https://crrev.com/8b3b39bc38e111503353d5c5afa37b4b9d091d2e
Cr-Commit-Position: refs/heads/master@{#318922}