Bug 223304 – RenderImage::updateInnerContentRect() pays the cost of constructing a full image URL, but only SVG images need this

• Status: NEW
• Component: Images

[Simon Fraser (smfr)]

RenderImage::updateInnerContentRect() does:

        URL imageSourceURL;
        if (HTMLImageElement* imageElement = is<HTMLImageElement>(element()) ? downcast<HTMLImageElement>(element()) : nullptr)
            imageSourceURL = document().completeURL(imageElement->imageSourceURL());
        imageResource().setContainerContext(containerSize, imageSourceURL);

but CachedImage::setContainerContextForClient() only uses the URL for SVG images.

We should avoid paying the cost when it's not needed. That cost is 0.1% on the MM leaves subtest.

[Radar WebKit Bug Importer]

<rdar://problem/75769980>