Bug 193879

Summary: Have composited RenderIFrame layers make FrameHosting scrolling tree nodes to parent the iframe's scrolling node
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: New BugsAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: fred.wang, koivisto, ryanhaddad, simon.fraser, thorton, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=193955
Attachments:
Description Flags
Patch koivisto: review+

Description Simon Fraser (smfr) 2019-01-26 20:33:38 PST
Have composited RenderIFrame layers make FrameHosting scrolling tree nodes to parent the iframe's scrolling node
Comment 1 Simon Fraser (smfr) 2019-01-26 21:08:38 PST
Created attachment 360265 [details]
Patch
Comment 2 Radar WebKit Bug Importer 2019-01-26 21:11:35 PST
<rdar://problem/47578495>
Comment 3 Simon Fraser (smfr) 2019-01-27 08:59:53 PST
https://trac.webkit.org/changeset/240553/webkit
I fixed the crash too!
Comment 4 Ryan Haddad 2019-01-28 21:47:16 PST
This change caused compositing/iframes/remove-reinsert-webview-with-iframe.html to consistently crash on iOS Simulator bots, significantly slowing down EWS:
https://build.webkit.org/results/Apple%20iOS%2012%20Simulator%20Release%20WK2%20(Tests)/r240636%20(2254)/compositing/iframes/remove-reinsert-webview-with-iframe-crash-log.txt
Comment 5 Frédéric Wang (:fredw) 2019-01-29 00:30:31 PST
(In reply to Ryan Haddad from comment #4)
> This change caused
> compositing/iframes/remove-reinsert-webview-with-iframe.html to consistently
> crash on iOS Simulator bots, significantly slowing down EWS:
> https://build.webkit.org/results/
> Apple%20iOS%2012%20Simulator%20Release%20WK2%20(Tests)/r240636%20(2254)/
> compositing/iframes/remove-reinsert-webview-with-iframe-crash-log.txt

Trace looks similar to bug 193907.
Comment 6 Simon Fraser (smfr) 2019-01-29 13:33:52 PST
Crashes should be fixed by https://trac.webkit.org/changeset/240677/webkit