Replace RawPtr with RefPtr on StylePendingImage

third_party/WebKit/Source/core/css/CSSImageSetValue.cpp

 /*
  * Copyright (C) 2012 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 17 matching lines @@
 
 #include "core/css/CSSImageValue.h"
 #include "core/css/CSSPrimitiveValue.h"
 #include "core/dom/Document.h"
 #include "core/fetch/FetchInitiatorTypeNames.h"
 #include "core/fetch/FetchRequest.h"
 #include "core/fetch/ImageResource.h"
 #include "core/fetch/ResourceFetcher.h"
 #include "core/fetch/ResourceLoaderOptions.h"
 #include "core/style/StyleFetchedImageSet.h"
-#include "core/style/StylePendingImage.h"
 #include "platform/weborigin/KURL.h"
 #include "platform/weborigin/SecurityPolicy.h"
 #include "wtf/text/StringBuilder.h"
 
 namespace blink {
 
 CSSImageSetValue::CSSImageSetValue()
     : CSSValueList(ImageSetClass, CommaSeparator)
-    , m_accessedBestFitImage(false)
-    , m_scaleFactor(1)
+    , m_isCachePending(true)
+    , m_cachedScaleFactor(1)
 {
 }
 
 CSSImageSetValue::~CSSImageSetValue()
 {
 #if !ENABLE(OILPAN)
-    if (m_imageSet && m_imageSet->isImageResourceSet())
-        toStyleFetchedImageSet(m_imageSet)->clearImageSetValue();
+    if (m_cachedImageSet && m_cachedImageSet->isImageResourceSet())
+        toStyleFetchedImageSet(m_cachedImageSet)->clearImageSetValue();
 #endif
 }
 
 void CSSImageSetValue::fillImageSet()
 {
     size_t length = this->length();
     size_t i = 0;
     while (i < length) {
         CSSImageValue* imageValue = toCSSImageValue(item(i));
         String imageURL = imageValue->url();
 
         ++i;
         ASSERT_WITH_SECURITY_IMPLICATION(i < length);
         CSSValue* scaleFactorValue = item(i);
         float scaleFactor = toCSSPrimitiveValue(scaleFactorValue)->getFloatValue();
 
         ImageWithScale image;
         image.imageURL = imageURL;
         image.referrer = SecurityPolicy::generateReferrer(imageValue->referrer().referrerPolicy, KURL(ParsedURLString, imageURL), imageValue->referrer().referrer);
         image.scaleFactor = scaleFactor;
         m_imagesInSet.append(image);
         ++i;
     }
 
     // Sort the images so that they are stored in order from lowest resolution to highest.
     std::sort(m_imagesInSet.begin(), m_imagesInSet.end(), CSSImageSetValue::compareByScaleFactor);
 }
 
-CSSImageSetValue::ImageWithScale CSSImageSetValue::bestImageForScaleFactor()
+CSSImageSetValue::ImageWithScale CSSImageSetValue::bestImageForScaleFactor(float scaleFactor)
 {
     ImageWithScale image;
     size_t numberOfImages = m_imagesInSet.size();
     for (size_t i = 0; i < numberOfImages; ++i) {
         image = m_imagesInSet.at(i);
-        if (image.scaleFactor >= m_scaleFactor)
+        if (image.scaleFactor >= scaleFactor)
             return image;
     }
     return image;
 }
 
-StyleFetchedImageSet* CSSImageSetValue::cachedImageSet(Document* document, float deviceScaleFactor, const ResourceLoaderOptions& options)
+bool CSSImageSetValue::isCachePending(float deviceScaleFactor) const
+{
+    return m_isCachePending || deviceScaleFactor != m_cachedScaleFactor;
+}
+
+StyleImage* CSSImageSetValue::cachedImageSet(float deviceScaleFactor)
+{
+    ASSERT(!isCachePending(deviceScaleFactor));
+    return m_cachedImageSet.get();
+}
+
+StyleFetchedImageSet* CSSImageSetValue::cacheImageSet(Document* document, float deviceScaleFactor, const ResourceLoaderOptions& options)
 {
     ASSERT(document);
 
-    m_scaleFactor = deviceScaleFactor;
-
     if (!m_imagesInSet.size())
         fillImageSet();
 
-    if (!m_accessedBestFitImage) {
+    if (m_isCachePending || deviceScaleFactor != m_cachedScaleFactor) {
         // FIXME: In the future, we want to take much more than deviceScaleFactor into acount here.
         // All forms of scale should be included: Page::pageScaleFactor(), LocalFrame::pageZoomFactor(),
         // and any CSS transforms. https://bugs.webkit.org/show_bug.cgi?id=81698
-        ImageWithScale image = bestImageForScaleFactor();
+        ImageWithScale image = bestImageForScaleFactor(deviceScaleFactor);
         FetchRequest request(ResourceRequest(document->completeURL(image.imageURL)), FetchInitiatorTypeNames::css, options);
         request.mutableResourceRequest().setHTTPReferrer(image.referrer);
 
         if (options.corsEnabled == IsCORSEnabled)
             request.setCrossOriginAccessControl(document->securityOrigin(), options.allowCredentials, options.credentialsRequested);
 
         if (ResourcePtr<ImageResource> cachedImage = ImageResource::fetch(request, document->fetcher())) {
-            m_imageSet = StyleFetchedImageSet::create(cachedImage.get(), image.scaleFactor, this);
-            m_accessedBestFitImage = true;
+            m_cachedImageSet = StyleFetchedImageSet::create(cachedImage.get(), image.scaleFactor, this);
+            m_cachedScaleFactor = deviceScaleFactor;
+            m_isCachePending = false;
         }
     }
 
-    return (m_imageSet && m_imageSet->isImageResourceSet()) ? toStyleFetchedImageSet(m_imageSet) : nullptr;
+    return (m_cachedImageSet && m_cachedImageSet->isImageResourceSet()) ? toStyleFetchedImageSet(m_cachedImageSet) : nullptr;
 }
 
-StyleFetchedImageSet* CSSImageSetValue::cachedImageSet(Document* document, float deviceScaleFactor)
+StyleFetchedImageSet* CSSImageSetValue::cacheImageSet(Document* document, float deviceScaleFactor)
 {
-    return cachedImageSet(document, deviceScaleFactor, ResourceFetcher::defaultResourceOptions());
-}
-
-StyleImage* CSSImageSetValue::cachedOrPendingImageSet(float deviceScaleFactor)
-{
-    if (!m_imageSet) {
-        m_imageSet = StylePendingImage::create(this);
-    } else if (!m_imageSet->isPendingImage()) {
-        // If the deviceScaleFactor has changed, we may not have the best image loaded, so we have to re-assess.
-        if (deviceScaleFactor != m_scaleFactor) {
-            m_accessedBestFitImage = false;
-            m_imageSet = StylePendingImage::create(this);
-        }
-    }
-
-    return m_imageSet.get();
+    return cacheImageSet(document, deviceScaleFactor, ResourceFetcher::defaultResourceOptions());
 }
 
 String CSSImageSetValue::customCSSText() const
 {
     StringBuilder result;
     result.append("-webkit-image-set(");
 
     size_t length = this->length();
     size_t i = 0;
     while (i < length) {
@@ skipping 14 matching lines @@
 
         ++i;
     }
 
     result.append(')');
     return result.toString();
 }
 
 bool CSSImageSetValue::hasFailedOrCanceledSubresources() const
 {
-    if (!m_imageSet || !m_imageSet->isImageResourceSet())
+    if (!m_cachedImageSet || !m_cachedImageSet->isImageResourceSet())
         return false;
-    if (Resource* cachedResource = toStyleFetchedImageSet(m_imageSet)->cachedImage())
+    if (Resource* cachedResource = toStyleFetchedImageSet(m_cachedImageSet)->cachedImage())
         return cachedResource->loadFailedOrCanceled();
     return true;
 }
 
 DEFINE_TRACE_AFTER_DISPATCH(CSSImageSetValue)
 {
-    visitor->trace(m_imageSet);
+    visitor->trace(m_cachedImageSet);
     CSSValueList::traceAfterDispatch(visitor);
 }
 
 PassRefPtrWillBeRawPtr<CSSImageSetValue> CSSImageSetValue::valueWithURLsMadeAbsolute()
 {
     RefPtrWillBeRawPtr<CSSImageSetValue> value = CSSImageSetValue::create();
     for (auto& item : *this)
         item->isImageValue() ? value->append(toCSSImageValue(*item).valueWithURLMadeAbsolute()) : value->append(item);
     return value.release();
 }
 
 
 } // namespace blink