Changeset 119294 in webkit

Timestamp:

Jun 1, 2012 3:53:53 PM (12 years ago)

Author:

abarth@webkit.org

Message:

Overflow scrolling doesn't need to create a stacking context is the overflow is hidden
​https://bugs.webkit.org/show_bug.cgi?id=88057

Reviewed by James Robinson.

This patch causes us to match the iOS implementation of this feature.

• css/StyleResolver.cpp:

(WebCore::StyleResolver::collectMatchingRulesForList):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• css/StyleResolver.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-06-01  Adam Barth  <abarth@webkit.org>
+
+        Overflow scrolling doesn't need to create a stacking context is the overflow is hidden
+        https://bugs.webkit.org/show_bug.cgi?id=88057
+
+        Reviewed by James Robinson.
+
+        This patch causes us to match the iOS implementation of this feature.
+
+        * css/StyleResolver.cpp:
+        (WebCore::StyleResolver::collectMatchingRulesForList):
+
 2012-06-01  James Robinson  <jamesr@chromium.org>
 

trunk/Source/WebCore/css/StyleResolver.cpp

@@ -2072 +2072 @@
 #if ENABLE(OVERFLOW_SCROLLING)
         // Touch overflow scrolling creates a stacking context.
-        || style->useTouchOverflowScrolling()
+        || ((style->overflowX() != OHIDDEN || style->overflowY() != OHIDDEN) && style->useTouchOverflowScrolling())
 #endif
         ))