Also see bug 55257.
The problem is that checking 'node()->isDocumentNode()' is too permissive. The frame might have the attribute scrolling=no set and will never be scrollable. Also see bug 55257, where this would lead to creation of an unwanted compositing layer.
Created attachment 84681 [details] Patch
Doing some quick testing it seems that iframes with scrolling=no are programmatically scrollable in Firefox nightlies + the latest Opera stable. My testcase: <!DOCTYPE html> <iframe src="data:text/html;charset=utf-8,<!DOCTYPE html><script>setTimeout('window.scroll(0,10)',500)</script><div style='height:500px'>lala</div>" scrolling=no></iframe> Are you sure we want to change this?
I see, in that case it's probably ok to allow layers in bug 55257 for that case also. Thanks! (In reply to comment #3) > Doing some quick testing it seems that iframes with scrolling=no are programmatically scrollable in Firefox nightlies + the latest Opera stable. My testcase: > > <!DOCTYPE html> > <iframe src="data:text/html;charset=utf-8,<!DOCTYPE html><script>setTimeout('window.scroll(0,10)',500)</script><div style='height:500px'>lala</div>" scrolling=no></iframe> > > Are you sure we want to change this?
Comment on attachment 84681 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=84681&action=review > Source/WebCore/ChangeLog:10 > + No new tests. (OOPS!) Why no tests? We normally require tests for all bug fixes.
Comment on attachment 84681 [details] Patch I see. This bug was closed. Clearing the review flag, then.