[Webkit-unassigned] [Bug 43779] RenderImage::imageChanged invalidates wrong area

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Aug 12 00:03:27 PDT 2010


Nikolas Zimmermann <zimmermann at kde.org> changed:

           What    |Removed                     |Added
  Attachment #64163|review?                     |review-
               Flag|                            |

--- Comment #16 from Nikolas Zimmermann <zimmermann at kde.org>  2010-08-12 00:03:26 PST ---
(From update of attachment 64163)
Also some comments:
 +      m_cachedImage = RenderImageResource::create(this);
Can be moved in the initialization list.

 +      m_cachedImage.clear();
Not needed IIRC.

 +                  if (svgStyle->shapeRendering() == SR_CRISPEDGES)
Is that necessary for images? I don't think so.

 +      // This empty image may be cached by SVG filter effects which must be invalidated.
That change is not correct, you basically reverted to an older version of the imageChanged version.
The SVGResourcesCache::... call is correct.

 +      if (RenderSVGImage* imageObj = toRenderSVGImage(renderer())) {
Change to early return.

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list