Pass mojo::MediaRendererPtr in MojoRendererImpl's constructor.

Pass mojo::MediaRendererPtr in MojoRendererImpl's constructor.

BUG=416570
TEST=Existing tests pass.

Committed: https://crrev.com/8d5642a7521dc0440435ede181f87ff95dd9c97d
Cr-Commit-Position: refs/heads/master@{#307121}

[xhwang, 2014-12-05 22:32:57]

xhwang@chromium.org changed reviewers:
+ jamesr@chromium.org

[xhwang, 2014-12-05 22:32:57]

A simple CL to address a comment we discussed before.

PTAL, thanks!

[jamesr, 2014-12-05 23:12:31]

lgtm

https://codereview.chromium.org/785643002/diff/1/mojo/services/html_viewer/we...
File mojo/services/html_viewer/webmediaplayer_factory.cc (right):

https://codereview.chromium.org/785643002/diff/1/mojo/services/html_viewer/we...
mojo/services/html_viewer/webmediaplayer_factory.cc:64: mojo::MediaRendererPtr
mojo_media_renderer;
we're in namespace mojo here, so you can drop the "mojo::" from this line and
the next

[xhwang, 2014-12-05 23:31:00]

https://codereview.chromium.org/785643002/diff/1/mojo/services/html_viewer/we...
File mojo/services/html_viewer/webmediaplayer_factory.cc (right):

https://codereview.chromium.org/785643002/diff/1/mojo/services/html_viewer/we...
mojo/services/html_viewer/webmediaplayer_factory.cc:64: mojo::MediaRendererPtr
mojo_media_renderer;
On 2014/12/05 23:12:31, jamesr wrote:
> we're in namespace mojo here, so you can drop the "mojo::" from this line and
> the next

Done.

[xhwang, 2014-12-05 23:31:16]

The CQ bit was checked by xhwang@chromium.org

[commit-bot: I haz the power, 2014-12-05 23:31:36]

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

[commit-bot: I haz the power, 2014-12-05 23:42:09]

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

[commit-bot: I haz the power, 2014-12-05 23:42:10]

Try jobs failed on following builders:
  linux_chromium_gn_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[xhwang, 2014-12-05 23:49:57]

The CQ bit was checked by xhwang@chromium.org

[commit-bot: I haz the power, 2014-12-05 23:51:15]

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

[commit-bot: I haz the power, 2014-12-06 00:29:14]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2014-12-06 00:30:20]

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