[Webkit-unassigned] [Bug 115178] [BlackBerry] Make scroll position adjustment work with pages with fixed position elements

[bugzilla-daemon at webkit.org]

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


Arvid Nilsson <anilsson at rim.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |anilsson at rim.com




--- Comment #3 from Arvid Nilsson <anilsson at rim.com>  2013-04-25 09:27:16 PST ---
(In reply to comment #2)
> (From update of attachment 199664 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=199664&action=review
> 
> > Source/WebKit/blackberry/Api/WebPage.cpp:5255
> > +void WebPage::findFixedElementRect(LayerCompositingThread* layer, Platform::IntRect& fixedElementRect)
> 
> It seems like this method should be on LayerCompositingThread, since every operation uses layer.

I was thinking of suggesting to put this method on WebPageCompositorPrivate, at least. Maybe moving it to LayerCompositingThread makes even more sense - but I think traversing the layers should be the responsibility of some managing class like LayerRenderer or WebPageCompositor(Private), not LayerCompositingThread itself.

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