Move Image ownership out of CSSValue

Source/core/css/CSSImageValue.cpp

 /*
  * (C) 1999-2003 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2008 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
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Library General Public License for more details.
  *
  * You should have received a copy of the GNU Library General Public License
  * along with this library; see the file COPYING.LIB.  If not, write to
  * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
  * Boston, MA 02110-1301, USA.
  */
 
 #include "config.h"
 #include "core/css/CSSImageValue.h"
 
 #include "core/css/CSSMarkup.h"
+#include "core/css/resolver/ElementStyleResources.h"
 #include "core/dom/Document.h"
 #include "core/fetch/FetchInitiatorTypeNames.h"
 #include "core/fetch/FetchRequest.h"
 #include "core/fetch/ImageResource.h"
 #include "core/loader/MixedContentChecker.h"
 #include "core/style/StyleFetchedImage.h"
 #include "core/style/StylePendingImage.h"
 #include "platform/weborigin/KURL.h"
 #include "platform/weborigin/SecurityPolicy.h"
 
 namespace blink {
 
-CSSImageValue::CSSImageValue(const String& rawValue, const KURL& url, StyleImage* image)
+CSSImageValue::CSSImageValue(const String& rawValue, const String& url, StyleImage* image)
     : CSSValue(ImageClass)
     , m_relativeURL(rawValue)
-    , m_absoluteURL(url.string())
+    , m_absoluteURL(url)
     , m_image(image)
     , m_accessedImage(image)
 {
+    printf("** Made a CSSImageValue: %s, %s, image? %d, accessed? %d\n", m_relativeURL.ascii().data(), m_absoluteURL.ascii().data(), (bool)m_image, m_accessedImage);
 }
 
 CSSImageValue::~CSSImageValue()
 {
+    printf("** Freed a CSSImageValue: %s, %s, image? %d, accessed? %d\n", m_relativeURL.ascii().data(), m_absoluteURL.ascii().data(), (bool)m_image, m_accessedImage);
 }
 
+
 StyleImage* CSSImageValue::cachedOrPendingImage()
 {
+    printf("Ahh old cachedOrPendingImage() function\n");
     if (!m_image)
         m_image = StylePendingImage::create(this);
 
     return m_image.get();
 }
 
-StyleFetchedImage* CSSImageValue::cachedImage(Document* document, const ResourceLoaderOptions& options)
+
+StyleImage* CSSImageValue::cachedOrPendingImage(CSSStyleImageMap& imageMap)
 {
+    RefPtrWillBeMember<StyleImage>& value = imageMap.getStyleImageEntry(this).storedValue->value;
+    if (!value)
+        value = StylePendingImage::create(this);
+    return value.get();
+}
+
+StyleFetchedImage* CSSImageValue::cachedImage(Document* document, const ResourceLoaderOptions& options, CSSStyleImageMap& imageMap)
+{
+    printf("Yay new cachedImage() function\n");
     ASSERT(document);
 
+    RefPtrWillBeMember<StyleImage>& value = imageMap.getStyleImageEntry(this).storedValue->value;
     if (!m_accessedImage) {
         m_accessedImage = true;
 
         FetchRequest request(ResourceRequest(m_absoluteURL), m_initiatorName.isEmpty() ? FetchInitiatorTypeNames::css : m_initiatorName, options);
         request.mutableResourceRequest().setHTTPReferrer(SecurityPolicy::generateReferrer(m_referrer.referrerPolicy, request.url(), m_referrer.referrer));
 
         if (options.corsEnabled == IsCORSEnabled)
             request.setCrossOriginAccessControl(document->securityOrigin(), options.allowCredentials, options.credentialsRequested);
 
+        printf("Downloading image from %s.\n", m_absoluteURL.ascii().data());
         if (ResourcePtr<ImageResource> cachedImage = ImageResource::fetch(request, document->fetcher()))
-            m_image = StyleFetchedImage::create(cachedImage.get(), document);
+            value = StyleFetchedImage::create(cachedImage.get(), document);
     }
 
-    return (m_image && m_image->isImageResource()) ? toStyleFetchedImage(m_image) : 0;
+    return (value && value->isImageResource()) ? toStyleFetchedImage(value) : 0;
 }
 
-void CSSImageValue::restoreCachedResourceIfNeeded(Document& document)
+void CSSImageValue::restoreCachedResourceIfNeeded(Document& document, CSSStyleImageMap& imageMap)
 {
-    if (!m_accessedImage || !m_image->isImageResource() || !document.fetcher())
+    RefPtrWillBeMember<StyleImage>& value = imageMap.getStyleImageEntry(this).storedValue->value;
+    if (!m_accessedImage || !value->isImageResource() || !document.fetcher())
         return;
     if (document.fetcher()->cachedResource(KURL(ParsedURLString, m_absoluteURL)))
         return;
 
-    ImageResource* resource = m_image->cachedImage();
+    ImageResource* resource = value->cachedImage();
     if (!resource)
         return;
 
     FetchRequest request(ResourceRequest(m_absoluteURL), m_initiatorName.isEmpty() ? FetchInitiatorTypeNames::css : m_initiatorName, resource->options());
     MixedContentChecker::shouldBlockFetch(document.frame(), resource->lastResourceRequest(),
         resource->lastResourceRequest().url(), MixedContentChecker::SendReport);
     document.fetcher()->requestLoadStarted(resource, request, ResourceFetcher::ResourceLoadingFromCache);
 }
 
 bool CSSImageValue::hasFailedOrCanceledSubresources() const
 {
+    printf("Oh no! In hasFailedOrCancelledSubresources\n");
     if (!m_image || !m_image->isImageResource())
         return false;
     if (Resource* cachedResource = toStyleFetchedImage(m_image)->cachedImage())
         return cachedResource->loadFailedOrCanceled();
     return true;
 }
 
 bool CSSImageValue::equals(const CSSImageValue& other) const
 {
     return m_absoluteURL == other.m_absoluteURL;
 }
 
 String CSSImageValue::customCSSText() const
 {
     return "url(" + quoteCSSURLIfNeeded(m_absoluteURL) + ")";
 }
 
 bool CSSImageValue::knownToBeOpaque(const LayoutObject* layoutObject) const
 {
     return m_image ? m_image->knownToBeOpaque(layoutObject) : false;
 }
 
 DEFINE_TRACE_AFTER_DISPATCH(CSSImageValue)
 {
     visitor->trace(m_image);
     CSSValue::traceAfterDispatch(visitor);
 }
 
+/*
 void CSSImageValue::reResolveURL(const Document& document)
 {
     KURL url = document.completeURL(m_relativeURL);
     if (url == m_absoluteURL)
         return;
     m_absoluteURL = url.string();
     m_accessedImage = false;
     m_image.clear();
 }
+*/
 
 } // namespace blink