[WeakMemoryCache] Remove LRU lists, prune order control and live/dead distinction

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 462 matching lines @@
 }
 
 void ImageResource::decodedSizeChangedTo(const blink::Image* image,
                                          size_t newSize) {
   if (!image || image != m_image)
     return;
 
   setDecodedSize(newSize);
 }
 
 void ImageResource::didDraw(const blink::Image* image) {

[Nate Chapin, 2016/10/21 18:48:20]

Looks like this callback is ignored by all ImageObserver subclasses. We should
get rid fo it.

[hiroshige, 2016/10/24 10:35:28]

Created a separate patch: https://codereview.chromium.org/2447623002/
(to prevent this CL from becoming larger)

-  if (!image || image != m_image)
-    return;
-  // decodedSize() == 0 indicates that the image is decoded into
-  // DiscardableMemory, not in MemoryCache. So we don't need to call
-  // Resource::didAccessDecodedData() to update MemoryCache.
-  if (decodedSize() != 0)
-    Resource::didAccessDecodedData();
 }
 
 bool ImageResource::shouldPauseAnimation(const blink::Image* image) {
   if (!image || image != m_image)
     return false;
 
   for (auto* observer : m_finishedObservers.asVector()) {
     if (m_finishedObservers.contains(observer) && observer->willRenderImage())
       return false;
   }
@@ skipping 112 matching lines @@
            WebServiceWorkerResponseTypeOpaque;
   }
   if (!getImage()->currentFrameHasSingleSecurityOrigin())
     return false;
   if (passesAccessControlCheck(securityOrigin))
     return true;
   return !securityOrigin->taintsCanvas(response().url());
 }
 
 }  // namespace blink