Bug 173958

Summary: Crash in WebCore::ScrollingTreeFixedNode::updateLayersAfterAncestorChange
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: Layout and RenderingAssignee: Ryosuke Niwa <rniwa>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, buildbot, cmarcelo, jamesr, luiz, simon.fraser, tonikitoo, zalan
Priority: P2 Keywords: InRadar
Version: Safari Technology Preview   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Fixes the crash none

Description Ryosuke Niwa 2017-06-28 20:25:32 PDT
We're crashing inside CALayer sometimes:

              24 QuartzCore: -[CALayer setPosition:]
         ==> 24 WebCore: WebCore::ScrollingTreeFixedNode::updateLayersAfterAncestorChange(WebCore::ScrollingTreeNode const&, WebCore::FloatRect const&, WebCore::FloatSize const&) <==
                  24 WebCore: WebCore::ScrollingTreeFrameScrollingNodeIOS::updateLayersAfterViewportChange(WebCore::FloatRect const&, double)
                    24 WebKit: -[WKContentView didUpdateVisibleRect:unobscuredRect:unobscuredRectInScrollViewCoordinates:obscuredInset:inputViewBounds:scale:minimumScale:inStableState:isChangingObscuredInsetsInteractively:enclosedInScrollableAncestorView:]
Comment 1 Ryosuke Niwa 2017-06-28 20:25:44 PDT
<rdar://problem/31272462
Comment 2 Ryosuke Niwa 2017-06-28 20:42:32 PDT
Created attachment 314104 [details]
Fixes the crash
Comment 3 Ryosuke Niwa 2017-06-28 20:58:18 PDT
Comment on attachment 314104 [details]
Fixes the crash

Clearing flags on attachment: 314104

Committed r218915: <http://trac.webkit.org/changeset/218915>
Comment 4 Ryosuke Niwa 2017-06-28 20:58:20 PDT
All reviewed patches have been landed.  Closing bug.