Remove unused overlap checking functions from FrameView.
authorzalan@apple.com <zalan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 6 Sep 2016 21:27:15 +0000 (21:27 +0000)
committerzalan@apple.com <zalan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 6 Sep 2016 21:27:15 +0000 (21:27 +0000)
https://bugs.webkit.org/show_bug.cgi?id=161635

Reviewed by Simon Fraser.

No change in functionality.

* page/FrameView.cpp:
(WebCore::FrameView::isOverlappedIncludingAncestors): Deleted.
* page/FrameView.h:

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

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

index 53d59b8..9e2282e 100644 (file)
@@ -1,3 +1,16 @@
+2016-09-06  Zalan Bujtas  <zalan@apple.com>
+
+        Remove unused overlap checking functions from FrameView.
+        https://bugs.webkit.org/show_bug.cgi?id=161635
+
+        Reviewed by Simon Fraser.
+
+        No change in functionality.
+
+        * page/FrameView.cpp:
+        (WebCore::FrameView::isOverlappedIncludingAncestors): Deleted.
+        * page/FrameView.h:
+
 2016-09-06  Simon Fraser  <simon.fraser@apple.com>
 
         Align element.scroll() / scrollTo() / scrollBy() with the CSSOM specification
index b2f4bc1..8331fc8 100644 (file)
@@ -2050,19 +2050,6 @@ void FrameView::setIsOverlapped(bool isOverlapped)
     updateCanBlitOnScrollRecursively();
 }
 
-bool FrameView::isOverlappedIncludingAncestors() const
-{
-    if (isOverlapped())
-        return true;
-
-    if (FrameView* parentView = parentFrameView()) {
-        if (parentView->isOverlapped())
-            return true;
-    }
-
-    return false;
-}
-
 void FrameView::setContentIsOpaque(bool contentIsOpaque)
 {
     if (contentIsOpaque == m_contentIsOpaque)
index b556fba..1561655 100644 (file)
@@ -258,8 +258,6 @@ public:
 
     void setCannotBlitToWindow();
     void setIsOverlapped(bool);
-    bool isOverlapped() const { return m_isOverlapped; }
-    bool isOverlappedIncludingAncestors() const;
     void setContentIsOpaque(bool);
 
     void addSlowRepaintObject(RenderElement*);
@@ -714,7 +712,7 @@ private:
     
     bool m_canHaveScrollbars;
     bool m_cannotBlitToWindow;
-    bool m_isOverlapped;
+    bool m_isOverlapped { false };
     bool m_contentIsOpaque;
 
     Timer m_layoutTimer;