ImageResource: Reset the encoded image size after receiving all data

third_party/WebKit/Source/core/fetch/ImageResource.cpp

 /*
     Copyright (C) 1998 Lars Knoll (knoll@mpi-hd.mpg.de)
     Copyright (C) 2001 Dirk Mueller (mueller@kde.org)
     Copyright (C) 2002 Waldo Bastian (bastian@kde.org)
     Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
     Copyright (C) 2004, 2005, 2006, 2007 Apple Inc. All rights reserved.
 
     This library is free software; you can redistribute it and/or
     modify it under the terms of the GNU Library General Public
     License as published by the Free Software Foundation; either
@@ skipping 402 matching lines @@
     if (data())
       updateImageAndClearBuffer();
   } else {
     updateImage(true);
     // As encoded image data can be created from m_image  (see
     // ImageResource::resourceBuffer(), we don't have to keep m_data. Let's
     // clear this. As for the lifetimes of m_image and m_data, see this
     // document:
     // https://docs.google.com/document/d/1v0yTAZ6wkqX2U_M6BNIGUJpM1s0TIw1VsqpxoL7aciY/edit?usp=sharing
     clearData();
+    setEncodedSize(0);

[haraken, 2016/10/05 05:59:39]

Would it be better to move setEncodedSize into clearData?

[hajimehoshi, 2016/10/05 06:04:31]

I agree but hiroshige@ will do this in another CL. That would not be simple
according to hiroshige@.

   }
   Resource::finish(loadFinishTime);
 }
 
 void ImageResource::onMemoryDump(WebMemoryDumpLevelOfDetail levelOfDetail,
                                  WebProcessMemoryDump* memoryDump) const {
   Resource::onMemoryDump(levelOfDetail, memoryDump);
   const String name = getMemoryDumpName() + "/encoded_data";
   if (m_image && !m_image->isNull())
     m_image->data()->onMemoryDump(name, memoryDump);
@@ skipping 153 matching lines @@
     return response().serviceWorkerResponseType() !=
            WebServiceWorkerResponseTypeOpaque;
   if (!getImage()->currentFrameHasSingleSecurityOrigin())
     return false;
   if (passesAccessControlCheck(securityOrigin))
     return true;
   return !securityOrigin->taintsCanvas(response().url());
 }
 
 }  // namespace blink