BackgroundImage incorrectly returns empty url() when created on-the-fly

Source/core/css/CSSCrossfadeValue.cpp

Index: Source/core/css/CSSCrossfadeValue.cpp
diff --git a/Source/core/css/CSSCrossfadeValue.cpp b/Source/core/css/CSSCrossfadeValue.cpp
index 43536472b972fa8c1adccba67c7ba68eaea491a5..efb0bdfeda806e609b636bfde3f1b2d685970382 100644
--- a/Source/core/css/CSSCrossfadeValue.cpp
+++ b/Source/core/css/CSSCrossfadeValue.cpp
@@ -117,10 +117,22 @@ String CSSCrossfadeValue::customCSSText() const
     return result.toString();
 }
 
+PassRefPtrWillBeRawPtr<CSSCrossfadeValue> CSSCrossfadeValue::valueWithAbsoluteURL()
+{
+    if (m_fromValue->isImageValue() && m_toValue->isImageValue()) {

[Timothy Loh, 2015/09/15 12:20:47]

The logic here seems weird. What if only one side is an image value?

[nainar, 2015/09/16 07:16:48]

Fixed.

+        return CSSCrossfadeValue::create(toCSSImageValue(*m_fromValue).valueWithAbsoluteURL(), toCSSImageValue(*m_toValue).valueWithAbsoluteURL(), m_percentageValue);
+    }
+    return this;
+}
+
 IntSize CSSCrossfadeValue::fixedSize(const LayoutObject* layoutObject)
 {
-    Image* fromImage = renderableImageForCSSValue(m_fromValue.get(), layoutObject);
-    Image* toImage = renderableImageForCSSValue(m_toValue.get(), layoutObject);
+    float percentage = m_percentageValue->getFloatValue();

[Timothy Loh, 2015/09/15 12:20:47]

???

[nainar, 2015/09/16 07:16:48]

Fixed.

+    float inversePercentage = 1 - percentage;
+
+    Document* document = &layoutObject->document();
+    ImageResource* cachedFromImage = cachedImageForCSSValue(m_fromValue.get(), document);
+    ImageResource* cachedToImage = cachedImageForCSSValue(m_toValue.get(), document);
 
     if (!fromImage || !toImage)
         return IntSize();