On Apr 1, 2013, at 2:47 PM, Xianzhu Wang <[email protected]> wrote:

> Hi,
> 
> ScrollableArea::scrollSize() and ScrollableArea::maximumScrollPosition() seem 
> to return the same value if the parameters of the scrollbars are updated. Can 
> we remove one of them?

I don't understand the " if the parameters of the scrollbars are updated" part.

Do they return the same thing if scrollOrigin is not 0,0? This happens in RTL 
contexts.

Simon

> 
> I think the benefits are:
> - no redundancy;
> - ScrollableArea could be independent on scrollbars;
> - maximumScrollPosition() is more up-to-date without the need of 
> updateScrollbars;
> - the ambiguity of the name scrollSize (e.g. RenderLayer::m_scrollSize and 
> RenderLayer::scrollSize() are totally different things) can be eliminated
> 
> What do you think? Am I missing anything?
> 
> Xianzhu
> 
_______________________________________________
webkit-dev mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-dev

Reply via email to