Bug 143692

Summary: [iOS WK2] Interactive elements of developer.apple.com are broken
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: New BugsAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: bdakin, commit-queue, ddkilzer, esprehn+autocc, glenn, kondapallykalyan, simon.fraser, thorton
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch sam: review+

Simon Fraser (smfr)
Reported 2015-04-13 21:00:41 PDT
[iOS WK2] Interactive elements of developer.apple.com are broken
Attachments
Patch (10.06 KB, patch)
2015-04-13 21:04 PDT, Simon Fraser (smfr)
sam: review+
Simon Fraser (smfr)
Comment 1 2015-04-13 21:04:47 PDT
David Kilzer (:ddkilzer)
Comment 2 2015-04-14 02:35:05 PDT
Sam Weinig
Comment 3 2015-04-14 08:50:25 PDT
Comment on attachment 250690 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=250690&action=review > Source/WebCore/ChangeLog:9 > + WHen a composited RenderLayer had nodes in the scrolling tree by virtue of WHen -> When > Source/WebCore/ChangeLog:14 > + Fix by having RnderLayerCompositor::updateScrollCoordinationForThisFrame() RnderLayerCompositor -> RenderLayerCompositor > LayoutTests/platform/ios-simulator-wk2/scrolling/remove-scrolling-role.html:41 > + window.setTimeout(function() { Why do we need the 0-delay timer here?
Simon Fraser (smfr)
Comment 4 2015-04-14 10:25:07 PDT
(In reply to comment #3) > Comment on attachment 250690 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=250690&action=review > > > Source/WebCore/ChangeLog:9 > > + WHen a composited RenderLayer had nodes in the scrolling tree by virtue of > > WHen -> When > > > Source/WebCore/ChangeLog:14 > > + Fix by having RnderLayerCompositor::updateScrollCoordinationForThisFrame() > > RnderLayerCompositor -> RenderLayerCompositor This is what happens when typing over a slow VNC connection. > > LayoutTests/platform/ios-simulator-wk2/scrolling/remove-scrolling-role.html:41 > > + window.setTimeout(function() { > > Why do we need the 0-delay timer here? To get an initial layout with the pre-changed style.
Simon Fraser (smfr)
Comment 5 2015-04-14 10:26:54 PDT
Note You need to log in before you can comment on or make changes to this bug.