Don't crash when an image fails to decode.

Don't crash when an image fails to decode.

The code that extracts the image for the default frame from an animated
bitmap should check that the image data is actually available.
It is possible for decoding to fail for invalid frames, Blink should not
crash when this happens.

BUG=408026
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=182496

[ckulakowski, 2014-09-03 12:08:32]

ckulakowski@opera.com changed reviewers:
+ junov@chromium.org, piotaixr@chromium.org

[Justin Novosad, 2014-09-05 15:15:33]

On 2014/09/05 08:28:26, ckulakowski wrote:

lgtm

[ckulakowski, 2014-09-22 06:27:45]

The CQ bit was checked by ckulakowski@opera.com

[commit-bot: I haz the power, 2014-09-22 06:28:13]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/535823002/1

[commit-bot: I haz the power, 2014-09-22 06:28:20]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-22 06:28:21]

Failed to apply patch for Source/platform/graphics/BitmapImage.cpp:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Source/platform/graphics/BitmapImage.cpp
  Hunk #1 FAILED at 370.
  1 out of 1 hunk FAILED -- saving rejects to file
Source/platform/graphics/BitmapImage.cpp.rej

Patch:       Source/platform/graphics/BitmapImage.cpp
Index: Source/platform/graphics/BitmapImage.cpp
diff --git a/Source/platform/graphics/BitmapImage.cpp
b/Source/platform/graphics/BitmapImage.cpp
index
be13c72e967d2576fef161acff8bd3f967ff9c82..0864659f869b72d43a9b6b083a36ce607c446666
100644
--- a/Source/platform/graphics/BitmapImage.cpp
+++ b/Source/platform/graphics/BitmapImage.cpp
@@ -370,7 +370,7 @@ PassRefPtr<NativeImageSkia>
BitmapImage::nativeImageForCurrentFrame()
 
 PassRefPtr<Image> BitmapImage::imageForDefaultFrame()
 {
-    if (isAnimated())
+    if (isAnimated() && frameAtIndex(0))
         return BitmapImage::create(frameAtIndex(0));
 
     return Image::imageForDefaultFrame();

[ckulakowski, 2014-09-23 12:27:08]

The CQ bit was checked by ckulakowski@opera.com

[commit-bot: I haz the power, 2014-09-23 12:28:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/535823002/20001

[commit-bot: I haz the power, 2014-09-23 13:29:09]

Committed patchset #2 (id:20001) as 182496