diff options
author | Matt Sarett <msarett@google.com> | 2016-12-08 08:32:44 -0500 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2016-12-08 15:40:15 +0000 |
commit | a9da22df9606559424853cab547784aa73b11534 (patch) | |
tree | 81ad442f8176c51f796e5956c9f5353a038674f9 /gm/bitmapimage.cpp | |
parent | 2979b1a7eaffcdcf994e506caf743a9adf35b8ed (diff) |
Fix bitmap-image srgb gm
This changed behavior when asLegacyBitmap() was modified to always
initiate a decode in legacy mode.
This fixes the gm to work as originally intended.
BUG=skia:
Change-Id: I34d35f6f445bd4606c6e28f83905702ec0b36bbb
Reviewed-on: https://skia-review.googlesource.com/5696
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Matt Sarett <msarett@google.com>
Diffstat (limited to 'gm/bitmapimage.cpp')
-rw-r--r-- | gm/bitmapimage.cpp | 26 |
1 files changed, 15 insertions, 11 deletions
diff --git a/gm/bitmapimage.cpp b/gm/bitmapimage.cpp index ad6a74dda8..8c37d5da44 100644 --- a/gm/bitmapimage.cpp +++ b/gm/bitmapimage.cpp @@ -7,6 +7,7 @@ #include "gm.h" #include "Resources.h" +#include "SkCodec.h" namespace skiagm { @@ -17,7 +18,7 @@ public: protected: SkString onShortName() override { - return SkString("bitmap-image-srgb-linear"); + return SkString("bitmap-image-srgb-legacy"); } SkISize onISize() override { @@ -26,26 +27,29 @@ protected: void onDraw(SkCanvas* canvas) override { // Create image. - sk_sp<SkImage> image = GetResourceAsImage("mandrill_512_q075.jpg"); + const char* path = "mandrill_512_q075.jpg"; + sk_sp<SkImage> image = GetResourceAsImage(path); if (!image) { SkDebugf("Failure: Is the resource path set properly?"); return; } // Create matching bitmap. + std::unique_ptr<SkCodec> codec(SkCodec::NewFromStream(GetResourceAsStream(path))); SkBitmap bitmap; - SkAssertResult(image->asLegacyBitmap(&bitmap, SkImage::kRO_LegacyBitmapMode)); + bitmap.allocPixels(codec->getInfo()); + codec->getPixels(codec->getInfo(), bitmap.getPixels(), bitmap.rowBytes()); // The GM will be displayed in a 2x2 grid. - // The top two squares show an sRGB image, then bitmap, drawn to a linear canvas. + // The top two squares show an sRGB image, then bitmap, drawn to a legacy canvas. SkImageInfo linearInfo = SkImageInfo::MakeN32(2*kSize, kSize, kOpaque_SkAlphaType); - SkBitmap linearBMCanvas; - linearBMCanvas.allocPixels(linearInfo); - SkCanvas linearCanvas(linearBMCanvas); - linearCanvas.drawImage(image, 0.0f, 0.0f, nullptr); - linearCanvas.translate(SkScalar(kSize), 0.0f); - linearCanvas.drawBitmap(bitmap, 0.0f, 0.0f, nullptr); - canvas->drawBitmap(linearBMCanvas, 0.0f, 0.0f, nullptr); + SkBitmap legacyBMCanvas; + legacyBMCanvas.allocPixels(linearInfo); + SkCanvas legacyCanvas(legacyBMCanvas); + legacyCanvas.drawImage(image, 0.0f, 0.0f, nullptr); + legacyCanvas.translate(SkScalar(kSize), 0.0f); + legacyCanvas.drawBitmap(bitmap, 0.0f, 0.0f, nullptr); + canvas->drawBitmap(legacyBMCanvas, 0.0f, 0.0f, nullptr); canvas->translate(0.0f, SkScalar(kSize)); // The bottom two squares show an sRGB image, then bitmap, drawn to a srgb canvas. |