Bug 114462 – Cannot scroll to footer on a page with overflow:hidden on the body

• Component: Layout and Rendering
• Status: RESOLVED FIXED

[Beth Dakin]

If a WKView has a header/footer, you cannot scroll to the footer if the page has overflow:hidden on the body.

<rdar://problem/13530042>

[Beth Dakin]

Created attachment 197668 [details]
Patch

[Simon Fraser (smfr)]

Comment on attachment 197668 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=197668&action=review

> Source/WebCore/page/FrameView.cpp:614
> +        || (m_frame->page() && m_frame->page()->mainFrame() == m_frame && (headerHeight() || footerHeight()));

Why don't we have Frame::isMainFrame()? :(

[Beth Dakin]

(In reply to comment #2)
> (From update of attachment 197668 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=197668&action=review
> 
> > Source/WebCore/page/FrameView.cpp:614
> > +        || (m_frame->page() && m_frame->page()->mainFrame() == m_frame && (headerHeight() || footerHeight()));
> 
> Why don't we have Frame::isMainFrame()? :(

We REALLY should add this. 

http://trac.webkit.org/changeset/148234