Index: gm/gmmain.cpp |
diff --git a/gm/gmmain.cpp b/gm/gmmain.cpp |
index b73971441c78e7590a01776597534e213d8fde8b..6d3f619b98900f2b81db968f2fbd7a8d2185e6cd 100644 |
--- a/gm/gmmain.cpp |
+++ b/gm/gmmain.cpp |
@@ -151,7 +151,7 @@ enum ConfigFlags { |
}; |
struct ConfigData { |
- SkBitmap::Config fConfig; |
+ SkColorType fColorType; |
Backend fBackend; |
GLContextType fGLContextType; // GPU backend only |
int fSampleCnt; // GPU backend only |
@@ -477,7 +477,8 @@ public: |
*/ |
static void setup_bitmap(const ConfigData& gRec, SkISize& size, |
SkBitmap* bitmap) { |
- bitmap->allocConfigPixels(gRec.fConfig, size.width(), size.height()); |
+ bitmap->allocPixels(SkImageInfo::Make(size.width(), size.height(), |
+ gRec.fColorType, kPremul_SkAlphaType)); |
bitmap->eraseColor(SK_ColorTRANSPARENT); |
} |
@@ -1287,39 +1288,36 @@ static const GLContextType kDontCare_GLContextType = 0; |
#endif |
static const ConfigData gRec[] = { |
- { SkBitmap::kARGB_8888_Config, kRaster_Backend, kDontCare_GLContextType, 0, kRW_ConfigFlag, "8888", true }, |
-#if 0 // stop testing this (for now at least) since we want to remove support for it (soon please!!!) |
- { SkBitmap::kARGB_4444_Config, kRaster_Backend, kDontCare_GLContextType, 0, kRW_ConfigFlag, "4444", true }, |
-#endif |
- { SkBitmap::kRGB_565_Config, kRaster_Backend, kDontCare_GLContextType, 0, kRW_ConfigFlag, "565", true }, |
+ { kN32_SkColorType, kRaster_Backend, kDontCare_GLContextType, 0, kRW_ConfigFlag, "8888", true }, |
+ { kRGB_565_SkColorType, kRaster_Backend, kDontCare_GLContextType, 0, kRW_ConfigFlag, "565", true }, |
#if SK_SUPPORT_GPU |
- { SkBitmap::kARGB_8888_Config, kGPU_Backend, GrContextFactory::kNative_GLContextType, 0, kRW_ConfigFlag, "gpu", true }, |
- { SkBitmap::kARGB_8888_Config, kGPU_Backend, GrContextFactory::kNative_GLContextType, 16, kRW_ConfigFlag, "msaa16", false}, |
- { SkBitmap::kARGB_8888_Config, kGPU_Backend, GrContextFactory::kNative_GLContextType, 4, kRW_ConfigFlag, "msaa4", false}, |
- { SkBitmap::kARGB_8888_Config, kGPU_Backend, GrContextFactory::kNVPR_GLContextType, 4, kRW_ConfigFlag, "nvprmsaa4", true }, |
- { SkBitmap::kARGB_8888_Config, kGPU_Backend, GrContextFactory::kNVPR_GLContextType, 16, kRW_ConfigFlag, "nvprmsaa16", false}, |
+ { kN32_SkColorType, kGPU_Backend, GrContextFactory::kNative_GLContextType, 0, kRW_ConfigFlag, "gpu", true }, |
+ { kN32_SkColorType, kGPU_Backend, GrContextFactory::kNative_GLContextType, 16, kRW_ConfigFlag, "msaa16", false}, |
+ { kN32_SkColorType, kGPU_Backend, GrContextFactory::kNative_GLContextType, 4, kRW_ConfigFlag, "msaa4", false}, |
+ { kN32_SkColorType, kGPU_Backend, GrContextFactory::kNVPR_GLContextType, 4, kRW_ConfigFlag, "nvprmsaa4", true }, |
+ { kN32_SkColorType, kGPU_Backend, GrContextFactory::kNVPR_GLContextType, 16, kRW_ConfigFlag, "nvprmsaa16", false}, |
/* The gpudebug context does not generate meaningful images, so don't record |
* the images it generates! We only run it to look for asserts. */ |
- { SkBitmap::kARGB_8888_Config, kGPU_Backend, GrContextFactory::kDebug_GLContextType, 0, kNone_ConfigFlag, "gpudebug", kDebugOnly}, |
+ { kN32_SkColorType, kGPU_Backend, GrContextFactory::kDebug_GLContextType, 0, kNone_ConfigFlag, "gpudebug", kDebugOnly}, |
/* The gpunull context does the least amount of work possible and doesn't |
generate meaninful images, so don't record them!. It can be run to |
isolate the CPU-side processing expense from the GPU-side. |
*/ |
- { SkBitmap::kARGB_8888_Config, kGPU_Backend, GrContextFactory::kNull_GLContextType, 0, kNone_ConfigFlag, "gpunull", kDebugOnly}, |
+ { kN32_SkColorType, kGPU_Backend, GrContextFactory::kNull_GLContextType, 0, kNone_ConfigFlag, "gpunull", kDebugOnly}, |
#if SK_ANGLE |
- { SkBitmap::kARGB_8888_Config, kGPU_Backend, GrContextFactory::kANGLE_GLContextType, 0, kRW_ConfigFlag, "angle", true }, |
- { SkBitmap::kARGB_8888_Config, kGPU_Backend, GrContextFactory::kANGLE_GLContextType, 16, kRW_ConfigFlag, "anglemsaa16", true }, |
+ { kN32_SkColorType, kGPU_Backend, GrContextFactory::kANGLE_GLContextType, 0, kRW_ConfigFlag, "angle", true }, |
+ { kN32_SkColorType, kGPU_Backend, GrContextFactory::kANGLE_GLContextType, 16, kRW_ConfigFlag, "anglemsaa16", true }, |
#endif // SK_ANGLE |
#ifdef SK_MESA |
- { SkBitmap::kARGB_8888_Config, kGPU_Backend, GrContextFactory::kMESA_GLContextType, 0, kRW_ConfigFlag, "mesa", true }, |
+ { kN32_SkColorType, kGPU_Backend, GrContextFactory::kMESA_GLContextType, 0, kRW_ConfigFlag, "mesa", true }, |
#endif // SK_MESA |
#endif // SK_SUPPORT_GPU |
#ifdef SK_SUPPORT_XPS |
/* At present we have no way of comparing XPS files (either natively or by converting to PNG). */ |
- { SkBitmap::kARGB_8888_Config, kXPS_Backend, kDontCare_GLContextType, 0, kWrite_ConfigFlag, "xps", true }, |
+ { kN32_SkColorType, kXPS_Backend, kDontCare_GLContextType, 0, kWrite_ConfigFlag, "xps", true }, |
#endif // SK_SUPPORT_XPS |
#ifdef SK_SUPPORT_PDF |
- { SkBitmap::kARGB_8888_Config, kPDF_Backend, kDontCare_GLContextType, 0, kRW_ConfigFlag, "pdf", true }, |
+ { kN32_SkColorType, kPDF_Backend, kDontCare_GLContextType, 0, kRW_ConfigFlag, "pdf", true }, |
#endif // SK_SUPPORT_PDF |
}; |
@@ -1740,7 +1738,7 @@ ErrorCombination run_multiple_configs(GMMain &gmmain, GM *gm, |
} |
if ((gmFlags & GM::kSkip565_Flag) && |
(kRaster_Backend == config.fBackend) && |
- (SkBitmap::kRGB_565_Config == config.fConfig)) { |
+ (kRGB_565_SkColorType == config.fColorType)) { |
gmmain.RecordSkippedTest(shortNamePlusConfig, |
renderModeDescriptor, |
config.fBackend); |