| Index: src/effects/SkColorFilterImageFilter.cpp
|
| diff --git a/src/effects/SkColorFilterImageFilter.cpp b/src/effects/SkColorFilterImageFilter.cpp
|
| index fdf6de7b76daf755f2fea5d49c87a753374e3cb7..210fb57b98fd6a3fa0534581bee4c25dc05d9a37 100755
|
| --- a/src/effects/SkColorFilterImageFilter.cpp
|
| +++ b/src/effects/SkColorFilterImageFilter.cpp
|
| @@ -21,10 +21,12 @@ SkColorFilterImageFilter* SkColorFilterImageFilter::Create(SkColorFilter* cf,
|
| return NULL;
|
| }
|
|
|
| - SkColorFilter* inputColorFilter;
|
| - if (input && input->asColorFilter(&inputColorFilter)) {
|
| - SkAutoUnref autoUnref(inputColorFilter);
|
| - SkAutoTUnref<SkColorFilter> newCF(cf->newComposed(inputColorFilter));
|
| + SkColorFilter* inputCF;
|
| + if (input && input->isColorFilterNode(&inputCF)) {
|
| + // This is an optimization, as it collapses the hierarchy by just combining the two
|
| + // colorfilters into a single one, which the new imagefilter will wrap.
|
| + SkAutoUnref autoUnref(inputCF);
|
| + SkAutoTUnref<SkColorFilter> newCF(SkColorFilter::CreateComposeFilter(cf, inputCF));
|
| if (newCF) {
|
| return SkNEW_ARGS(SkColorFilterImageFilter, (newCF, input->getInput(0), cropRect, 0));
|
| }
|
| @@ -85,11 +87,11 @@ bool SkColorFilterImageFilter::onFilterImage(Proxy* proxy, const SkBitmap& sourc
|
| return true;
|
| }
|
|
|
| -bool SkColorFilterImageFilter::asColorFilter(SkColorFilter** filter) const {
|
| - if (!cropRectIsSet()) {
|
| +bool SkColorFilterImageFilter::onIsColorFilterNode(SkColorFilter** filter) const {
|
| + SkASSERT(1 == this->countInputs());
|
| + if (!this->cropRectIsSet()) {
|
| if (filter) {
|
| - *filter = fColorFilter;
|
| - fColorFilter->ref();
|
| + *filter = SkRef(fColorFilter);
|
| }
|
| return true;
|
| }
|
|
|