Cannot scroll to footer on a page with overflow:hidden on the body
authorbdakin@apple.com <bdakin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 11 Apr 2013 21:01:07 +0000 (21:01 +0000)
committerbdakin@apple.com <bdakin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 11 Apr 2013 21:01:07 +0000 (21:01 +0000)
https://bugs.webkit.org/show_bug.cgi?id=114462
-and corresponding-
<rdar://problem/13530042>

Reviewed by Simon Fraser.

Add a new scenario in which we want to overrideHidden -- whenever we are the
mainFrame and there is a header or footer.

* page/FrameView.cpp:
(WebCore::FrameView::applyOverflowToViewport):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@148234 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/page/FrameView.cpp

index aa1c138..09b36ea 100644 (file)
@@ -1,3 +1,18 @@
+2013-04-11  Beth Dakin  <bdakin@apple.com>
+
+        Cannot scroll to footer on a page with overflow:hidden on the body
+        https://bugs.webkit.org/show_bug.cgi?id=114462
+        -and corresponding-
+        <rdar://problem/13530042>
+
+        Reviewed by Simon Fraser.
+
+        Add a new scenario in which we want to overrideHidden -- whenever we are the 
+        mainFrame and there is a header or footer.
+
+        * page/FrameView.cpp:
+        (WebCore::FrameView::applyOverflowToViewport):
+
 2013-04-11  Commit Queue  <rniwa@webkit.org>
 
         Unreviewed, rolling out r148034, r148052, r148097, and
index 1231252..3cd0d65 100644 (file)
@@ -607,9 +607,11 @@ void FrameView::applyOverflowToViewport(RenderObject* o, ScrollbarMode& hMode, S
     // use the root element.
 
     // To combat the inability to scroll on a page with overflow:hidden on the root when scaled, disregard hidden when
-    // there is a frameScaleFactor that is greater than one on the main frame.
+    // there is a frameScaleFactor that is greater than one on the main frame. Also disregard hidden if there is a
+    // header or footer.
 
-    bool overrideHidden = m_frame->page() && m_frame->page()->mainFrame() == m_frame && m_frame->frameScaleFactor() > 1;
+    bool overrideHidden = (m_frame->page() && m_frame->page()->mainFrame() == m_frame && m_frame->frameScaleFactor() > 1)
+        || (m_frame->page() && m_frame->page()->mainFrame() == m_frame && (headerHeight() || footerHeight()));
 
     EOverflow overflowX = o->style()->overflowX();
     EOverflow overflowY = o->style()->overflowY();