Convert child FPs from refs to pending executions when parent converts

Convert child FPs from refs to pending executions when parent converts

Committed: https://skia.googlesource.com/skia/+/4204800cd8f311f11491cf2eb7e32ca681f05489

[bsalomon, 2015-08-27 18:46:47]

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

[bsalomon, 2015-08-27 18:46:48]

This allows GPU resources owned by child processors to be recycled once the
parent processor goes into pending-execution state (and drawing code isn't still
holding direct refs on the child processors).

[bsalomon, 2015-08-27 18:46:54]

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

[commit-bot: I haz the power, 2015-08-27 18:47:06]

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

[commit-bot: I haz the power, 2015-08-27 18:52:07]

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

[commit-bot: I haz the power, 2015-08-27 18:52:07]

Dry run: This issue passed the CQ dry run.

[robertphillips, 2015-08-27 19:39:06]

lgtm + dox request

https://codereview.chromium.org/1315923004/diff/40001/include/gpu/GrProgramEl...
File include/gpu/GrProgramElement.h (right):

https://codereview.chromium.org/1315923004/diff/40001/include/gpu/GrProgramEl...
include/gpu/GrProgramElement.h:24: * when the program element is scheduled for
deferred execution.
// Expand on this comment ?

[robertphillips, 2015-08-27 19:45:23]

I've created https://code.google.com/p/skia/issues/detail?id=4263 to, hopefully,
capture some thoughts on preventing hard refs from intruding into the deferredIO
system.

[bsalomon, 2015-08-27 20:02:49]

On 2015/08/27 19:45:23, robertphillips wrote:
> I've created https://code.google.com/p/skia/issues/detail?id=4263 to,
hopefully,
> capture some thoughts on preventing hard refs from intruding into the
deferredIO
> system.

Thanks, I updated the comments. If you don't mind giving it a read let me know
if it's clearer.

[bsalomon, 2015-08-27 23:04:08]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2015-08-27 23:04:08]

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

[commit-bot: I haz the power, 2015-08-27 23:04:13]

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

[commit-bot: I haz the power, 2015-08-27 23:43:51]

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

[robertphillips, 2015-08-28 12:36:55]

The comment looks good.