Update testing frameworks/tests for MDB

Update testing frameworks/tests for MDB

This CL:

Fixes an ordering problem w.r.t. drawTarget clean up in GrContext::abandonContext (for text test that abandons context)
Fixes when the lastDrawTarget field is set on a RenderTarget (now in GrDrawTarget ctor) due to GrTestTarget use case
Updates the ProgramUnitTest to use multiple drawTargets
Adds renderTarget creation to GrTestTargets (in MDB drawTargets require them)

BUG=skia:4094
Committed: https://skia.googlesource.com/skia/+/9c8605144a0f15e3e69a4e1dcd5d3e63f339380e

Committed: https://skia.googlesource.com/skia/+/0dfa62c97707ac9b0fc0bdba7acfc8405db5c836

[robertphillips, 2015-11-12 15:43:21]

Description was changed from

==========
Update testing frameworks/tests for MDB

BUG=skia:4094
==========

to

==========
Update testing frameworks/tests for MDB

This CL:

Fixes an ordering problem w.r.t. drawTarget clean up in
GrContext::abandonContext (for text test that abandons context)
Fixes when the lastDrawTarget field is set on a RenderTarget (now in
GrDrawTarget ctor) due to GrTestTarget use case
Updates the ProgramUnitTest to use multiple drawTargets
Adds renderTarget creation to GrTestTargets (in MDB drawTargets require them)

BUG=skia:4094
==========

[robertphillips, 2015-11-12 15:43:21]

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

[joshualitt, 2015-11-12 20:48:54]

joshualitt@google.com changed reviewers:
+ joshualitt@google.com

[joshualitt, 2015-11-12 20:48:55]

lgtm, minor question

https://codereview.chromium.org/1441533003/diff/40001/src/gpu/GrContext.cpp
File src/gpu/GrContext.cpp (right):

https://codereview.chromium.org/1441533003/diff/40001/src/gpu/GrContext.cpp#n...
src/gpu/GrContext.cpp:122: fDrawingManager->abandon();
if moving this is intentional, maybe leave a comment?

[robertphillips, 2015-11-13 12:41:40]

https://codereview.chromium.org/1441533003/diff/40001/src/gpu/GrContext.cpp
File src/gpu/GrContext.cpp (right):

https://codereview.chromium.org/1441533003/diff/40001/src/gpu/GrContext.cpp#n...
src/gpu/GrContext.cpp:122: fDrawingManager->abandon();
On 2015/11/12 20:48:55, joshualitt wrote:
> if moving this is intentional, maybe leave a comment?

Done.

[robertphillips, 2015-11-13 12:41:49]

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

[commit-bot: I haz the power, 2015-11-13 12:41:58]

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

[commit-bot: I haz the power, 2015-11-13 12:53:10]

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

[commit-bot: I haz the power, 2015-11-13 12:53:10]

Dry run: This issue passed the CQ dry run.

[robertphillips, 2015-11-13 14:05:22]

The CQ bit was checked by robertphillips@google.com

[robertphillips, 2015-11-13 14:05:23]

The patchset sent to the CQ was uploaded after l-g-t-m from
joshualitt@google.com
Link to the patchset: https://codereview.chromium.org/1441533003/#ps60001
(title: "Added comment")

[commit-bot: I haz the power, 2015-11-13 14:05:29]

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

[commit-bot: I haz the power, 2015-11-13 14:06:07]

Committed patchset #4 (id:60001) as
https://skia.googlesource.com/skia/+/9c8605144a0f15e3e69a4e1dcd5d3e63f339380e

[egdaniel, 2015-11-13 21:42:31]

A revert of this CL (patchset #4 id:60001) has been created in
https://codereview.chromium.org/1450513002/ by egdaniel@google.com.

The reason for reverting is: Speculative revert for leaks based on ASAN bot (and
some local valgrind testing).

[robertphillips, 2015-11-16 13:16:50]

Description was changed from

==========
Update testing frameworks/tests for MDB

This CL:

Fixes an ordering problem w.r.t. drawTarget clean up in
GrContext::abandonContext (for text test that abandons context)
Fixes when the lastDrawTarget field is set on a RenderTarget (now in
GrDrawTarget ctor) due to GrTestTarget use case
Updates the ProgramUnitTest to use multiple drawTargets
Adds renderTarget creation to GrTestTargets (in MDB drawTargets require them)

BUG=skia:4094

Committed:
https://skia.googlesource.com/skia/+/9c8605144a0f15e3e69a4e1dcd5d3e63f339380e
==========

to

==========
Update testing frameworks/tests for MDB

This CL:

Fixes an ordering problem w.r.t. drawTarget clean up in
GrContext::abandonContext (for text test that abandons context)
Fixes when the lastDrawTarget field is set on a RenderTarget (now in
GrDrawTarget ctor) due to GrTestTarget use case
Updates the ProgramUnitTest to use multiple drawTargets
Adds renderTarget creation to GrTestTargets (in MDB drawTargets require them)

BUG=skia:4094

Committed:
https://skia.googlesource.com/skia/+/9c8605144a0f15e3e69a4e1dcd5d3e63f339380e
==========

[robertphillips, 2015-11-16 14:10:52]

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

[commit-bot: I haz the power, 2015-11-16 14:11:04]

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

[commit-bot: I haz the power, 2015-11-16 14:22:34]

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

[commit-bot: I haz the power, 2015-11-16 14:22:35]

Dry run: This issue passed the CQ dry run.

[robertphillips, 2015-11-16 14:22:48]

The CQ bit was checked by robertphillips@google.com

[robertphillips, 2015-11-16 14:22:49]

The patchset sent to the CQ was uploaded after l-g-t-m from
joshualitt@google.com
Link to the patchset: https://codereview.chromium.org/1441533003/#ps80001
(title: "Fix ref counting bug")

[commit-bot: I haz the power, 2015-11-16 14:22:56]

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

[commit-bot: I haz the power, 2015-11-16 14:23:35]

Committed patchset #5 (id:80001) as
https://skia.googlesource.com/skia/+/0dfa62c97707ac9b0fc0bdba7acfc8405db5c836