Revert "Add idea of DataType to SkYUVAPixmapInfo."
This reverts commit ed6344458750c0389b7ccce588b96d0c25397b19.
Reason for revert: trying to unblock android roller, which incorrectly thinks this has a merge conflict: https://b.corp.google.com/issues/167576324
Original change's description:
> Add idea of DataType to SkYUVAPixmapInfo.
>
> DataType describes the data type of YUVA channels
> independent of how they are grouped into planes.
>
> Adds mapping functions between SkColorType/channel count
> and DataType.
>
> SkYUVAPixmapInfo can be constructed from DataType and will
> choose appropriate SkColorTypes for each plane.
>
> Valid SkYUVAPixmapInfos now have the same DataType for each
> plane (could relax this in the future, esp for alpha plane).
>
> SkYUVAPixmapInfo::SupportedDataTypes specifies the supported
> combinations of SkYUVAInfo::PlanarConfig and
> kYUVAPixmapInfo::DataType supported by a GrContext (based on
> supported texture formats).
>
> SkImageGenerator/SkCodec YUVA query API now takes a
> SupportedDataTypes.
>
> Change-Id: I8791234638e6ba3396d1e7960b7bc210edc6dd57
> Bug: skia:10632
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/314276
> Reviewed-by: Leon Scroggins <scroggo@google.com>
> Reviewed-by: Robert Phillips <robertphillips@google.com>
> Commit-Queue: Brian Salomon <bsalomon@google.com>
TBR=bsalomon@google.com,robertphillips@google.com,scroggo@google.com
Change-Id: I72c39539a4766f10cac3ca3cdef6c503a8319ff1
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:10632
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/314895
Reviewed-by: Michael Ludwig <michaelludwig@google.com>
Commit-Queue: Michael Ludwig <michaelludwig@google.com>
diff --git a/src/codec/SkJpegCodec.cpp b/src/codec/SkJpegCodec.cpp
index d7b3974..53b5cae 100644
--- a/src/codec/SkJpegCodec.cpp
+++ b/src/codec/SkJpegCodec.cpp
@@ -745,7 +745,6 @@
static bool is_yuv_supported(const jpeg_decompress_struct* dinfo,
const SkJpegCodec& codec,
- const SkYUVAPixmapInfo::SupportedDataTypes* supportedDataTypes,
SkYUVAPixmapInfo* yuvaPixmapInfo) {
// Scaling is not supported in raw data mode.
SkASSERT(dinfo->scale_num == dinfo->scale_denom);
@@ -812,10 +811,6 @@
} else {
return false;
}
- if (supportedDataTypes &&
- !supportedDataTypes->supported(tempPlanarConfig, SkYUVAPixmapInfo::DataType::kUnorm8)) {
- return false;
- }
if (yuvaPixmapInfo) {
SkColorType colorTypes[SkYUVAPixmapInfo::kMaxPlanes];
size_t rowBytes[SkYUVAPixmapInfo::kMaxPlanes];
@@ -834,16 +829,15 @@
return true;
}
-bool SkJpegCodec::onQueryYUVAInfo(const SkYUVAPixmapInfo::SupportedDataTypes& supportedDataTypes,
- SkYUVAPixmapInfo* yuvaPixmapInfo) const {
+bool SkJpegCodec::onQueryYUVAInfo(SkYUVAPixmapInfo* yuvaPixmapInfo) const {
jpeg_decompress_struct* dinfo = fDecoderMgr->dinfo();
- return is_yuv_supported(dinfo, *this, &supportedDataTypes, yuvaPixmapInfo);
+ return is_yuv_supported(dinfo, *this, yuvaPixmapInfo);
}
SkCodec::Result SkJpegCodec::onGetYUVAPlanes(const SkYUVAPixmaps& yuvaPixmaps) {
// Get a pointer to the decompress info since we will use it quite frequently
jpeg_decompress_struct* dinfo = fDecoderMgr->dinfo();
- if (!is_yuv_supported(dinfo, *this, nullptr, nullptr)) {
+ if (!is_yuv_supported(dinfo, *this, nullptr)) {
return fDecoderMgr->returnFailure("onGetYUVAPlanes", kInvalidInput);
}
// Set the jump location for libjpeg errors
@@ -866,7 +860,7 @@
// was caused by a bug in the old code, but we'll be safe and check here.
// Also check that pixmap properties agree with expectations.
SkYUVAPixmapInfo info;
- SkASSERT(is_yuv_supported(dinfo, *this, nullptr, &info));
+ SkASSERT(is_yuv_supported(dinfo, *this, &info));
SkASSERT(info.yuvaInfo() == yuvaPixmaps.yuvaInfo());
for (int i = 0; i < info.numPlanes(); ++i) {
SkASSERT(planes[i].colorType() == kAlpha_8_SkColorType);