Add null check for bmp->tex result in SkImageFilter::filterInputGpu

Add null check for bmp->tex result in SkImageFilter::filterInputGpu

BUG=chromium:570479
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1555333002

Committed: https://skia.googlesource.com/skia/+/45eefcfec26ee48c0b2ee1d316aae3c3dc529145

[bsalomon, 2016-01-05 14:21:02]

Description was changed from

==========
Add null check for bmp->tex result in SkImageFilter::filterInputGpu

BUG=chromium:570479
==========

to

==========
Add null check for bmp->tex result in SkImageFilter::filterInputGpu

BUG=chromium:570479
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[bsalomon, 2016-01-05 14:21:48]

bsalomon@google.com changed reviewers:
+ senorblanco@chromium.org

[bsalomon, 2016-01-05 14:21:56]

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

[commit-bot: I haz the power, 2016-01-05 14:22:05]

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

[commit-bot: I haz the power, 2016-01-05 14:36:49]

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

[commit-bot: I haz the power, 2016-01-05 14:36:49]

Dry run: This issue passed the CQ dry run.

[Stephen White, 2016-01-05 15:07:29]

On 2016/01/05 14:36:49, commit-bot: I haz the power wrote:
> Dry run: This issue passed the CQ dry run.

LGTM, but a unit test would be nice.

[bsalomon, 2016-01-05 15:28:07]

On 2016/01/05 15:07:29, Stephen White wrote:
> On 2016/01/05 14:36:49, commit-bot: I haz the power wrote:
> > Dry run: This issue passed the CQ dry run.
> 
> LGTM, but a unit test would be nice.


Happy to write one but I'm not sure how to trigger this condition. Is it a
filter network where one node does its work in sw and so produces a sw-bitmap
that's too large for a texture?

[Stephen White, 2016-01-05 15:30:10]

On 2016/01/05 15:28:07, bsalomon wrote:
> On 2016/01/05 15:07:29, Stephen White wrote:
> > On 2016/01/05 14:36:49, commit-bot: I haz the power wrote:
> > > Dry run: This issue passed the CQ dry run.
> > 
> > LGTM, but a unit test would be nice.
> 
> 
> Happy to write one but I'm not sure how to trigger this condition. Is it a
> filter network where one node does its work in sw and so produces a sw-bitmap
> that's too large for a texture?

Not actually sure. My guess is that it's just failing to allocate the texture in
the upstream filter node. Having a really large (100k x 100k) source seems to
trigger it.

I've reduced the HTML test case a bit if that helps; will attach it to the bug.

[bsalomon, 2016-01-05 16:02:03]

Thanks! Unit test added. PTAL.

[Stephen White, 2016-01-05 16:10:19]

LGTM

[bsalomon, 2016-01-05 16:21:19]

The CQ bit was checked by bsalomon@google.com

[commit-bot: I haz the power, 2016-01-05 16:21:22]

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

[commit-bot: I haz the power, 2016-01-05 16:39:30]

Description was changed from

==========
Add null check for bmp->tex result in SkImageFilter::filterInputGpu

BUG=chromium:570479
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Add null check for bmp->tex result in SkImageFilter::filterInputGpu

BUG=chromium:570479
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/45eefcfec26ee48c0b2ee1d316aae3c3dc529145
==========

[commit-bot: I haz the power, 2016-01-05 16:39:30]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/45eefcfec26ee48c0b2ee1d316aae3c3dc529145