[Webkit-unassigned] [Bug 45773] Add isMainFrame flag to scroll functions

[bugzilla-daemon at webkit.org]

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





--- Comment #4 from João Paulo Rechi Vita <jprvita at profusion.mobi>  2010-12-09 11:00:03 PST ---
(In reply to comment #3)
> (From update of attachment 67592 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=67592&action=review
> 
> I don't really understand why this is needed, and as such I cannot say if this is the best way of accomplishing what you are trying to do.
> 
> r- for this reason.
> 
> > WebCore/ChangeLog:8
> > +        This information is needed by the new EFL tiled backing store.
> 
> This would probably need more explanation. Why is that needed for the EFL tiled backing store. - you should explain that instead
> 

When when the scroll is on the main frame we want to simply scroll the backing store instead of doing repaints. To accomplish that we need to know whether a scroll is on the main frame or on a inner frame when processing the scroll. Although this is not necessary with frame flattening, we want to support frames as well.

If you have any suggestion on other ways to do this I'll be happy to hear.

> > WebCore/loader/EmptyClients.h:151
> > +    virtual void invalidateContentsForSlowScroll(const IntSize&, const IntRect&, bool, bool) {};
> > +    virtual void scroll(const IntSize&, const IntRect&, const IntRect&, bool) { }
> 
> We are trying to more away from bool arguments. Consider using enums instead.

I'll have to send an updated version of the patch anyway, because we found a small bug in this modifications (that's why this bug has been left for so long). I'll take this into consideration when sending the new version. But could please share your thoughts on the overall approach before I do so?

Thanks.

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