blink: Fix frame completion querying for static images.

third_party/WebKit/Source/platform/image-decoders/jpeg/JPEGImageDecoder.cpp

Index: third_party/WebKit/Source/platform/image-decoders/jpeg/JPEGImageDecoder.cpp
diff --git a/third_party/WebKit/Source/platform/image-decoders/jpeg/JPEGImageDecoder.cpp b/third_party/WebKit/Source/platform/image-decoders/jpeg/JPEGImageDecoder.cpp
index 12e24974e53f5e6a20da6b5f11614fdf1397ab50..7e2aea53f3404b413f3a986dbc46d62d5995a37d 100644
--- a/third_party/WebKit/Source/platform/image-decoders/jpeg/JPEGImageDecoder.cpp
+++ b/third_party/WebKit/Source/platform/image-decoders/jpeg/JPEGImageDecoder.cpp
@@ -992,7 +992,7 @@ inline bool IsComplete(const JPEGImageDecoder* decoder, bool only_size) {
   if (decoder->HasImagePlanes() && !only_size)
     return true;
 
-  return decoder->FrameIsCompleteAtIndex(0);
+  return decoder->FrameIsDecodedAtIndex(0);

[scroggo_chromium, 2017/05/30 15:51:34]

This is the only caller of the new API, and this method is in turn only called
by JPEGImageDecoder, which has access to the internal info needed by this
method. Maybe we should delete this method and have JPEGImageDecoder check
directly? To keep the clarity of this inline method, we could make this a
private method on JPEGImageDecoder:

JPEGImageDecoder::IsComplete(bool only_size) const {
  if (HasImagePlanes() && !only_size)
    return true;
  return frame_buffer_cache_[0].GetStatus() == ImageFrame::kFrameComplete;
}

[scroggo_chromium, 2017/07/17 19:25:35]

Uploaded crrev.com/2984463002

 }
 
 void JPEGImageDecoder::Decode(bool only_size) {