[Webkit-unassigned] [Bug 25703] Stack overflow crash rendering element with mega-huge number of background layers
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri May 28 04:40:53 PDT 2010
https://bugs.webkit.org/show_bug.cgi?id=25703
--- Comment #4 from Jeremy Orlow <jorlow at chromium.org> 2010-05-28 04:40:52 PST ---
(From update of attachment 57315)
I'm not qualified to officially review this, but it seems fairly sound. A couple nits though.
LayoutTests/css3/many-background-layers.html:12
+ for (var i = 0; i < 100000; i++)
Nit: ++i
Nit: Maybe put 100000 in a constant and initialize that constant with 100*1000 to make it easier to read?
WebCore/rendering/RenderBox.cpp:765
+ for (size_t i = layers.size(); i > 0; i--)
nit: --i
This isn't the most elegant code, but I don't have a better answer off the top of my head.
--
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