CmaRenderer is dead. Long live MojoRenderer.

CmaRenderer is dead. Long live MojoRenderer.

BUG=571155
Committed: https://crrev.com/60fb6b0f601074d94dc03bbea9f1351f678f18fa
Cr-Commit-Position: refs/heads/master@{#416361}

[alokp, 2016-09-01 21:06:15]

alokp@chromium.org changed reviewers:
+ halliwell@chromium.org, servolk@chromium.org, slan@chromium.org

[alokp, 2016-09-01 21:06:16]

This will land after:
https://codereview.chromium.org/2279863003/

[DaleCurtis, 2016-09-01 21:08:32]

\o/

[servolk, 2016-09-01 22:13:25]

On 2016/09/01 21:06:16, alokp wrote:
> This will land after:
> https://codereview.chromium.org/2279863003/

lgtm, thanks!

[halliwell, 2016-09-01 22:38:57]

On 2016/09/01 22:13:25, servolk wrote:
> On 2016/09/01 21:06:16, alokp wrote:
> > This will land after:
> > https://codereview.chromium.org/2279863003/
> 
> lgtm, thanks!

Awesome, lgtm.

After landing this, it's worth looking at whether any further simplifications
are possible.  e.g. I have a recollection that the DecoderBuffer class hierarchy
has complexity that's there purely for the DecoderBufferFromMsg subclass?

[alokp, 2016-09-02 02:25:01]

The CQ bit was checked by alokp@chromium.org

[commit-bot: I haz the power, 2016-09-02 02:25:13]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[alokp, 2016-09-02 02:26:59]

alokp@chromium.org changed reviewers:
+ dpranke@chromium.org

[alokp, 2016-09-02 02:27:00]

dpranke@: testing/buildbot/

[commit-bot: I haz the power, 2016-09-02 02:32:15]

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

[commit-bot: I haz the power, 2016-09-02 02:32:17]

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

[Dirk Pranke, 2016-09-02 16:55:40]

lgtm

[slan, 2016-09-02 16:57:13]

lgtm - woohoo!

https://codereview.chromium.org/2300993003/diff/1/chromecast/browser/cast_bro...
File chromecast/browser/cast_browser_main_parts.cc (right):

https://codereview.chromium.org/2300993003/diff/1/chromecast/browser/cast_bro...
chromecast/browser/cast_browser_main_parts.cc:234: #elif
defined(ARCH_CPU_ARM_FAMILY)
Unrelated, but how does this work on MIPS?

[alokp, 2016-09-02 17:04:15]

https://codereview.chromium.org/2300993003/diff/1/chromecast/browser/cast_bro...
File chromecast/browser/cast_browser_main_parts.cc (right):

https://codereview.chromium.org/2300993003/diff/1/chromecast/browser/cast_bro...
chromecast/browser/cast_browser_main_parts.cc:234: #elif
defined(ARCH_CPU_ARM_FAMILY)
On 2016/09/02 16:57:13, slan wrote:
> Unrelated, but how does this work on MIPS?

I am not sure what you mean? I just deleted the command-line flag.

[alokp, 2016-09-02 17:04:24]

The CQ bit was checked by alokp@chromium.org

[commit-bot: I haz the power, 2016-09-02 17:04:44]

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-02 17:11:49]

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

[commit-bot: I haz the power, 2016-09-02 17:11:51]

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

[alokp, 2016-09-02 17:21:33]

The CQ bit was checked by alokp@chromium.org

[alokp, 2016-09-02 17:21:33]

The patchset sent to the CQ was uploaded after l-g-t-m from
dpranke@chromium.org, servolk@chromium.org, halliwell@chromium.org,
slan@chromium.org
Link to the patchset: https://codereview.chromium.org/2300993003/#ps60001
(title: "update OWNERS file")

[commit-bot: I haz the power, 2016-09-02 17:21:54]

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-02 19:38:04]

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

[commit-bot: I haz the power, 2016-09-02 19:38:05]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[alokp, 2016-09-02 20:46:31]

The CQ bit was checked by alokp@chromium.org

[commit-bot: I haz the power, 2016-09-02 20:47: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, 2016-09-02 22:08:33]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-09-02 22:11:55]

Description was changed from

==========
CmaRenderer is dead. Long live MojoRenderer.

BUG=571155
==========

to

==========
CmaRenderer is dead. Long live MojoRenderer.

BUG=571155

Committed: https://crrev.com/60fb6b0f601074d94dc03bbea9f1351f678f18fa
Cr-Commit-Position: refs/heads/master@{#416361}
==========

[commit-bot: I haz the power, 2016-09-02 22:11:57]

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