StyleResolver::loadPendingImage() should take a reference to StylePendingImage. Clean up as noted in Bug 128671 Comment #4.
Created attachment 224580 [details] Patch v1
Comment on attachment 224580 [details] Patch v1 View in context: https://bugs.webkit.org/attachment.cgi?id=224580&action=review r=me > Source/WebCore/css/StyleResolver.cpp:3495 > + if (CSSImageSetValue* imageSetValue = pendingImage.cssImageSetValue()) Shouldn't this just be 'auto imageSetValue', too?
Committed r164489: <http://trac.webkit.org/changeset/164489>
(In reply to comment #2) > (From update of attachment 224580 [details]) > > Source/WebCore/css/StyleResolver.cpp:3495 > > + if (CSSImageSetValue* imageSetValue = pendingImage.cssImageSetValue()) > > Shouldn't this just be 'auto imageSetValue', too? I also fixed this while I was in the neighborhood. Thanks!