Remove GrRenderable param from GrProxyProvider::createTextureProxy()
Every caller passes kNo.
Change-Id: I9c21c77f57f6215246794b39fc8ec485d4ae23d0
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/234331
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
diff --git a/src/gpu/GrBlurUtils.cpp b/src/gpu/GrBlurUtils.cpp
index 6a08f6f..d6353ab 100644
--- a/src/gpu/GrBlurUtils.cpp
+++ b/src/gpu/GrBlurUtils.cpp
@@ -149,8 +149,8 @@
return false;
}
- filteredMask = proxyProvider->createTextureProxy(std::move(image), GrRenderable::kNo, 1,
- SkBudgeted::kYes, SkBackingFit::kApprox);
+ filteredMask = proxyProvider->createTextureProxy(std::move(image), 1, SkBudgeted::kYes,
+ SkBackingFit::kApprox);
if (!filteredMask) {
return false;
}
diff --git a/src/gpu/GrProxyProvider.cpp b/src/gpu/GrProxyProvider.cpp
index bfa6a63..95a5042 100644
--- a/src/gpu/GrProxyProvider.cpp
+++ b/src/gpu/GrProxyProvider.cpp
@@ -246,7 +246,6 @@
}
sk_sp<GrTextureProxy> GrProxyProvider::createTextureProxy(sk_sp<SkImage> srcImage,
- GrRenderable renderable,
int sampleCnt,
SkBudgeted budgeted,
SkBackingFit fit,
@@ -261,7 +260,7 @@
const SkImageInfo& info = srcImage->imageInfo();
GrColorType ct = SkColorTypeToGrColorType(info.colorType());
- GrBackendFormat format = this->caps()->getDefaultBackendFormat(ct, renderable);
+ GrBackendFormat format = this->caps()->getDefaultBackendFormat(ct, GrRenderable::kNo);
if (!format.isValid()) {
SkBitmap copy8888;
@@ -272,19 +271,12 @@
copy8888.setImmutable();
srcImage = SkMakeImageFromRasterBitmap(copy8888, kNever_SkCopyPixelsMode);
ct = GrColorType::kRGBA_8888;
- format = this->caps()->getDefaultBackendFormat(ct, renderable);
+ format = this->caps()->getDefaultBackendFormat(ct, GrRenderable::kNo);
if (!format.isValid()) {
return nullptr;
}
}
- if (renderable == GrRenderable::kYes) {
- sampleCnt = this->caps()->getRenderTargetSampleCount(sampleCnt, format);
- if (!sampleCnt) {
- return nullptr;
- }
- }
-
GrPixelConfig config = GrColorTypeToPixelConfig(ct);
if (kUnknown_GrPixelConfig == config) {
return nullptr;
@@ -296,17 +288,17 @@
desc.fConfig = config;
sk_sp<GrTextureProxy> proxy = this->createLazyProxy(
- [desc, format, renderable, sampleCnt, budgeted, srcImage, fit,
+ [desc, format, sampleCnt, budgeted, srcImage, fit,
ct](GrResourceProvider* resourceProvider) {
SkPixmap pixMap;
SkAssertResult(srcImage->peekPixels(&pixMap));
GrMipLevel mipLevel = { pixMap.addr(), pixMap.rowBytes() };
return LazyInstantiationResult(resourceProvider->createTexture(
- desc, format, renderable, sampleCnt, budgeted, fit, GrProtected::kNo, ct,
- mipLevel, GrResourceProvider::Flags::kNoPendingIO));
+ desc, format, GrRenderable::kNo, sampleCnt, budgeted, fit, GrProtected::kNo,
+ ct, mipLevel, GrResourceProvider::Flags::kNoPendingIO));
},
- format, desc, renderable, sampleCnt, kTopLeft_GrSurfaceOrigin, GrMipMapped::kNo,
+ format, desc, GrRenderable::kNo, sampleCnt, kTopLeft_GrSurfaceOrigin, GrMipMapped::kNo,
GrMipMapsStatus::kNotAllocated, surfaceFlags, fit, budgeted, GrProtected::kNo);
if (!proxy) {
@@ -377,8 +369,8 @@
// If mips weren't requested (or this was too small to have any), then take the fast path
if (GrMipMapped::kNo == mipMapped ||
0 == SkMipMap::ComputeLevelCount(baseLevel->width(), baseLevel->height())) {
- return this->createTextureProxy(std::move(baseLevel), GrRenderable::kNo, 1,
- SkBudgeted::kYes, SkBackingFit::kExact);
+ return this->createTextureProxy(std::move(baseLevel), 1, SkBudgeted::kYes,
+ SkBackingFit::kExact);
}
GrSurfaceDesc desc = GrImageInfoToSurfaceDesc(bitmap.info());
diff --git a/src/gpu/GrProxyProvider.h b/src/gpu/GrProxyProvider.h
index 58c884e..7c2fa25 100644
--- a/src/gpu/GrProxyProvider.h
+++ b/src/gpu/GrProxyProvider.h
@@ -65,7 +65,7 @@
* actually upload the data to the gpu.
*/
sk_sp<GrTextureProxy> createTextureProxy(
- sk_sp<SkImage> srcImage, GrRenderable, int sampleCnt, SkBudgeted, SkBackingFit,
+ sk_sp<SkImage> srcImage, int sampleCnt, SkBudgeted, SkBackingFit,
GrInternalSurfaceFlags = GrInternalSurfaceFlags::kNone);
/*
diff --git a/src/gpu/GrSWMaskHelper.cpp b/src/gpu/GrSWMaskHelper.cpp
index 505a215..879a028 100644
--- a/src/gpu/GrSWMaskHelper.cpp
+++ b/src/gpu/GrSWMaskHelper.cpp
@@ -107,6 +107,6 @@
return nullptr;
}
- return context->priv().proxyProvider()->createTextureProxy(std::move(img), GrRenderable::kNo, 1,
- SkBudgeted::kYes, fit);
+ return context->priv().proxyProvider()->createTextureProxy(std::move(img), 1, SkBudgeted::kYes,
+ fit);
}
diff --git a/src/gpu/GrYUVProvider.cpp b/src/gpu/GrYUVProvider.cpp
index 93eb6ab..baabcf0 100644
--- a/src/gpu/GrYUVProvider.cpp
+++ b/src/gpu/GrYUVProvider.cpp
@@ -150,8 +150,8 @@
dataStoragePtr);
auto proxyProvider = ctx->priv().proxyProvider();
- yuvTextureProxies[i] = proxyProvider->createTextureProxy(yuvImage, GrRenderable::kNo, 1,
- SkBudgeted::kYes, fit);
+ yuvTextureProxies[i] =
+ proxyProvider->createTextureProxy(yuvImage, 1, SkBudgeted::kYes, fit);
SkASSERT(yuvTextureProxies[i]->width() == yuvSizeInfo.fSizes[i].fWidth);
SkASSERT(yuvTextureProxies[i]->height() == yuvSizeInfo.fSizes[i].fHeight);
diff --git a/src/gpu/SkGr.cpp b/src/gpu/SkGr.cpp
index d979f19..fd63056 100644
--- a/src/gpu/SkGr.cpp
+++ b/src/gpu/SkGr.cpp
@@ -194,8 +194,7 @@
kTopLeft_GrSurfaceOrigin);
}
if (!proxy) {
- proxy = proxyProvider->createTextureProxy(srcImage, GrRenderable::kNo, 1, SkBudgeted::kYes,
- fit);
+ proxy = proxyProvider->createTextureProxy(srcImage, 1, SkBudgeted::kYes, fit);
if (proxy && originalKey.isValid()) {
proxyProvider->assignUniqueKeyToProxy(originalKey, proxy.get());
const SkBitmap* bm = as_IB(srcImage.get())->onPeekBitmap();
diff --git a/src/gpu/effects/GrCircleBlurFragmentProcessor.fp b/src/gpu/effects/GrCircleBlurFragmentProcessor.fp
index 9fe9295..4228a03 100644
--- a/src/gpu/effects/GrCircleBlurFragmentProcessor.fp
+++ b/src/gpu/effects/GrCircleBlurFragmentProcessor.fp
@@ -248,7 +248,7 @@
bm.setImmutable();
sk_sp<SkImage> image = SkImage::MakeFromBitmap(bm);
- blurProfile = proxyProvider->createTextureProxy(std::move(image), GrRenderable::kNo, 1,
+ blurProfile = proxyProvider->createTextureProxy(std::move(image), 1,
SkBudgeted::kYes, SkBackingFit::kExact);
if (!blurProfile) {
return nullptr;
diff --git a/src/gpu/effects/GrConfigConversionEffect.fp b/src/gpu/effects/GrConfigConversionEffect.fp
index a4320b2..ba58cc2 100644
--- a/src/gpu/effects/GrConfigConversionEffect.fp
+++ b/src/gpu/effects/GrConfigConversionEffect.fp
@@ -65,7 +65,6 @@
sk_sp<SkImage> image = SkImage::MakeFromRaster(pixmap, nullptr, nullptr);
sk_sp<GrTextureProxy> dataProxy = proxyProvider->createTextureProxy(std::move(image),
- GrRenderable::kNo,
1,
SkBudgeted::kYes,
SkBackingFit::kExact);
diff --git a/src/gpu/effects/GrRectBlurEffect.fp b/src/gpu/effects/GrRectBlurEffect.fp
index 12e3614..f625af2 100644
--- a/src/gpu/effects/GrRectBlurEffect.fp
+++ b/src/gpu/effects/GrRectBlurEffect.fp
@@ -69,7 +69,7 @@
return nullptr;
}
- blurProfile = proxyProvider->createTextureProxy(std::move(image), GrRenderable::kNo,
+ blurProfile = proxyProvider->createTextureProxy(std::move(image),
1, SkBudgeted::kYes,
SkBackingFit::kExact);
if (!blurProfile) {
diff --git a/src/gpu/effects/generated/GrCircleBlurFragmentProcessor.cpp b/src/gpu/effects/generated/GrCircleBlurFragmentProcessor.cpp
index 9445a5a..ccef347 100644
--- a/src/gpu/effects/generated/GrCircleBlurFragmentProcessor.cpp
+++ b/src/gpu/effects/generated/GrCircleBlurFragmentProcessor.cpp
@@ -225,8 +225,8 @@
bm.setImmutable();
sk_sp<SkImage> image = SkImage::MakeFromBitmap(bm);
- blurProfile = proxyProvider->createTextureProxy(std::move(image), GrRenderable::kNo, 1,
- SkBudgeted::kYes, SkBackingFit::kExact);
+ blurProfile = proxyProvider->createTextureProxy(std::move(image), 1, SkBudgeted::kYes,
+ SkBackingFit::kExact);
if (!blurProfile) {
return nullptr;
}
diff --git a/src/gpu/effects/generated/GrConfigConversionEffect.h b/src/gpu/effects/generated/GrConfigConversionEffect.h
index 8cbabb6..ee45889 100644
--- a/src/gpu/effects/generated/GrConfigConversionEffect.h
+++ b/src/gpu/effects/generated/GrConfigConversionEffect.h
@@ -69,7 +69,7 @@
sk_sp<SkImage> image = SkImage::MakeFromRaster(pixmap, nullptr, nullptr);
sk_sp<GrTextureProxy> dataProxy = proxyProvider->createTextureProxy(
- std::move(image), GrRenderable::kNo, 1, SkBudgeted::kYes, SkBackingFit::kExact);
+ std::move(image), 1, SkBudgeted::kYes, SkBackingFit::kExact);
if (!dataProxy) {
return false;
}
diff --git a/src/gpu/effects/generated/GrRectBlurEffect.h b/src/gpu/effects/generated/GrRectBlurEffect.h
index 361aa26..7083222 100644
--- a/src/gpu/effects/generated/GrRectBlurEffect.h
+++ b/src/gpu/effects/generated/GrRectBlurEffect.h
@@ -49,8 +49,8 @@
return nullptr;
}
- blurProfile = proxyProvider->createTextureProxy(std::move(image), GrRenderable::kNo, 1,
- SkBudgeted::kYes, SkBackingFit::kExact);
+ blurProfile = proxyProvider->createTextureProxy(std::move(image), 1, SkBudgeted::kYes,
+ SkBackingFit::kExact);
if (!blurProfile) {
return nullptr;
}