[Webkit-unassigned] [Bug 33150] Do not render the full frame when there is some elements with fixed positioning

[bugzilla-daemon at webkit.org]

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


Benjamin Poulain <benjamin.poulain at nokia.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #46040|0                           |1
        is obsolete|                            |
  Attachment #46040|review?, commit-queue?      |
               Flag|                            |
  Attachment #46047|                            |review?, commit-queue?
               Flag|                            |




--- Comment #19 from Benjamin Poulain <benjamin.poulain at nokia.com>  2010-01-07 05:13:08 PST ---
Created an attachment (id=46047)
 --> (https://bugs.webkit.org/attachment.cgi?id=46047)
Repaint only the invalidated area after scrolling

Patch updates to apply the comments of Kenneth:

> > > I dislike the method name. Shouldn't it always be fast? 
> > 
> scrollContentsFastPath :-) ?

Done.

> OK. maybe invalidated is better than invalid.

Done

> I wonder if it would be better to use addFixedObject and removeFixedObject
> instead as we use add/remove elsewhere (like addSlowRepaintObject)

I choose the name register to emphasize the fact that you only register some
fixed object. I am afraid that with addFixedObject(), it might be understood
that all objects have to be registered.

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