[Webkit-unassigned] [Bug 10408] SVGSVGElement does not support suspendRedraw, unsuspendRedraw

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=10408





--- Comment #13 from Raks <rakssvg at gmail.com>  2012-06-07 08:57:50 PST ---
(In reply to comment #10)
> (In reply to comment #9)
> > (In reply to comment #8)
> > > (In reply to comment #7)
> > > > (In reply to comment #6)
> > > > > We are using SVG to create UI for our WeApp and we feel a need for suspendRedraw. The case is as follows. As scrolling is not supported by SVG we have implemented our own scrolling using translation but what we have observed is that scrolling is not smooth if the whole of UI has lots of images like 200 of them. Now this could be because the whole page is repainted and drawing only that  part of the svg document which has scrolling would probably solve our problem.
> > > > > 
> > > > > So I would urge someone to support suspendRedraw and so please prioritize it
> > > > > 
> > > > > In general once you provide the feature people will use them and infact find interesting uses of them. So please do not think of dropping this
> > > > 
> > > > Would suspendredraw actually solve your usecase? If repaint is suspended, the portions of the svg document that were not visible initially would not have been drawn to begin with, and therefore would not be available when you scroll the svg portion. While drawing is suspended, the SVG portion would not be redrawn at all, which is probably not what you want.
> > > 
> > > I would not suspend the node which has scrolling but all the other nodes. The whole of UI has lots of images but the node having srolling may have a tenth of all the images. Unless SVG repaint is already optimized to paint only the part of the screen which is dirtied, calling suspendRedraw all other nodes which are not changing during scrolling will help
> > 
> > SVG's repaint is already optimized to only paint the parts of the screen that are dirty.
> 
> Just a quick followup--it's always good to double-check these things. I instrumented the code and wroteup a testcase and verified that RenderSVGImage::paint skips the hard work if the dirty rect doesn't intersect the image repaint rect.

@Philip, yes infact I was expecting that  Webkit should paint only the dirtied area but since I was'nt sure about it and suspendRedraw was there in the specs, I used it and it did'nt work. I agree I should have looked in the code to see optimization already done.

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