aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/images
diff options
context:
space:
mode:
authorGravatar Matt Sarett <msarett@google.com>2016-10-26 17:39:54 -0400
committerGravatar Skia Commit-Bot <skia-commit-bot@chromium.org>2016-10-26 22:03:10 +0000
commit18a11a6f374fe430c8736c53ff4f8bc3c72cd54b (patch)
tree59a34d91320f71d6d1c83808e4bee00194d28ee9 /src/images
parentcfc4f44aebdd560c600991ad900a4f07c3958c8f (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.cpp2
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;
}