[Webkit-unassigned] [Bug 30813] scrolling="no" attribute of iframe doesn't work in Chrome

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=30813


johnnyding <johnnyding.webkit at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hyatt at apple.com




--- Comment #2 from johnnyding <johnnyding.webkit at gmail.com>  2009-10-27 07:31:41 PDT ---
It is because in a frame, WebKit will respect the scrolling type defined on
body tag even the owner frame element of the frame has specified value "no" to
"scrolling" attribute. (Please refer to function FrameView::layout,
file:FrameView.cpp line:583/586).

To resolve this problem, I personally think that we should respect the
scrolling attribute if ownerFrame element has specified "no" to the attribute.

A draft fix is adding the following code before line 568

 HTMLFrameOwnerElement* frameElt = m_frame->ownerElement();
 if (frameElt->scrollingMode() == ScrollbarAlwaysOff) {
    hMode = vMode = ScrollbarAlwaysOff;
 } else if (!subtree)
...

Does it make sense?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.