Revert of cc: Move SkShader construction to a single spot in PaintShader

Revert of cc: Move SkShader construction to a single spot in PaintShader (patchset #8 id:140001 of https://codereview.chromium.org/2893083002/ )

Reason for revert:
Broke Linux ChromiumOS Builder (dbg):
https://luci-milo.appspot.com/buildbot/chromium.chromiumos/Linux%20ChromiumOS%20Builder%20%28dbg%29/88131

Original issue's description:
> cc: Move SkShader construction to a single spot in PaintShader
> 
> This patch moves SkShader construction to be only in PaintShader
> (for the purposes of paint, at least). This ensures that we can
> transport all of the relevant parameters required for the shader
> and defer construction until raster.
> 
> BUG=728359
> R=enne@chromium.org
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Review-Url: https://codereview.chromium.org/2893083002
> Cr-Commit-Position: refs/heads/master@{#477810}
> Committed: https://chromium.googlesource.com/chromium/src/+/cda52d0cc0fea52b996fce1663a8de5141fe48c3

TBR=enne@chromium.org,pdr@chromium.org,sky@chromium.org,piman@chromium.org,junov@chromium.org,senorblanco@chromium.org,vmpstr@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=728359
Review-Url: https://codereview.chromium.org/2928703005
Cr-Commit-Position: refs/heads/master@{#477827}
Committed: https://chromium.googlesource.com/chromium/src/+/8cae4d389092473a097baf547b1fecfe2259e748

[Sam McNally, 2017-06-08 00:29:48]

The CQ bit was checked by sammc@chromium.org

[Sam McNally, 2017-06-08 00:29:49]

Created Revert of cc: Move SkShader construction to a single spot in PaintShader

[commit-bot: I haz the power, 2017-06-08 00:30:13]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-08 00:30:50]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496881788860850, "parent_rev":
"c0f846bdbfa4e685f05c48443923db9dfceaa35d", "commit_rev":
"8cae4d389092473a097baf547b1fecfe2259e748"}

[commit-bot: I haz the power, 2017-06-08 00:30:59]

Description was changed from

==========
Revert of cc: Move SkShader construction to a single spot in PaintShader
(patchset #8 id:140001 of https://codereview.chromium.org/2893083002/ )

Reason for revert:
Broke Linux ChromiumOS Builder (dbg):
https://luci-milo.appspot.com/buildbot/chromium.chromiumos/Linux%20ChromiumOS...

Original issue's description:
> cc: Move SkShader construction to a single spot in PaintShader
> 
> This patch moves SkShader construction to be only in PaintShader
> (for the purposes of paint, at least). This ensures that we can
> transport all of the relevant parameters required for the shader
> and defer construction until raster.
> 
> BUG=728359
> R=enne@chromium.org
>
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Review-Url: https://codereview.chromium.org/2893083002
> Cr-Commit-Position: refs/heads/master@{#477810}
> Committed:
https://chromium.googlesource.com/chromium/src/+/cda52d0cc0fea52b996fce1663a8...

TBR=enne@chromium.org,pdr@chromium.org,sky@chromium.org,piman@chromium.org,ju...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=728359
==========

to

==========
Revert of cc: Move SkShader construction to a single spot in PaintShader
(patchset #8 id:140001 of https://codereview.chromium.org/2893083002/ )

Reason for revert:
Broke Linux ChromiumOS Builder (dbg):
https://luci-milo.appspot.com/buildbot/chromium.chromiumos/Linux%20ChromiumOS...

Original issue's description:
> cc: Move SkShader construction to a single spot in PaintShader
> 
> This patch moves SkShader construction to be only in PaintShader
> (for the purposes of paint, at least). This ensures that we can
> transport all of the relevant parameters required for the shader
> and defer construction until raster.
> 
> BUG=728359
> R=enne@chromium.org
>
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Review-Url: https://codereview.chromium.org/2893083002
> Cr-Commit-Position: refs/heads/master@{#477810}
> Committed:
https://chromium.googlesource.com/chromium/src/+/cda52d0cc0fea52b996fce1663a8...

TBR=enne@chromium.org,pdr@chromium.org,sky@chromium.org,piman@chromium.org,ju...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=728359

Review-Url: https://codereview.chromium.org/2928703005
Cr-Commit-Position: refs/heads/master@{#477827}
Committed:
https://chromium.googlesource.com/chromium/src/+/8cae4d389092473a097baf547b1f...
==========

[commit-bot: I haz the power, 2017-06-08 00:31:01]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/8cae4d389092473a097baf547b1f...