Title: [119294] trunk/Source/WebCore
- Revision
- 119294
- Author
- [email protected]
- Date
- 2012-06-01 15:53:53 -0700 (Fri, 01 Jun 2012)
Log 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):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (119293 => 119294)
--- trunk/Source/WebCore/ChangeLog 2012-06-01 22:50:51 UTC (rev 119293)
+++ trunk/Source/WebCore/ChangeLog 2012-06-01 22:53:53 UTC (rev 119294)
@@ -1,3 +1,15 @@
+2012-06-01 Adam Barth <[email protected]>
+
+ 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 <[email protected]>
[chromium] Unreviewed compile fix for r119283
Modified: trunk/Source/WebCore/css/StyleResolver.cpp (119293 => 119294)
--- trunk/Source/WebCore/css/StyleResolver.cpp 2012-06-01 22:50:51 UTC (rev 119293)
+++ trunk/Source/WebCore/css/StyleResolver.cpp 2012-06-01 22:53:53 UTC (rev 119294)
@@ -2071,7 +2071,7 @@
#endif
#if ENABLE(OVERFLOW_SCROLLING)
// Touch overflow scrolling creates a stacking context.
- || style->useTouchOverflowScrolling()
+ || ((style->overflowX() != OHIDDEN || style->overflowY() != OHIDDEN) && style->useTouchOverflowScrolling())
#endif
))
style->setZIndex(0);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
