RESOLVED FIXED 107213
Allow fixed background layers to be moved by the ScrollingCoordinator
https://bugs.webkit.org/show_bug.cgi?id=107213
Summary Allow fixed background layers to be moved by the ScrollingCoordinator
Simon Fraser (smfr)
Reported 2013-01-17 18:35:03 PST
Allow fixed background layers to be moved by the ScrollingCoordinator
Attachments
Patch (37.03 KB, patch)
2013-01-17 18:36 PST, Simon Fraser (smfr)
no flags
Patch (276.62 KB, patch)
2013-01-18 15:51 PST, Simon Fraser (smfr)
no flags
Patch (276.78 KB, patch)
2013-01-18 16:02 PST, Simon Fraser (smfr)
thorton: review+
Simon Fraser (smfr)
Comment 1 2013-01-17 18:36:24 PST
Build Bot
Comment 2 2013-01-17 19:44:14 PST
Comment on attachment 183345 [details] Patch Attachment 183345 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/15938424 New failing tests: fast/parser/append-child-followed-by-document-write.html
Build Bot
Comment 3 2013-01-17 22:19:56 PST
Comment on attachment 183345 [details] Patch Attachment 183345 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://queues.webkit.org/results/15938492 New failing tests: platform/mac/tiled-drawing/fixed-background/fixed-body-background-opacity-html.html platform/mac/tiled-drawing/fixed-background/fixed-body-background-body-layer.html platform/mac/tiled-drawing/fixed-background/fixed-body-background-transformed-html.html fast/parser/append-child-followed-by-document-write.html platform/mac/tiled-drawing/fixed-background/fixed-body-background-zoomed.html platform/mac/tiled-drawing/fixed-background/fixed-body-background.html platform/mac/tiled-drawing/fixed-background/fixed-html-background.html
WebKit Review Bot
Comment 4 2013-01-18 00:49:28 PST
Comment on attachment 183345 [details] Patch Attachment 183345 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/15944426 New failing tests: fast/parser/append-child-followed-by-document-write.html
WebKit Review Bot
Comment 5 2013-01-18 02:02:16 PST
Comment on attachment 183345 [details] Patch Attachment 183345 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/15955111 New failing tests: fast/parser/append-child-followed-by-document-write.html
James Robinson
Comment 6 2013-01-18 10:31:49 PST
Not really a fan of the concept of a 'counter-scrolling' layer - can't we set this up the same was as position:fixed elements? We'll have to have special logic for both of these for the overscroll bounce effect.
Simon Fraser (smfr)
Comment 7 2013-01-18 10:36:34 PST
The problem with having it like a position:fixed layer is that it then needs special code to insert it in the hierarchy (since it will not be grouped with the other GraphicsLayers owned by the same RenderLayerBacking), and making it work with zooming was really tricky. The counter-scrolling solution came out easier.
Simon Fraser (smfr)
Comment 8 2013-01-18 15:51:17 PST
Tim Horton
Comment 9 2013-01-18 16:01:04 PST
Comment on attachment 183564 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=183564&action=review > Source/WebCore/page/scrolling/ScrollingStateScrollingNode.h:120 > + void setCounterScrollingLayerDidChange(bool layerDidChange) { m_counterScrollingLayerDidChange = layerDidChange; } // FIXME: Need this? Well, do you? > Source/WebCore/page/scrolling/mac/ScrollingStateScrollingNodeMac.mm:2 > + * Copyright (C) 2012 Apple Inc. All rights reserved. Nope. > Source/WebCore/rendering/RenderBoxModelObject.cpp:1251 > + else { > + if (FrameView* frameView = view()->frameView()) > + viewportRect.setLocation(IntPoint(frameView->scrollOffsetForFixedPosition())); > + } Is this actually our style? I'm not positive.
Simon Fraser (smfr)
Comment 10 2013-01-18 16:02:42 PST
WebKit Review Bot
Comment 11 2013-01-18 16:06:06 PST
Attachment 183566 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/plat..." exit_code: 1 LayoutTests/platform/mac/tiled-drawing/fixed-background/fixed-body-background-zoomed-expected.png:0: Have to enable auto props in the subversion config file (/home/alancutter/.subversion/config "enable-auto-props = yes"). Have to set the svn:mime-type in the subversion config file (/home/alancutter/.subversion/config "*.png = svn:mime-type=image/png"). [image/png] [5] Total errors found: 1 in 33 files If any of these errors are false positives, please file a bug against check-webkit-style.
Tim Horton
Comment 12 2013-01-18 16:08:49 PST
Comment on attachment 183564 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=183564&action=review > Source/WebCore/ChangeLog:9 > + fixed root backgounds (those which have background images all of which have s/backgounds/backgrounds/
Tim Horton
Comment 13 2013-01-18 16:21:05 PST
Comment on attachment 183566 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=183566&action=review > Source/WebCore/ChangeLog:24 > + (WebCore::ScrollingCoordinator::updateMainFrameScrollPosition): Sync or set the postion of the s/postion/position/
Simon Fraser (smfr)
Comment 14 2013-01-18 17:22:48 PST
Note You need to log in before you can comment on or make changes to this bug.