diff options
author | Matt Sarett <msarett@google.com> | 2016-10-26 17:39:54 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2016-10-26 22:03:10 +0000 |
commit | 18a11a6f374fe430c8736c53ff4f8bc3c72cd54b (patch) | |
tree | 59a34d91320f71d6d1c83808e4bee00194d28ee9 /src/images | |
parent | cfc4f44aebdd560c600991ad900a4f07c3958c8f (diff) |
Speculative fix for flaky webp encode/decode on Nexus 5
I'm not really expecting this to work anymore...
The docs don't say anything about the rgb pointer needing
to stay valid, and it looks like we are done with it
after the call to import.
Still, worth a try I think.
BUG=skia:5876
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4024
Change-Id: I6e65ef96a2b661b01eb92bedab0acd73ed4f6218
Reviewed-on: https://skia-review.googlesource.com/4024
Reviewed-by: Mike Klein <mtklein@chromium.org>
Commit-Queue: Matt Sarett <msarett@google.com>
Diffstat (limited to 'src/images')
-rw-r--r-- | src/images/SkWEBPImageEncoder.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/images/SkWEBPImageEncoder.cpp b/src/images/SkWEBPImageEncoder.cpp index 116608a253..6ae79efd0f 100644 --- a/src/images/SkWEBPImageEncoder.cpp +++ b/src/images/SkWEBPImageEncoder.cpp @@ -227,10 +227,10 @@ bool SkWEBPImageEncoder::onEncode(SkWStream* stream, const SkBitmap& bm, } else { ok = SkToBool(WebPPictureImportRGBA(&pic, rgb, rgbStride)); } - delete[] rgb; ok = ok && WebPEncode(&webp_config, &pic); WebPPictureFree(&pic); + delete[] rgb; return ok; } |