Save a bitmap copy when advancing to dependent GIF and WebP animation frames

third_party/WebKit/Source/platform/exported/WebImage.cpp

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 57 matching lines @@
         if (frameArea < (desiredSize.width * desiredSize.height))
             break; // No more frames that are large enough.
 
         if (!i || (frameArea < frameAreaAtIndex)) {
             index = i; // Closer to desired area than previous best match.
             frameAreaAtIndex = frameArea;
         }
     }
 
     ImageFrame* frame = decoder->frameBufferAtIndex(index);
-    if (!frame)
-        return WebImage();
-
-    return WebImage(frame->bitmap());
+    return (frame && !decoder->failed()) ? WebImage(frame->bitmap()) : WebImage();
 }
 
 WebVector<WebImage> WebImage::framesFromData(const WebData& data)
 {
     // This is to protect from malicious images. It should be big enough that it's never hit in pracice.
     const size_t maxFrameCount = 8;
 
     RefPtr<SharedBuffer> buffer = PassRefPtr<SharedBuffer>(data);
     std::unique_ptr<ImageDecoder> decoder(ImageDecoder::create(buffer, true,
         ImageDecoder::AlphaPremultiplied, ImageDecoder::GammaAndColorProfileIgnored));
     if (!decoder || !decoder->isSizeAvailable())
         return WebVector<WebImage>();
 
     // Frames are arranged by decreasing size, then decreasing bit depth.
     // Keep the first frame at every size, has the highest bit depth.
     const size_t frameCount = decoder->frameCount();
     IntSize lastSize;
 
     Vector<WebImage> frames;
     for (size_t i = 0; i < std::min(frameCount, maxFrameCount); ++i) {
         const IntSize frameSize = decoder->frameSizeAtIndex(i);
         if (frameSize == lastSize)
             continue;
         lastSize = frameSize;
 
         ImageFrame* frame = decoder->frameBufferAtIndex(i);
         if (!frame)
             continue;
 
-        const SkBitmap& bitmap = frame->bitmap();
-        if (!bitmap.isNull() && bitmap.isImmutable())
+        SkBitmap bitmap = frame->bitmap();
+        if (!bitmap.isNull() && frame->getStatus() == ImageFrame::FrameComplete)
             frames.append(WebImage(bitmap));
     }
 
     return frames;
 }
 
 void WebImage::reset()
 {
     m_bitmap.reset();
 }
@@ skipping 16 matching lines @@
 WebImage::WebImage(PassRefPtr<Image> image)
 {
     if (!image)
         return;
 
     if (sk_sp<SkImage> skImage = image->imageForCurrentFrame())
         skImage->asLegacyBitmap(&m_bitmap, SkImage::kRO_LegacyBitmapMode);
 }
 
 } // namespace blink