Revert of Make cc/paint have concrete types

Revert of Make cc/paint have concrete types (patchset #36 id:700001 of https://codereview.chromium.org/2690583002/ )

Reason for revert:
Invalid casts from SkMiniPicture to PaintRecord

BUG=699105
Original issue's description:
> Make cc/paint have concrete types
> 
> This changes PaintCanvas, PaintFlags, PaintSurface, and PaintRecorder
> to all be real types (that forward to Skia types internally).  PaintShader
> is left as-is for now.  This will force callers to use the correct
> types in the rest of Chromium as the internals of these classes are
> rewritten in future patches.
> 
> This code also changes a number of callers elsewhere in the codebase
> that want to wrap an SkCanvas in a PaintCanvas.  As SkCanvas has no
> constructor that takes an SkCanvas*, this change had to wait for this
> final conversion patch.
> 
> In general, if code wants to raster directly into a bitmap with local
> code, it should use Skia directly.  If code wants to raster into a bitmap
> with paint callers, it can wrap that bitmap in a PaintCanvas.  Similarly,
> if code wants to go into an accelerated SkSurface, it should wrap that
> surface's canvas in a PaintCanvas (as blink html canvas code does).
> 
> BUG=671433
> 
> 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/2690583002
> Cr-Commit-Position: refs/heads/master@{#454962}
> Committed: https://chromium.googlesource.com/chromium/src/+/762276ecdfbff938d3b7431ad0ed00c3ab6a4136

TBR=danakj@chromium.org,ddorwin@chromium.org,derat@chromium.org,junov@chromium.org,nick@chromium.org,vitalybuka@chromium.rg,vmpstr@chromium.org,thestig@chromium.org,vitalybuka@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=671433
Review-Url: https://codereview.chromium.org/2739533003
Cr-Commit-Position: refs/heads/master@{#455164}
Committed: https://chromium.googlesource.com/chromium/src/+/6667be4ffb8c4e01654f2dc2471371e83dfecacd

[enne (OOO), 2017-03-07 18:51:42]

The CQ bit was checked by enne@chromium.org

[enne (OOO), 2017-03-07 18:51:43]

Created Revert of Make cc/paint have concrete types

[commit-bot: I haz the power, 2017-03-07 18:53:20]

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-03-07 18:59:02]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488912702885330, "parent_rev":
"b1d2e017aef891e5d9bbd4d58b22fe9b08c46db3", "commit_rev":
"6667be4ffb8c4e01654f2dc2471371e83dfecacd"}

[commit-bot: I haz the power, 2017-03-07 18:59:45]

Description was changed from

==========
Revert of Make cc/paint have concrete types (patchset #36 id:700001 of
https://codereview.chromium.org/2690583002/ )

Reason for revert:
Invalid casts from SkMiniPicture to PaintRecord

BUG=699105

Original issue's description:
> Make cc/paint have concrete types
> 
> This changes PaintCanvas, PaintFlags, PaintSurface, and PaintRecorder
> to all be real types (that forward to Skia types internally).  PaintShader
> is left as-is for now.  This will force callers to use the correct
> types in the rest of Chromium as the internals of these classes are
> rewritten in future patches.
> 
> This code also changes a number of callers elsewhere in the codebase
> that want to wrap an SkCanvas in a PaintCanvas.  As SkCanvas has no
> constructor that takes an SkCanvas*, this change had to wait for this
> final conversion patch.
> 
> In general, if code wants to raster directly into a bitmap with local
> code, it should use Skia directly.  If code wants to raster into a bitmap
> with paint callers, it can wrap that bitmap in a PaintCanvas.  Similarly,
> if code wants to go into an accelerated SkSurface, it should wrap that
> surface's canvas in a PaintCanvas (as blink html canvas code does).
> 
> BUG=671433
> 
>
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/2690583002
> Cr-Commit-Position: refs/heads/master@{#454962}
> Committed:
https://chromium.googlesource.com/chromium/src/+/762276ecdfbff938d3b7431ad0ed...

TBR=danakj@chromium.org,ddorwin@chromium.org,derat@chromium.org,junov@chromiu...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=671433
==========

to

==========
Revert of Make cc/paint have concrete types (patchset #36 id:700001 of
https://codereview.chromium.org/2690583002/ )

Reason for revert:
Invalid casts from SkMiniPicture to PaintRecord

BUG=699105

Original issue's description:
> Make cc/paint have concrete types
> 
> This changes PaintCanvas, PaintFlags, PaintSurface, and PaintRecorder
> to all be real types (that forward to Skia types internally).  PaintShader
> is left as-is for now.  This will force callers to use the correct
> types in the rest of Chromium as the internals of these classes are
> rewritten in future patches.
> 
> This code also changes a number of callers elsewhere in the codebase
> that want to wrap an SkCanvas in a PaintCanvas.  As SkCanvas has no
> constructor that takes an SkCanvas*, this change had to wait for this
> final conversion patch.
> 
> In general, if code wants to raster directly into a bitmap with local
> code, it should use Skia directly.  If code wants to raster into a bitmap
> with paint callers, it can wrap that bitmap in a PaintCanvas.  Similarly,
> if code wants to go into an accelerated SkSurface, it should wrap that
> surface's canvas in a PaintCanvas (as blink html canvas code does).
> 
> BUG=671433
> 
>
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/2690583002
> Cr-Commit-Position: refs/heads/master@{#454962}
> Committed:
https://chromium.googlesource.com/chromium/src/+/762276ecdfbff938d3b7431ad0ed...

TBR=danakj@chromium.org,ddorwin@chromium.org,derat@chromium.org,junov@chromiu...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=671433

Review-Url: https://codereview.chromium.org/2739533003
Cr-Commit-Position: refs/heads/master@{#455164}
Committed:
https://chromium.googlesource.com/chromium/src/+/6667be4ffb8c4e01654f2dc24713...
==========

[commit-bot: I haz the power, 2017-03-07 18:59:46]

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