Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3709)

Unified Diff: cc/tiles/software_image_decode_controller.cc

Issue 1869753003: Replace many skia::RefPtr with sk_sp<> in cc/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: cc/tiles/software_image_decode_controller.cc
diff --git a/cc/tiles/software_image_decode_controller.cc b/cc/tiles/software_image_decode_controller.cc
index 8ed5a0eb4e33fdaf94e30d8daedb76c8e30019b5..bb133b2ec8a5c01f9c5d3e7f98ba0c32099d61e9 100644
--- a/cc/tiles/software_image_decode_controller.cc
+++ b/cc/tiles/software_image_decode_controller.cc
@@ -63,7 +63,7 @@ class ImageDecodeTaskImpl : public ImageDecodeTask {
: controller_(controller),
image_key_(image_key),
image_(image),
- image_ref_(skia::SharePtr(image.image())),
+ image_ref_(sk_ref_sp(image.image())),
danakj 2016/04/14 19:37:33 make image() return sk_sp
tomhudson 2016/04/25 20:48:15 Done.
source_prepare_tiles_id_(source_prepare_tiles_id) {}
// Overridden from Task:
@@ -89,7 +89,7 @@ class ImageDecodeTaskImpl : public ImageDecodeTask {
SoftwareImageDecodeController* controller_;
SoftwareImageDecodeController::ImageKey image_key_;
DrawImage image_;
- skia::RefPtr<const SkImage> image_ref_;
+ sk_sp<const SkImage> image_ref_;
uint64_t source_prepare_tiles_id_;
DISALLOW_COPY_AND_ASSIGN(ImageDecodeTaskImpl);
@@ -880,9 +880,9 @@ SoftwareImageDecodeController::DecodedImage::DecodedImage(
memory_(std::move(memory)),
src_rect_offset_(src_rect_offset),
tracing_id_(tracing_id) {
- image_ = skia::AdoptRef(SkImage::NewFromRaster(
- image_info_, memory_->data(), image_info_.minRowBytes(),
- [](const void* pixels, void* context) {}, nullptr));
+ SkPixmap p(image_info_, memory_->data(), image_info_.minRowBytes());
+ image_ = SkImage::MakeFromRaster(p, [](const void* pixels, void* context) {},
+ nullptr);
}
SoftwareImageDecodeController::DecodedImage::~DecodedImage() {

Powered by Google App Engine
This is Rietveld 408576698