AuroraMiddleware/skia2
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

remove RGB_888x pixel tests exceptions.

Browse Source 
Should have been removed with
https://skia.googlesource.com/skia/+/974c821502042c0ab233affb0f38bf2b49f5ecfe

Bug: skia:8862
Change-Id: I3e6445a006165cc2d5713c4b21b23ed674c0a6fe
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/405399
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
This commit is contained in:
Brian Salomon 2021-05-06 15:18:34 -04:00 committed by Skia Commit-Bot
parent 1dc2d0fe0f
commit 6e0e7cbe89
1 changed files with 0 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
tests/ReadWritePixelsGpuTest.cpp
View File

@ -600,10 +600,6 @@ DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SurfaceAsyncReadPixels, reporter, ctxInfo) {
for (GrSurfaceOrigin origin : {kTopLeft_GrSurfaceOrigin, kBottomLeft_GrSurfaceOrigin}) {
auto factory = std::function<GpuSrcFactory<Surface>>(
[context = ctxInfo.directContext(), origin](const SkPixmap& src) {
// skbug.com/8862
if (src.colorType() == kRGB_888x_SkColorType) {
return Surface();
}
auto surf = SkSurface::MakeRenderTarget(context,
SkBudgeted::kYes,
src.info(),
@ -679,10 +675,6 @@ DEF_GPUTEST_FOR_RENDERING_CONTEXTS(ImageAsyncReadPixels, reporter, ctxInfo) {
for (auto origin : {kTopLeft_GrSurfaceOrigin, kBottomLeft_GrSurfaceOrigin}) {
for (auto renderable : {GrRenderable::kNo, GrRenderable::kYes}) {
auto factory = std::function<GpuSrcFactory<Image>>([&](const SkPixmap& src) {
// skbug.com/8862
if (src.colorType() == kRGB_888x_SkColorType) {
return Image();
}
return sk_gpu_test::MakeBackendTextureImage(ctxInfo.directContext(), src,
renderable, origin);
});
@ -1123,10 +1115,6 @@ DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SurfaceContextWritePixelsMipped, reporter, ct
for (int c = 0; c < kGrColorTypeCnt; ++c) {
auto ct = static_cast<GrColorType>(c);
// skbug.com/8862
if (ct == GrColorType::kRGB_888x) {
continue;
}
// Below we use rendering to read the level pixels back.
auto format = direct->priv().caps()->getDefaultBackendFormat(ct, GrRenderable::kYes);
if (!format.isValid()) {