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

Source/core/style/StyleFetchedImageSet.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 34 matching lines @@
 StyleFetchedImageSet::~StyleFetchedImageSet()
 {
     m_bestFitImage->removeClient(this);
 }
 
 PassRefPtrWillBeRawPtr<CSSValue> StyleFetchedImageSet::cssValue() const
 {
     return m_imageSetValue;
 }
 
+PassRefPtrWillBeRawPtr<CSSValue> StyleFetchedImageSet::computedCSSValue() const
+{
+    return m_imageSetValue ? m_imageSetValue->valueWithAbsoluteURL() : nullptr;

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

Looks like m_imageSetValue is never null?

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

Done.

+}
+
 bool StyleFetchedImageSet::canRender(const LayoutObject& layoutObject, float multiplier) const
 {
     return m_bestFitImage->canRender(layoutObject, multiplier);
 }
 
 bool StyleFetchedImageSet::isLoaded() const
 {
     return m_bestFitImage->isLoaded();
 }
 
@@ skipping 54 matching lines @@
     return m_bestFitImage->currentFrameKnownToBeOpaque(layoutObject);
 }
 
 DEFINE_TRACE(StyleFetchedImageSet)
 {
     visitor->trace(m_imageSetValue);
     StyleImage::trace(visitor);
 }
 
 } // namespace blink