Bug 136265

Summary: RenderView::hitTest() should ASSERT(!needsLayout())
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: simon.fraser
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   

Description Simon Fraser (smfr) 2014-08-26 14:21:03 PDT
We need to ensure that layout is up-to-date when hit testing, but there are many code paths that don't do this currently. We should add ASSERT(!needsLayout()) to RenderView::needsLayout() and fix all the callers.

Also need to remove the renderer().document().updateLayout(); in RenderLayer:hitTest().
Comment 1 Simon Fraser (smfr) 2014-09-22 17:51:31 PDT
Done by the patch in bug 136651.

*** This bug has been marked as a duplicate of bug 136651 ***