[Webkit-unassigned] [Bug 39123] Unneeded fillRect call in RenderBoxModelObject::paintFillLayerExtended

[bugzilla-daemon at webkit.org]

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





--- Comment #12 from Pat Galizia <pgalizia at codeaurora.org>  2010-05-20 11:03:22 PST ---
(In reply to comment #10)
> (In reply to comment #9)
> > This patch isn't avoiding doing a fillRect for a fully transparent color. It's checking to see if a later fillRect will fully obscure an earlier one, which I think is a fine optimization.
> 
> Sorry, I was totally confused. Now that I know what this is, I can make a different comment.
> 
> We have tried this optimization multiple times in the past and have never need able to actually squeeze a speed-up out of it. I’m excited that things are different this time, and also a little skeptical because of the previous attempts.

The improvement may be due to our particular device configuration.  I'll look into applying this to a different device and see if we end up losing cycles with that change there.  If so, we can determine what to do next, since it makes no sense to apply this for one type of device if it regresses others.

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