82051 2012-03-23 06:35:16 -0700 Direct shadow casting by the background is an unnaturally *slow* path on some ports for some apps 2012-03-23 11:15:52 -0700 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) Unspecified Unspecified NEW P2 Normal --- 1 tomhudson webkit-unassigned hyatt jasonparekh mitz simon.fraser thorton oldest_to_newest 586286 0 tomhudson 2012-03-23 06:35:16 -0700 http://trac.webkit.org/changeset/107836 (https://bugs.webkit.org/show_bug.cgi?id=78728) caused a severe performance regression for Chromium: an app under development with a large scrolling div that casts a shadow went from 6ms/frame paint time to 40-50ms/frame paint time while scrolling, all of the additional time spent in a blur. 586288 1 tomhudson 2012-03-23 06:37:25 -0700 If I comment out two lines of the BoxShadow CL from Feb 15 (turning off the "fast path" in RenderBoxModelObject::fillPaintLayerExtended() and reactivating the "slow path" in RenderBox::paintBoxDecorations()), Chromium performance returns to what we were seeing before this change. 586332 2 mitz 2012-03-23 07:33:26 -0700 Sorry about that! I had no intention to make any port or any app slower. Are you able to attach a test case that shows this slowdown, or otherwise to tell whether r107836 made that case slower in the OS X, Quartz-based WebKit? I think the way to address this probably would be to refine the logic that determines if the shadow should be cast by the background, but I am wondering if the decision should be dependent on the GraphicsContext implementation. 586461 3 simon.fraser 2012-03-23 10:21:09 -0700 Perhaps this should be considered to be a Skia performance bug? 586465 4 mitz 2012-03-23 10:29:13 -0700 (In reply to comment #3) > Perhaps this should be considered to be a Skia performance bug? We shouldn’t make changes in WebKit that make it slower, regardless of platform bugs. 586509 5 tomhudson 2012-03-23 11:15:52 -0700 It might be something in the platform glue for Chromium, rather than Skia? The new code path ends up being presented to Skia as a 4px radius gaussian blur of the 1000x800 div that's being scrolled; I presume CG is doing something much smarter if this is a fast path. I filed http://code.google.com/p/chromium/issues/detail?id=119778 against Chromium to see if I can get somebody familiar with those layers to take a look.