Ensure |m_image| is (re-)created in ImageResource::didAddClient()

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 97 matching lines @@
 void ImageResource::markClientsAndObserversFinished()
 {
     HashCountedSet<ImageResourceObserver*> observers;
     m_observers.swap(observers);
     for (const auto& it : observers)
         m_finishedObservers.add(it.key, it.value);
 
     Resource::markClientsAndObserversFinished();
 }
 
+void ImageResource::didAddClient(ResourceClient* client)
+{
+    if (m_data && !m_image && !errorOccurred()) {
+        createImage();
+        m_image->setData(m_data, true);

[hajimehoshi, 2016/07/12 08:38:26]

There is same code in addObserver. How about creating a new function for this
(like ensureImage)?

[hiroshige, 2016/07/12 08:58:21]

Done.

+    }
+
+    Resource::didAddClient(client);
+}
+
 void ImageResource::addObserver(ImageResourceObserver* observer)
 {
     willAddClientOrObserver();
 
     m_observers.add(observer);
 
     if (isCacheValidator())
         return;
 
     if (m_data && !m_image && !errorOccurred()) {
@@ skipping 436 matching lines @@
     if (response().wasFetchedViaServiceWorker())
         return response().serviceWorkerResponseType() != WebServiceWorkerResponseTypeOpaque;
     if (!getImage()->currentFrameHasSingleSecurityOrigin())
         return false;
     if (passesAccessControlCheck(securityOrigin))
         return true;
     return !securityOrigin->taintsCanvas(response().url());
 }
 
 } // namespace blink