Use a raster SkImage in DragImageBuilder

Use a raster SkImage in DragImageBuilder

DragImageBuilder is currently instantiating a picture-backed SkImage.
But there is no advantage to that, because DragImage::create() is
immediately rasterizing it (via SkImage::asLegacyBitmap).

Since we're trying to reduce the use of picture-backed images in Blink,
refactor to rasterize upfront.

R=reed@google.com
BUG=skia:6100
Review-Url: https://codereview.chromium.org/2619573002
Cr-Commit-Position: refs/heads/master@{#442039}
Committed: https://chromium.googlesource.com/chromium/src/+/bef18d10d18bd46e0c8dce3e8ece1176cdfec9fa

[f(malita), 2017-01-06 17:11:00]

The CQ bit was checked by fmalita@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-06 17:11:17]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[reed1, 2017-01-06 18:39:29]

lgtm

[f(malita), 2017-01-06 18:39:44]

Description was changed from

==========
Use a raster SkImage in DragImageBuilder

DragImageBuilder is currently instantiating a picture-backed SkImage.
But there is no advantage to that, because DragImage::create() is
immediately rasterizing it (via SkImage::asLegacyBitmap).

Since we're trying to reduce the use of picture-backed images in Blink,
refactor to rasterize upfront.

R=reed@google.com
==========

to

==========
Use a raster SkImage in DragImageBuilder

DragImageBuilder is currently instantiating a picture-backed SkImage.
But there is no advantage to that, because DragImage::create() is
immediately rasterizing it (via SkImage::asLegacyBitmap).

Since we're trying to reduce the use of picture-backed images in Blink,
refactor to rasterize upfront.

R=reed@google.com
BUG=skia:6100
==========

[f(malita), 2017-01-06 18:39:49]

The CQ bit was unchecked by fmalita@chromium.org

[f(malita), 2017-01-06 18:39:54]

The CQ bit was checked by fmalita@chromium.org

[commit-bot: I haz the power, 2017-01-06 18:40:43]

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-01-06 20:47:19]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1483727994756410, "parent_rev":
"4aa74f3897170953b1df843a46eef77f8a6a8055", "commit_rev":
"bef18d10d18bd46e0c8dce3e8ece1176cdfec9fa"}

[commit-bot: I haz the power, 2017-01-06 20:47:49]

Description was changed from

==========
Use a raster SkImage in DragImageBuilder

DragImageBuilder is currently instantiating a picture-backed SkImage.
But there is no advantage to that, because DragImage::create() is
immediately rasterizing it (via SkImage::asLegacyBitmap).

Since we're trying to reduce the use of picture-backed images in Blink,
refactor to rasterize upfront.

R=reed@google.com
BUG=skia:6100
==========

to

==========
Use a raster SkImage in DragImageBuilder

DragImageBuilder is currently instantiating a picture-backed SkImage.
But there is no advantage to that, because DragImage::create() is
immediately rasterizing it (via SkImage::asLegacyBitmap).

Since we're trying to reduce the use of picture-backed images in Blink,
refactor to rasterize upfront.

R=reed@google.com
BUG=skia:6100

Review-Url: https://codereview.chromium.org/2619573002
Cr-Commit-Position: refs/heads/master@{#442039}
Committed:
https://chromium.googlesource.com/chromium/src/+/bef18d10d18bd46e0c8dce3e8ece...
==========

[commit-bot: I haz the power, 2017-01-06 20:47:50]

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