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

Unified Diff: src/gpu/SkGrPixelRef.cpp

Issue 2167173002: Remove SkGrPixelRef (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 4 years, 5 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
« no previous file with comments | « src/gpu/SkGrPixelRef.h ('k') | src/gpu/SkGrPriv.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/gpu/SkGrPixelRef.cpp
diff --git a/src/gpu/SkGrPixelRef.cpp b/src/gpu/SkGrPixelRef.cpp
deleted file mode 100644
index df2300369d3359a8311a6f86c55422f7704ec795..0000000000000000000000000000000000000000
--- a/src/gpu/SkGrPixelRef.cpp
+++ /dev/null
@@ -1,212 +0,0 @@
-/*
- * Copyright 2010 Google Inc.
- *
- * Use of this source code is governed by a BSD-style license that can be
- * found in the LICENSE file.
- */
-
-
-
-#include "SkGrPixelRef.h"
-
-#include "GrContext.h"
-#include "GrTexture.h"
-#include "GrTexturePriv.h"
-#include "SkBitmapCache.h"
-#include "SkGr.h"
-#include "SkRect.h"
-
-SkROLockPixelsPixelRef::SkROLockPixelsPixelRef(const SkImageInfo& info)
- : INHERITED(info) {}
-
-SkROLockPixelsPixelRef::~SkROLockPixelsPixelRef() {}
-
-bool SkROLockPixelsPixelRef::onNewLockPixels(LockRec* rec) {
- fBitmap.reset();
-// SkDebugf("---------- calling readpixels in support of lockpixels\n");
- if (!this->onReadPixels(&fBitmap, this->info().colorType(), nullptr)) {
- SkDebugf("SkROLockPixelsPixelRef::onLockPixels failed!\n");
- return false;
- }
- fBitmap.lockPixels();
- if (nullptr == fBitmap.getPixels()) {
- return false;
- }
-
- rec->fPixels = fBitmap.getPixels();
- rec->fColorTable = nullptr;
- rec->fRowBytes = fBitmap.rowBytes();
- return true;
-}
-
-void SkROLockPixelsPixelRef::onUnlockPixels() {
- fBitmap.unlockPixels();
-}
-
-bool SkROLockPixelsPixelRef::onLockPixelsAreWritable() const {
- return false;
-}
-
-///////////////////////////////////////////////////////////////////////////////
-
-static SkGrPixelRef* copy_to_new_texture_pixelref(GrTexture* texture, SkColorType dstCT,
- SkColorSpace* dstCS, const SkIRect* subset) {
- if (nullptr == texture || kUnknown_SkColorType == dstCT) {
- return nullptr;
- }
- GrContext* context = texture->getContext();
- if (nullptr == context) {
- return nullptr;
- }
- GrSurfaceDesc desc;
-
- SkIRect srcRect;
-
- if (!subset) {
- desc.fWidth = texture->width();
- desc.fHeight = texture->height();
- srcRect = SkIRect::MakeWH(texture->width(), texture->height());
- } else {
- SkASSERT(SkIRect::MakeWH(texture->width(), texture->height()).contains(*subset));
- // Create a new texture that is the size of subset.
- desc.fWidth = subset->width();
- desc.fHeight = subset->height();
- srcRect = *subset;
- }
- desc.fFlags = kRenderTarget_GrSurfaceFlag;
- desc.fConfig = SkImageInfo2GrPixelConfig(dstCT, kPremul_SkAlphaType, dstCS, *context->caps());
- desc.fIsMipMapped = false;
-
- GrTexture* dst = context->textureProvider()->createTexture(desc, SkBudgeted::kNo, nullptr, 0);
- if (nullptr == dst) {
- return nullptr;
- }
-
- // Blink is relying on the above copy being sent to GL immediately in the case when the source
- // is a WebGL canvas backing store. We could have a TODO to remove this flush, but we have
- // a larger TODO to remove SkGrPixelRef entirely.
- context->copySurface(dst, texture, srcRect, SkIPoint::Make(0,0));
- context->flushSurfaceWrites(dst);
-
- SkImageInfo info = SkImageInfo::Make(desc.fWidth, desc.fHeight, dstCT, kPremul_SkAlphaType,
- sk_ref_sp(dstCS));
- SkGrPixelRef* pixelRef = new SkGrPixelRef(info, dst);
- SkSafeUnref(dst);
- return pixelRef;
-}
-
-///////////////////////////////////////////////////////////////////////////////
-
-SkGrPixelRef::SkGrPixelRef(const SkImageInfo& info, GrSurface* surface) : INHERITED(info) {
- // For surfaces that are both textures and render targets, the texture owns the
- // render target but not vice versa. So we ref the texture to keep both alive for
- // the lifetime of this pixel ref.
- fSurface = SkSafeRef(surface->asTexture());
- if (nullptr == fSurface) {
- fSurface = SkSafeRef(surface);
- }
-
- if (fSurface) {
- SkASSERT(info.width() <= fSurface->width());
- SkASSERT(info.height() <= fSurface->height());
- }
-}
-
-SkGrPixelRef::~SkGrPixelRef() {
- SkSafeUnref(fSurface);
-}
-
-GrTexture* SkGrPixelRef::getTexture() {
- if (fSurface) {
- return fSurface->asTexture();
- }
- return nullptr;
-}
-
-void SkGrPixelRef::onNotifyPixelsChanged() {
- GrTexture* texture = this->getTexture();
- if (texture) {
- texture->texturePriv().dirtyMipMaps(true);
- }
-}
-
-SkPixelRef* SkGrPixelRef::deepCopy(SkColorType dstCT, SkColorSpace* dstCS, const SkIRect* subset) {
- if (nullptr == fSurface) {
- return nullptr;
- }
-
- // Note that when copying a render-target-backed pixel ref, we
- // return a texture-backed pixel ref instead. This is because
- // render-target pixel refs are usually created in conjunction with
- // a GrTexture owned elsewhere (e.g., SkGpuDevice), and cannot live
- // independently of that texture. Texture-backed pixel refs, on the other
- // hand, own their GrTextures, and are thus self-contained.
- return copy_to_new_texture_pixelref(fSurface->asTexture(), dstCT, dstCS, subset);
-}
-
-static bool tryAllocBitmapPixels(SkBitmap* bitmap) {
- SkBitmap::Allocator* allocator = SkBitmapCache::GetAllocator();
- if (nullptr != allocator) {
- return allocator->allocPixelRef(bitmap, 0);
- } else {
- // DiscardableMemory is not available, fallback to default allocator
- return bitmap->tryAllocPixels();
- }
-}
-
-bool SkGrPixelRef::onReadPixels(SkBitmap* dst, SkColorType colorType, const SkIRect* subset) {
- if (nullptr == fSurface || fSurface->wasDestroyed()) {
- return false;
- }
-
- GrPixelConfig config;
- if (kRGBA_8888_SkColorType == colorType) {
- config = kRGBA_8888_GrPixelConfig;
- } else if (kBGRA_8888_SkColorType == colorType) {
- config = kBGRA_8888_GrPixelConfig;
- } else {
- return false;
- }
-
- SkIRect bounds;
- if (subset) {
- bounds = *subset;
- } else {
- bounds = SkIRect::MakeWH(this->info().width(), this->info().height());
- }
-
- //Check the cache
- if(!SkBitmapCache::Find(this->getGenerationID(), bounds, dst)) {
- //Cache miss
-
- SkBitmap cachedBitmap;
- cachedBitmap.setInfo(SkImageInfo::Make(bounds.width(), bounds.height(), colorType,
- this->info().alphaType(),
- sk_ref_sp(this->info().colorSpace())));
-
- // If we can't alloc the pixels, then fail
- if (!tryAllocBitmapPixels(&cachedBitmap)) {
- return false;
- }
-
- // Try to read the pixels from the surface
- void* buffer = cachedBitmap.getPixels();
- bool readPixelsOk = fSurface->readPixels(bounds.fLeft, bounds.fTop,
- bounds.width(), bounds.height(),
- config, buffer, cachedBitmap.rowBytes());
-
- if (!readPixelsOk) {
- return false;
- }
-
- // If we are here, pixels were read correctly from the surface.
- cachedBitmap.setImmutable();
- //Add to the cache
- SkBitmapCache::Add(this, bounds, cachedBitmap);
-
- dst->swap(cachedBitmap);
- }
-
- return true;
-
-}
« no previous file with comments | « src/gpu/SkGrPixelRef.h ('k') | src/gpu/SkGrPriv.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698