Stop dispatching notifyFinished for each part in a multipart response

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 447 matching lines @@
     load(fetcher, fetcher->defaultResourceOptions());
 }
 
 void ImageResource::changedInRect(const blink::Image* image, const IntRect& rect)
 {
     if (!image || image != m_image)
         return;
     notifyObservers(&rect);
 }
 
-void ImageResource::onePartInMultipartReceived(const ResourceResponse& response, bool isFirstPart)
+void ImageResource::onePartInMultipartReceived(const ResourceResponse& response)
 {
     ASSERT(m_multipartParser);
+    ++m_numParsedPartsInMultipart;
+
     m_response = response;
-    if (m_data) {
-        clear();
-        updateImage(true);
-        m_data.clear();
+    if (m_numParsedPartsInMultipart == 1) {
+        // We have nothing to do because we don't have any data.
+        return;
+    }
+    clear();

[hiroshige, 2016/03/15 21:38:37]

nit: this CL removes |if (m_data)| check. Is this intentional?

+    updateImage(true);
+    m_data.clear();
+
+    if (m_numParsedPartsInMultipart == 2) {
+        // Notify finished when the first part ends.
         setLoading(false);
         checkNotify();
+        if (m_loader)
+            m_loader->didFinishLoadingOnePart(0, WebURLLoaderClient::kUnknownEncodedDataLength);
     }
-    if (!isFirstPart && m_loader)
-        m_loader->didFinishLoadingOnePart(0, WebURLLoaderClient::kUnknownEncodedDataLength);
 }
 
 void ImageResource::multipartDataReceived(const char* bytes, size_t size)
 {
     ASSERT(m_multipartParser);
     Resource::appendData(bytes, size);
 }
 
 bool ImageResource::isAccessAllowed(SecurityOrigin* securityOrigin)
 {
     if (response().wasFetchedViaServiceWorker())
         return response().serviceWorkerResponseType() != WebServiceWorkerResponseTypeOpaque;
     if (!getImage()->currentFrameHasSingleSecurityOrigin())
         return false;
     if (passesAccessControlCheck(securityOrigin))
         return true;
     return !securityOrigin->taintsCanvas(response().url());
 }
 
 } // namespace blink