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

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jan 5 15:17:11 PST 2010


Dave Hyatt <hyatt at apple.com> changed:

           What    |Removed                     |Added
  Attachment #45892|review?, commit-queue?      |review-
               Flag|                            |

--- Comment #5 from Dave Hyatt <hyatt at apple.com>  2010-01-05 15:17:10 PST ---
(From update of attachment 45892)
There are a number of issues with this patch:

(1) ScrollView is not designed to know anything about the render tree or
RenderObjects directly.  I'd move the HashSet to FrameView and then make a
virtual method that is called in ScrollView.

(2) Please make a constant for the cutoff number in the code.  Don't just use 5
inline like that.

(3) You should disregard fixed positioned objects inside transforms, as they
aren't actually fixed to the ScrollView.  You can verify this by checking if
the containingBlock is actually the RenderView for each RenderObject before
allowing it to be registered.

(4) Why not make fixed background images work too?  They are just as common (if
not moreso) than fixed positioning.

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

More information about the webkit-unassigned mailing list