[Webkit-unassigned] [Bug 66969] Chromium: Add a layer for rubber-band overhang painting to the hardware path.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Sep 13 09:40:34 PDT 2011


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





--- Comment #48 from James Robinson <jamesr at chromium.org>  2011-09-13 09:40:34 PST ---
(In reply to comment #47)
> I understand the desire to avoid any redrawing while scrolling, but I fear to do that for the overhang stuff would result in a lot of code that will be different between the compositor / non-compositor path that would be a maintenance burden if we ever decide to change how the overhang looks. (For instance, the fact that we currently draw a drop-shadow from the top of the window would affect the design of the layers used under the compositor if we wish to avoid re-drawing things.)
> 
> So I'm hoping just splitting horizontal / vertical overhang layers would provide good enough performance so that we wouldn't need to write/maintain a lot of compositor specific code to draw the overhang.

That's the thing, the two codepaths *are* very different and so the compositor path by definition has to have a lot of compositor specific code.  We'll have to rewrite this patch for the compositor path within a few weeks, if it lands as-is.  I'd really like to not be blocking your progress as much as possible, but this is a lot of complexity.

Anyway, the performance looks OK in release, and I suppose we (the compositor team) can pick up the rewrite since there's some blocking work that we need to take care of.

-- 
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