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

Unified Diff: third_party/WebKit/Source/platform/graphics/skia/ImagePixelLocker.h

Issue 1813483002: ImagePixelLocker now manually allocates SkPixmap (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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: third_party/WebKit/Source/platform/graphics/skia/ImagePixelLocker.h
diff --git a/third_party/WebKit/Source/platform/graphics/skia/ImagePixelLocker.h b/third_party/WebKit/Source/platform/graphics/skia/ImagePixelLocker.h
index 3b486dfe375deaba7086108fc93393f9376f7f43..e6d0fbd5610173c43574f0c34028e2f54e8237a4 100644
--- a/third_party/WebKit/Source/platform/graphics/skia/ImagePixelLocker.h
+++ b/third_party/WebKit/Source/platform/graphics/skia/ImagePixelLocker.h
@@ -22,13 +22,14 @@ class ImagePixelLocker final {
WTF_MAKE_NONCOPYABLE(ImagePixelLocker);
public:
ImagePixelLocker(PassRefPtr<const SkImage>, SkAlphaType, SkColorType);
+ ~ImagePixelLocker();
const void* pixels() const { return m_pixels; }
private:
const RefPtr<const SkImage> m_image;
const void* m_pixels;
- SkAutoPixmapStorage m_pixelStorage;
+ SkPixmap m_pixelStorage;
f(malita) 2016/03/16 19:28:29 Let's rename this to m_pixmap.
robertphillips 2016/03/16 19:59:10 It is actually back to being more of a pure pixel
};
} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698