Move setPseudoStyle() to RenderImage (from RenderElement.)
Created attachment 214098 [details] Patch
Comment on attachment 214098 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=214098&action=review > Source/WebCore/rendering/RenderImage.cpp:79 > +void RenderImage::setPseudoStyle(PassRefPtr<RenderStyle> pseudoStyle) I thought we were getting rid of PassRefPtr
Comment on attachment 214098 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=214098&action=review > Source/WebCore/rendering/RenderImage.cpp:86 > + RefPtr<RenderStyle> style = RenderStyle::create(); auto
Comment on attachment 214098 [details] Patch Clearing flags on attachment: 214098 Committed r157371: <http://trac.webkit.org/changeset/157371>
All reviewed patches have been landed. Closing bug.
Comment on attachment 214098 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=214098&action=review >> Source/WebCore/rendering/RenderImage.cpp:79 >> +void RenderImage::setPseudoStyle(PassRefPtr<RenderStyle> pseudoStyle) > > I thought we were getting rid of PassRefPtr For return values there is no reason to use it any more. For arguments we have not yet figured out our new approach. But regardless of that, this function does not need a PassRefPtr. It doesn’t seem to take ownership at all.