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

Unified Diff: third_party/WebKit/Source/core/layout/LayoutBox.h

Issue 2916563003: Compute effective touch action in StyleAdjuster. (Closed)
Patch Set: Skip elements that don't need a layout object Created 3 years, 5 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
Index: third_party/WebKit/Source/core/layout/LayoutBox.h
diff --git a/third_party/WebKit/Source/core/layout/LayoutBox.h b/third_party/WebKit/Source/core/layout/LayoutBox.h
index cfc700921e3a53d96ada0f37f05b70a96e0670b8..0d6f6b63292389917575a69259176f450d240875 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBox.h
+++ b/third_party/WebKit/Source/core/layout/LayoutBox.h
@@ -989,17 +989,13 @@ class CORE_EXPORT LayoutBox : public LayoutBoxModelObject {
virtual void StopAutoscroll() {}
DISABLE_CFI_PERF bool HasAutoVerticalScrollbar() const {
- return HasOverflowClip() &&
- (Style()->OverflowY() == EOverflow::kAuto ||
- Style()->OverflowY() == EOverflow::kWebkitPagedY ||
- Style()->OverflowY() == EOverflow::kOverlay);
+ return HasOverflowClip() && Style()->HasAutoVerticalScroll();
}
DISABLE_CFI_PERF bool HasAutoHorizontalScrollbar() const {
- return HasOverflowClip() && (Style()->OverflowX() == EOverflow::kAuto ||
- Style()->OverflowX() == EOverflow::kOverlay);
+ return HasOverflowClip() && Style()->HasAutoHorizontalScroll();
}
DISABLE_CFI_PERF bool ScrollsOverflow() const {
- return ScrollsOverflowX() || ScrollsOverflowY();
+ return HasOverflowClip() && Style()->ScrollsOverflow();
}
virtual bool ShouldPlaceBlockDirectionScrollbarOnLogicalLeft() const {
return Style()->ShouldPlaceBlockDirectionScrollbarOnLogicalLeft();
@@ -1014,12 +1010,10 @@ class CORE_EXPORT LayoutBox : public LayoutBoxModelObject {
PixelSnappedScrollHeight() != PixelSnappedClientHeight();
}
virtual bool ScrollsOverflowX() const {
- return HasOverflowClip() && (Style()->OverflowX() == EOverflow::kScroll ||
- HasAutoHorizontalScrollbar());
+ return HasOverflowClip() && Style()->ScrollsOverflowX();
}
virtual bool ScrollsOverflowY() const {
- return HasOverflowClip() && (Style()->OverflowY() == EOverflow::kScroll ||
- HasAutoVerticalScrollbar());
+ return HasOverflowClip() && Style()->ScrollsOverflowY();
}
// Elements such as the <input> field override this to specify that they are

Powered by Google App Engine
This is Rietveld 408576698