Optionally pass rendertarget to getTestTarget

Optionally pass rendertarget to getTestTarget

This shouldn't really make any difference but allocating and holding on to a GrRenderTarget for each test target generates image differences for Mali GPUs. This CL allows an existing render target to be used for the test target.

TBR=bsalomon@google.com

Committed: https://skia.googlesource.com/skia/+/504ce5dc772e14be2f5697b00ac82fcf82127763

[robertphillips, 2015-11-16 18:45:56]

Description was changed from

==========
Optionally pass rendertarget to getTestTarget
==========

to

==========
Optionally pass rendertarget to getTestTarget

This shouldn't really make any difference but allocating and holding on to a
GrRenderTarget for each test target generates image differences for Mali GPUs.
This CL allows an existing render target to be used for the test target.
==========

[robertphillips, 2015-11-16 18:45:56]

robertphillips@google.com changed reviewers:
+ joshualitt@chromium.org, joshualitt@google.com

[joshualitt, 2015-11-16 18:47:54]

On 2015/11/16 18:46:43, robertphillips wrote:

lgtm

[robertphillips, 2015-11-16 18:48:29]

The CQ bit was checked by robertphillips@google.com to run a CQ dry run

[commit-bot: I haz the power, 2015-11-16 18:48:46]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1447113002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1447113002/1

[robertphillips, 2015-11-16 18:57:48]

Description was changed from

==========
Optionally pass rendertarget to getTestTarget

This shouldn't really make any difference but allocating and holding on to a
GrRenderTarget for each test target generates image differences for Mali GPUs.
This CL allows an existing render target to be used for the test target.
==========

to

==========
Optionally pass rendertarget to getTestTarget

This shouldn't really make any difference but allocating and holding on to a
GrRenderTarget for each test target generates image differences for Mali GPUs.
This CL allows an existing render target to be used for the test target.

TBR=bsalomon@google.com
==========

[robertphillips, 2015-11-16 18:57:48]

robertphillips@google.com changed reviewers:
+ bsalomon@google.com

[commit-bot: I haz the power, 2015-11-16 19:01:00]

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

[commit-bot: I haz the power, 2015-11-16 19:01:01]

Dry run: This issue passed the CQ dry run.

[robertphillips, 2015-11-16 19:01:26]

The CQ bit was checked by robertphillips@google.com

[commit-bot: I haz the power, 2015-11-16 19:01:35]

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

[commit-bot: I haz the power, 2015-11-16 19:02:08]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/504ce5dc772e14be2f5697b00ac82fcf82127763

[robertphillips, 2015-11-17 17:31:57]

Patchset #2 (id:20001) has been deleted