From 6ceeebd37a43d879c120b6ba100ae1febdd67a18 Mon Sep 17 00:00:00 2001 From: reed Date: Wed, 9 Mar 2016 14:26:26 -0800 Subject: unify peekPixels around pixmap parameter requires this chrome CL to land first https://codereview.chromium.org/1775393003/ BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1784563002 Review URL: https://codereview.chromium.org/1784563002 --- example/HelloWorld.cpp | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) (limited to 'example') diff --git a/example/HelloWorld.cpp b/example/HelloWorld.cpp index 1cc18fdc8d..e13cb3b438 100644 --- a/example/HelloWorld.cpp +++ b/example/HelloWorld.cpp @@ -156,18 +156,18 @@ void HelloWorldWindow::draw(SkCanvas* canvas) { if (kRaster_DeviceType == fType) { // need to send the raster bits to the (gpu) window - SkImage* snap = fSurface->newImageSnapshot(); - size_t rowBytes = 0; - SkImageInfo info; - const void* pixels = snap->peekPixels(&info, &rowBytes); - fRenderTarget->writePixels(0, 0, snap->width(), snap->height(), - SkImageInfo2GrPixelConfig(info.colorType(), - info.alphaType(), - info.profileType()), - pixels, - rowBytes, - GrContext::kFlushWrites_PixelOp); - SkSafeUnref(snap); + sk_sp snap = sk_sp(fSurface->newImageSnapshot()); + SkPixmap pmap; + if (snap->peekPixels(&pmap)) { + const SkImageInfo& info = pmap.info(); + fRenderTarget->writePixels(0, 0, snap->width(), snap->height(), + SkImageInfo2GrPixelConfig(info.colorType(), + info.alphaType(), + info.profileType()), + pmap.addr(), + pmap.rowBytes(), + GrContext::kFlushWrites_PixelOp); + } } INHERITED::present(); } -- cgit v1.2.3