Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(610)

Unified Diff: third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp

Issue 2658333002: Don't omit ScrollableAreas in non-clickable FrameViews. (Closed)
Patch Set: Remove obsolete FIXME Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « third_party/WebKit/LayoutTests/fast/scrolling/scrollable-area-frame-scrolling-yes-display-none-expected.txt ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp
diff --git a/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp b/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp
index 4265852bc64dcbcd41b4ff229e49a871c644774e..9827ef4f23b74bfe435803879bb92928c3035e96 100644
--- a/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp
+++ b/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp
@@ -1665,15 +1665,8 @@ void PaintLayerScrollableArea::updateScrollableAreaSet(bool hasOverflow) {
if (!frameView)
return;
- // FIXME: Does this need to be fixed later for OOPI?
bool isVisibleToHitTest = box().style()->visibleToHitTesting();
- if (HTMLFrameOwnerElement* owner = frame->deprecatedLocalOwner()) {
- isVisibleToHitTest &= owner->layoutObject() &&
- owner->layoutObject()->style()->visibleToHitTesting();
- }
-
bool didScrollOverflow = m_scrollsOverflow;
-
m_scrollsOverflow = hasOverflow && isVisibleToHitTest;
if (didScrollOverflow == scrollsOverflow())
return;
« no previous file with comments | « third_party/WebKit/LayoutTests/fast/scrolling/scrollable-area-frame-scrolling-yes-display-none-expected.txt ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698