Use AVDA copying strategy unless the unified pipeline flag is set.

Use AVDA copying strategy unless the unified pipeline flag is set.

Cause AVDA to use the copying strategy unless the right #defines
and runtime flags are provided to enable it.  We want to prevent
WebRTC from using the unfinished deferred rendering strategy, unless
the unifed media pipeline runtime flag is present to enable it.

BUG=565561
Committed: https://crrev.com/2ac89ad12d6f0c914c673e57299f69e6d2046941
Cr-Commit-Position: refs/heads/master@{#365596}

[DaleCurtis, 2015-12-15 23:42:56]

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

[DaleCurtis, 2015-12-15 23:42:57]

lgtm % nit -- going to need this to switch to runtime flags.

https://codereview.chromium.org/1515993003/diff/40001/content/common/gpu/medi...
File content/common/gpu/media/android_video_decode_accelerator.cc (right):

https://codereview.chromium.org/1515993003/diff/40001/content/common/gpu/medi...
content/common/gpu/media/android_video_decode_accelerator.cc:696: if
(UseDeferredRenderingStrategy())
{}

[liberato (no reviews please), 2015-12-16 16:09:57]

Description was changed from

==========
Use AVDA copying strategy unless the unified pipeline flag is set.

Cause AVDA to use the copying strategy unless the right #defines
and runtime flags are provided to enable it.  We want to prevent
WebRTC from using the unfinished deferred rendering strategy, unless
the unifed media pipeline runtime flag is present to enable it.

BUG=565561
==========

to

==========
Use AVDA copying strategy unless the unified pipeline flag is set.

Cause AVDA to use the copying strategy unless the right #defines
and runtime flags are provided to enable it.  We want to prevent
WebRTC from using the unfinished deferred rendering strategy, unless
the unifed media pipeline runtime flag is present to enable it.

BUG=565561
==========

[liberato (no reviews please), 2015-12-16 16:09:57]

liberato@chromium.org changed reviewers:
+ kbr@chromium.org

[liberato (no reviews please), 2015-12-16 16:11:05]

kbr@: can you PTAL at gpu_process_host.cc?

thanks
-fl

https://codereview.chromium.org/1515993003/diff/40001/content/common/gpu/medi...
File content/common/gpu/media/android_video_decode_accelerator.cc (right):

https://codereview.chromium.org/1515993003/diff/40001/content/common/gpu/medi...
content/common/gpu/media/android_video_decode_accelerator.cc:696: if
(UseDeferredRenderingStrategy())
On 2015/12/15 23:42:57, DaleCurtis wrote:
> {}

Done.

[Ken Russell (switch to Gerrit), 2015-12-16 18:31:35]

gpu_process_host lgtm

[liberato (no reviews please), 2015-12-16 18:34:24]

The CQ bit was checked by liberato@chromium.org

[liberato (no reviews please), 2015-12-16 18:34:25]

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/1515993003/#ps60001
(title: "cl feedback")

[commit-bot: I haz the power, 2015-12-16 18:34:39]

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

[commit-bot: I haz the power, 2015-12-16 20:00:23]

Description was changed from

==========
Use AVDA copying strategy unless the unified pipeline flag is set.

Cause AVDA to use the copying strategy unless the right #defines
and runtime flags are provided to enable it.  We want to prevent
WebRTC from using the unfinished deferred rendering strategy, unless
the unifed media pipeline runtime flag is present to enable it.

BUG=565561
==========

to

==========
Use AVDA copying strategy unless the unified pipeline flag is set.

Cause AVDA to use the copying strategy unless the right #defines
and runtime flags are provided to enable it.  We want to prevent
WebRTC from using the unfinished deferred rendering strategy, unless
the unifed media pipeline runtime flag is present to enable it.

BUG=565561
==========

[commit-bot: I haz the power, 2015-12-16 20:00:23]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-12-16 20:01:08]

Description was changed from

==========
Use AVDA copying strategy unless the unified pipeline flag is set.

Cause AVDA to use the copying strategy unless the right #defines
and runtime flags are provided to enable it.  We want to prevent
WebRTC from using the unfinished deferred rendering strategy, unless
the unifed media pipeline runtime flag is present to enable it.

BUG=565561
==========

to

==========
Use AVDA copying strategy unless the unified pipeline flag is set.

Cause AVDA to use the copying strategy unless the right #defines
and runtime flags are provided to enable it.  We want to prevent
WebRTC from using the unfinished deferred rendering strategy, unless
the unifed media pipeline runtime flag is present to enable it.

BUG=565561

Committed: https://crrev.com/2ac89ad12d6f0c914c673e57299f69e6d2046941
Cr-Commit-Position: refs/heads/master@{#365596}
==========

[commit-bot: I haz the power, 2015-12-16 20:01:09]

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