diff options
author | reed <reed@google.com> | 2016-10-31 05:27:28 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-10-31 05:27:28 -0700 |
commit | 9e2ed83f9b29d91058ba411a4137d099877a48b2 (patch) | |
tree | a21292f7ce0a99859e80a8476ef3a571f2c247d9 | |
parent | 2f7068aec9906168b3c142b5057e098114376cc9 (diff) |
add isAlphaOnly() query
BUG=skia:5904
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2461773002
TBR=
Review-Url: https://codereview.chromium.org/2461773002
-rw-r--r-- | include/core/SkImage.h | 5 | ||||
-rw-r--r-- | src/image/SkImage.cpp | 4 | ||||
-rw-r--r-- | tests/ImageIsOpaqueTest.cpp | 31 |
3 files changed, 40 insertions, 0 deletions
diff --git a/include/core/SkImage.h b/include/core/SkImage.h index b67f0a3d09..290996e0a9 100644 --- a/include/core/SkImage.h +++ b/include/core/SkImage.h @@ -167,6 +167,11 @@ public: SkIRect bounds() const { return SkIRect::MakeWH(fWidth, fHeight); } uint32_t uniqueID() const { return fUniqueID; } SkAlphaType alphaType() const; + + /** + * Returns true fi the image will be drawn as a mask, with no intrinsic color of its own. + */ + bool isAlphaOnly() const; bool isOpaque() const { return SkAlphaTypeIsOpaque(this->alphaType()); } /** diff --git a/src/image/SkImage.cpp b/src/image/SkImage.cpp index 09f7ecc4cf..986fdec3a4 100644 --- a/src/image/SkImage.cpp +++ b/src/image/SkImage.cpp @@ -371,6 +371,10 @@ bool SkImage::isLazyGenerated() const { return as_IB(this)->onIsLazyGenerated(); } +bool SkImage::isAlphaOnly() const { + return as_IB(this)->onImageInfo().colorType() == kAlpha_8_SkColorType; +} + ////////////////////////////////////////////////////////////////////////////////////// #if !SK_SUPPORT_GPU diff --git a/tests/ImageIsOpaqueTest.cpp b/tests/ImageIsOpaqueTest.cpp index ab995c8140..cc7255d37a 100644 --- a/tests/ImageIsOpaqueTest.cpp +++ b/tests/ImageIsOpaqueTest.cpp @@ -102,3 +102,34 @@ DEF_GPUTEST_FOR_RENDERING_CONTEXTS(ImageIsOpaqueTest_Gpu, reporter, ctxInfo) { } #endif + +/////////////////////////////////////////////////////////////////////////////////////////////////// +#include "SkPictureRecorder.h" + +static sk_sp<SkPicture> make_picture() { + SkPictureRecorder recorder; + SkCanvas* canvas = recorder.beginRecording({ 0, 0, 10, 10 }); + canvas->drawColor(SK_ColorRED); + return recorder.finishRecordingAsPicture(); +} + +DEF_TEST(Image_isAlphaOnly, reporter) { + SkPMColor pmColors = 0; + SkPixmap pmap = { + SkImageInfo::MakeN32Premul(1, 1), + &pmColors, + sizeof(pmColors) + }; + for (auto& image : { + SkImage::MakeRasterCopy(pmap), + GetResourceAsImage("mandrill_128.png"), + GetResourceAsImage("color_wheel.jpg"), + SkImage::MakeFromPicture(make_picture(), { 10, 10 }, nullptr, nullptr), + }) + { + REPORTER_ASSERT(reporter, image->isAlphaOnly() == false); + } + + REPORTER_ASSERT(reporter, SkImage::MakeRasterCopy({ + SkImageInfo::MakeA8(1, 1), (uint8_t*)&pmColors, 1})->isAlphaOnly() == true); +} |