When activating the FrameView::clearBackingStores() optimization on all platforms, some crashiness was introduced. It doesn't look like the optimization actually helps memory usage any any significant way anymore, so I think we should just get rid of it. <rdar://problem/29109053>
Created attachment 297721 [details] Patch
Comment on attachment 297721 [details] Patch Why did you enable it on Mac if it wasn't a measurable gain?
(In reply to comment #2) > Comment on attachment 297721 [details] > Patch > > Why did you enable it on Mac if it wasn't a measurable gain? For consistency, as I wrote in the original ChangeLog. I saw a chance to remove an iOS-only code path and potentially also get a win for Mac, and jumped at it. It was all seasonal wishful thinking :\
Comment on attachment 297721 [details] Patch Clearing flags on attachment: 297721 Committed r210142: <http://trac.webkit.org/changeset/210142>
All reviewed patches have been landed. Closing bug.