[Webkit-unassigned] [Bug 81786] Initial support fixed position elements in Qt WebKit2

[bugzilla-daemon at webkit.org]

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


Yael <yael.aharon at nokia.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #135972|0                           |1
        is obsolete|                            |
 Attachment #135972|review?                     |
               Flag|                            |
 Attachment #136157|                            |review?
               Flag|                            |




--- Comment #44 from Yael <yael.aharon at nokia.com>  2012-04-08 13:34:31 PST ---
Created an attachment (id=136157)
 --> (https://bugs.webkit.org/attachment.cgi?id=136157&action=review)
Patch

Address comment #43, except for the change in FrameView.cpp.

> > Source/WebCore/page/FrameView.cpp:1707
> > +        if (m_frame->page()->settings()->acceleratedCompositingForFixedPositionEnabled())
> > +            updateFixedElementsAfterScrolling();
> 
> Maybe we should do the check inside the method instead?

The default in chromium is to not create a graphics layer for fixed elements and moving the check will effectively revert http://trac.webkit.org/changeset/111139 .

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