Simplify SkGpuBlurUtils::GaussianBlur method

Simplify SkGpuBlurUtils::GaussianBlur method

No one was using the canClobberSrc capability and moving the direct filtering case forward makes the rest of the logic simpler.

Split out of: https://codereview.chromium.org/1959493002/ (Retract GrRenderTarget from SkGpuBlurUtils)

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

Committed: https://skia.googlesource.com/skia/+/56a85e69a8d034e0fdee00e8207cda0a9da06fee

[robertphillips, 2016-05-06 13:05:51]

Description was changed from

==========
Simplify SkGpuBlurUtils::GaussianBlur method
==========

to

==========
Simplify SkGpuBlurUtils::GaussianBlur method
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[robertphillips, 2016-05-06 13:08:07]

Description was changed from

==========
Simplify SkGpuBlurUtils::GaussianBlur method
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Simplify SkGpuBlurUtils::GaussianBlur method

No one was using the canClobberSrc capability and moving the direct filtering
case forward makes the rest of the logic simpler.

Split out of: const SkSurfaceProps* surfaceProps = nullptr (Retract
GrRenderTarget from SkGpuBlurUtils)

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

[robertphillips, 2016-05-06 13:08:07]

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

[robertphillips, 2016-05-06 13:08:33]

Description was changed from

==========
Simplify SkGpuBlurUtils::GaussianBlur method

No one was using the canClobberSrc capability and moving the direct filtering
case forward makes the rest of the logic simpler.

Split out of: const SkSurfaceProps* surfaceProps = nullptr (Retract
GrRenderTarget from SkGpuBlurUtils)

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

to

==========
Simplify SkGpuBlurUtils::GaussianBlur method

No one was using the canClobberSrc capability and moving the direct filtering
case forward makes the rest of the logic simpler.

Split out of: https://codereview.chromium.org/1959493002/ (Retract
GrRenderTarget from SkGpuBlurUtils)

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

[robertphillips, 2016-05-06 13:10:22]

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

[commit-bot: I haz the power, 2016-05-06 13:10:32]

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

[bsalomon, 2016-05-06 13:34:21]

lgtm

[robertphillips, 2016-05-06 13:34:51]

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

[commit-bot: I haz the power, 2016-05-06 13:35:02]

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

[commit-bot: I haz the power, 2016-05-06 13:55:53]

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

[commit-bot: I haz the power, 2016-05-06 13:55:54]

Dry run: This issue passed the CQ dry run.

[robertphillips, 2016-05-06 14:03:02]

The CQ bit was checked by robertphillips@google.com

[robertphillips, 2016-05-06 14:03:02]

The patchset sent to the CQ was uploaded after l-g-t-m from bsalomon@google.com
Link to the patchset: https://codereview.chromium.org/1958603002/#ps20001
(title: "Fix indents")

[commit-bot: I haz the power, 2016-05-06 14:03:12]

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

[commit-bot: I haz the power, 2016-05-06 14:17:52]

Description was changed from

==========
Simplify SkGpuBlurUtils::GaussianBlur method

No one was using the canClobberSrc capability and moving the direct filtering
case forward makes the rest of the logic simpler.

Split out of: https://codereview.chromium.org/1959493002/ (Retract
GrRenderTarget from SkGpuBlurUtils)

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

to

==========
Simplify SkGpuBlurUtils::GaussianBlur method

No one was using the canClobberSrc capability and moving the direct filtering
case forward makes the rest of the logic simpler.

Split out of: https://codereview.chromium.org/1959493002/ (Retract
GrRenderTarget from SkGpuBlurUtils)

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

Committed:
https://skia.googlesource.com/skia/+/56a85e69a8d034e0fdee00e8207cda0a9da06fee
==========

[commit-bot: I haz the power, 2016-05-06 14:17:53]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/56a85e69a8d034e0fdee00e8207cda0a9da06fee

[jvanverth1, 2016-05-06 20:04:51]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/1956023002/ by jvanverth@google.com.

The reason for reverting is: Looks like it's causing some issues with the
bleed_image GM..