[webkit-reviews] review denied: [Bug 33408] Add flag IGNORE_FIXED_BACKGROUNDS (disabled by default) to ignore fixed background images and accelerate web page scrolling on low-powered/mobile devices : [Attachment 46176] Patch 1 with test cases

[bugzilla-daemon at webkit.org]

Simon Fraser (smfr) <simon.fraser at apple.com> has denied Daniel Bates
<dbates at webkit.org>'s request for review:
Bug 33408: Add flag IGNORE_FIXED_BACKGROUNDS (disabled by default) to ignore
fixed background images and accelerate web page scrolling on low-powered/mobile
devices
https://bugs.webkit.org/show_bug.cgi?id=33408

Attachment 46176: Patch 1 with test cases
https://bugs.webkit.org/attachment.cgi?id=46176&action=review

------- Additional Comments from Simon Fraser (smfr) <simon.fraser at apple.com>
This seems like the wrong approach. Why not just change CSSStyleSelector to
override FixedBackgroundAttachment with ScrollBackgroundAttachment?

Also, having a whole new build setting, IGNORE_FIXED_BACKGROUNDS, for this
seems like overkill. What if you find some other CSS property that you'd like
to ignore to get fast scrolling? It seems like it should be
ALLOW_FAST_SCROLLING or something that indicates that this is for a
limited-capacity mobile device.