Added tree structure to GrGLFragmentProcessor, i.e. GL instances

When getGLInstance is called on a frag proc, the resulting GrGLFragmentProcessor will be the root of a tree of GrGLFragmentProcessors that mirrors the GrFragmentProcessor's tree. This allows setData() to be called recursively (removing the responsibility from compose shader) and allows gl instances direct access to their children gl instances so they can emit their code.

BUG=skia:4182
Committed: https://skia.googlesource.com/skia/+/b1daa86732fe70aa4630c89d75ff0fd619d77c77

[wangyix, 2015-08-14 21:21:45]

wangyix@google.com changed reviewers:
+ bsalomon@google.com, egdaniel@google.com, joshualitt@google.com,
tomhudson@google.com

[wangyix, 2015-08-14 21:21:45]

I tried to split up the non-interesting changes that affected many files (i.e.
refactoring) and the interesting changes into different patchsets.  Patchsets 1,
3, and 5 contain the important changes.

[joshualitt, 2015-08-17 18:05:18]

On 2015/08/14 21:21:45, wangyix wrote:
> I tried to split up the non-interesting changes that affected many files (i.e.
> refactoring) and the interesting changes into different patchsets.  Patchsets
1,
> 3, and 5 contain the important changes.

looks fine to me, Brian should review

[bsalomon, 2015-08-17 18:18:26]

https://codereview.chromium.org/1287023009/diff/100001/src/effects/SkAlphaThr...
File src/effects/SkAlphaThresholdFilter.cpp (right):

https://codereview.chromium.org/1287023009/diff/100001/src/effects/SkAlphaThr...
src/effects/SkAlphaThresholdFilter.cpp:134: void onSetData(const
GrGLProgramDataManager&, const GrProcessor&) override;
can this be private? Do we ever want a subclass to call it?

[wangyix, 2015-08-18 17:33:16]

https://codereview.chromium.org/1287023009/diff/100001/src/effects/SkAlphaThr...
File src/effects/SkAlphaThresholdFilter.cpp (right):

https://codereview.chromium.org/1287023009/diff/100001/src/effects/SkAlphaThr...
src/effects/SkAlphaThresholdFilter.cpp:134: void onSetData(const
GrGLProgramDataManager&, const GrProcessor&) override;
On 2015/08/17 18:18:26, bsalomon wrote:
> can this be private? Do we ever want a subclass to call it?

SkPerlinNoiseShader::onSetData calls INHERITED::onSetData, where INHERITED is
GrGLFragmentProcessor.

Some other subclasses of a subclass of GrGLFragmentProcessor also call
INHERITED::onSetData, specifically, the subclasses of GrGLLightingEffect and the
subclasses of GrGLGradientEffect.

[wangyix, 2015-08-18 17:42:48]

Patchset #6 (id:100001) has been deleted

[bsalomon, 2015-08-18 18:15:20]

lgtm

[wangyix, 2015-08-18 18:17:03]

The CQ bit was checked by wangyix@google.com

[commit-bot: I haz the power, 2015-08-18 18:17:14]

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

[commit-bot: I haz the power, 2015-08-18 18:29:35]

Committed patchset #6 (id:120001) as
https://skia.googlesource.com/skia/+/b1daa86732fe70aa4630c89d75ff0fd619d77c77