Title: [205506] trunk/Source/WebCore
- Revision
- 205506
- Author
- [email protected]
- Date
- 2016-09-06 14:27:15 -0700 (Tue, 06 Sep 2016)
Log Message
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:
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (205505 => 205506)
--- trunk/Source/WebCore/ChangeLog 2016-09-06 21:18:06 UTC (rev 205505)
+++ trunk/Source/WebCore/ChangeLog 2016-09-06 21:27:15 UTC (rev 205506)
@@ -1,3 +1,16 @@
+2016-09-06 Zalan Bujtas <[email protected]>
+
+ 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 <[email protected]>
Align element.scroll() / scrollTo() / scrollBy() with the CSSOM specification
Modified: trunk/Source/WebCore/page/FrameView.cpp (205505 => 205506)
--- trunk/Source/WebCore/page/FrameView.cpp 2016-09-06 21:18:06 UTC (rev 205505)
+++ trunk/Source/WebCore/page/FrameView.cpp 2016-09-06 21:27:15 UTC (rev 205506)
@@ -2050,19 +2050,6 @@
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)
Modified: trunk/Source/WebCore/page/FrameView.h (205505 => 205506)
--- trunk/Source/WebCore/page/FrameView.h 2016-09-06 21:18:06 UTC (rev 205505)
+++ trunk/Source/WebCore/page/FrameView.h 2016-09-06 21:27:15 UTC (rev 205506)
@@ -258,8 +258,6 @@
void setCannotBlitToWindow();
void setIsOverlapped(bool);
- bool isOverlapped() const { return m_isOverlapped; }
- bool isOverlappedIncludingAncestors() const;
void setContentIsOpaque(bool);
void addSlowRepaintObject(RenderElement*);
@@ -714,7 +712,7 @@
bool m_canHaveScrollbars;
bool m_cannotBlitToWindow;
- bool m_isOverlapped;
+ bool m_isOverlapped { false };
bool m_contentIsOpaque;
Timer m_layoutTimer;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes