Bug 87523

Summary: Make RenderLayer not arena-allocated
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, eric, esprehn+autocc, glenn, hyatt, jchaffraix, kling, kondapallykalyan, ojan, simon.fraser
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 122651    
Bug Blocks:    
Attachments:
Description Flags
Patch none

Description Simon Fraser (smfr) 2012-05-25 11:42:06 PDT
We don't think we're getting any benefit from RenderLayer being arena-allocated, and it adds code complexity.
Comment 1 Eric Seidel (no email) 2012-05-25 12:12:04 PDT
Agreed!
Comment 2 Ojan Vafai 2012-05-25 12:21:19 PDT
How is RenderLayer different than the other Render* classes WRT usefulness of being arena-allocated?
Comment 3 Simon Fraser (smfr) 2012-05-25 12:47:00 PDT
There don't tend to be that many RenderLayers on most pages (tens - hundreds, not thousands).
Comment 4 Ojan Vafai 2012-05-25 13:03:17 PDT
Fair enough.
Comment 5 Andreas Kling 2013-10-11 01:14:29 PDT
Created attachment 213975 [details]
Patch
Comment 6 WebKit Commit Bot 2013-10-11 02:37:57 PDT
Comment on attachment 213975 [details]
Patch

Clearing flags on attachment: 213975

Committed r157297: <http://trac.webkit.org/changeset/157297>
Comment 7 WebKit Commit Bot 2013-10-11 02:37:59 PDT
All reviewed patches have been landed.  Closing bug.
Comment 8 WebKit Commit Bot 2013-10-11 09:49:08 PDT
Re-opened since this is blocked by bug 122651
Comment 9 Alexey Proskuryakov 2013-10-11 09:53:46 PDT
This caused crashes on tests (debug and release WK2). Very strange that EWS didn't catch it, was the landed patch identical to the one posted here?

Rolled out in <https://trac.webkit.org/r157306>. Results:

http://build.webkit.org/results/Apple%20MountainLion%20Debug%20WK2%20(Tests)/r157297%20(13299)/results.html
Comment 10 Andreas Kling 2013-10-12 02:14:09 PDT
Committed r157333: <http://trac.webkit.org/changeset/157333>