Remove overflow: -webkit-marquee
authorzalan@apple.com <zalan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 16 Mar 2016 01:51:12 +0000 (01:51 +0000)
committerzalan@apple.com <zalan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 16 Mar 2016 01:51:12 +0000 (01:51 +0000)
https://bugs.webkit.org/show_bug.cgi?id=155517
<rdar://problem/25028481>

Reviewed by Simon Fraser.

This patch is based on Blink patch from jchaffraix@chromium.org (https://src.chromium.org/viewvc/blink?revision=151756&view=revision)

Source/WebCore:

* css/CSSParser.cpp:
(WebCore::isValidKeywordPropertyAndValue):
* css/CSSPrimitiveValueMappings.h:
(WebCore::CSSPrimitiveValue::CSSPrimitiveValue): Deleted.
(WebCore::CSSPrimitiveValue::operator EOverflow): Deleted.
* css/CSSValueKeywords.in:
* css/StyleResolver.cpp:
(WebCore::StyleResolver::adjustRenderStyle):
* css/html.css:
(marquee): Deleted.
* rendering/RenderBox.cpp:
(WebCore::RenderBox::sizesLogicalWidthToFitContent):
* rendering/RenderLayer.cpp:
(WebCore::RenderLayer::scrollTo):
(WebCore::RenderLayer::updateScrollInfoAfterLayout):
(WebCore::RenderLayer::calculateClipRects):
* rendering/RenderLayer.h:
* rendering/RenderMarquee.h:
* rendering/style/RenderStyleConstants.h:

LayoutTests:

* fast/css/getPropertyValue-webkit-marquee.html:
* fast/css/webkit-marquee-anonymous-node-crash-expected.txt: Removed.
* fast/css/webkit-marquee-anonymous-node-crash.html: Removed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@198255 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 files changed:
LayoutTests/ChangeLog
LayoutTests/fast/css/getPropertyValue-webkit-marquee.html
LayoutTests/fast/css/webkit-marquee-anonymous-node-crash-expected.txt [deleted file]
LayoutTests/fast/css/webkit-marquee-anonymous-node-crash.html [deleted file]
Source/WebCore/ChangeLog
Source/WebCore/css/CSSParser.cpp
Source/WebCore/css/CSSPrimitiveValueMappings.h
Source/WebCore/css/CSSValueKeywords.in
Source/WebCore/css/StyleResolver.cpp
Source/WebCore/css/html.css
Source/WebCore/rendering/RenderBox.cpp
Source/WebCore/rendering/RenderLayer.cpp
Source/WebCore/rendering/RenderLayer.h
Source/WebCore/rendering/RenderMarquee.h
Source/WebCore/rendering/style/RenderStyleConstants.h

index 9bad3ec..e22373a 100644 (file)
@@ -1,3 +1,17 @@
+2016-03-15  Zalan Bujtas  <zalan@apple.com>
+
+        Remove overflow: -webkit-marquee
+        https://bugs.webkit.org/show_bug.cgi?id=155517
+        <rdar://problem/25028481>
+
+        Reviewed by Simon Fraser.
+
+        This patch is based on Blink patch from jchaffraix@chromium.org (https://src.chromium.org/viewvc/blink?revision=151756&view=revision)
+
+        * fast/css/getPropertyValue-webkit-marquee.html:
+        * fast/css/webkit-marquee-anonymous-node-crash-expected.txt: Removed.
+        * fast/css/webkit-marquee-anonymous-node-crash.html: Removed.
+
 2016-03-15  Joanmarie Diggs  <jdiggs@igalia.com>
 
         AX: Expose pointers to SVG elements referenced by aria-labelledby
index 3e8d01d..c4d5cb3 100644 (file)
@@ -7,7 +7,6 @@
         display: inline-block;
         height: 5em;
         width: 10em;
-        overflow-x: -webkit-marquee;
       }
     </style>
     <script src="../../resources/js-test-pre.js"></script>
     </script>
   </head>
   <body>
-    <div id="marquee1" class="test" style="-webkit-marquee: forwards medium 10 slide normal;"></div>
-    <div id="marquee2" class="test" style="-webkit-marquee: auto large 10 slide;"></div>
-    <div id="marquee3" class="test" style="-webkit-marquee: forwards infinite slide normal;"></div>
-    <div id="marquee4" class="test" style="-webkit-marquee: forwards medium slide normal;"></div>
-    <div id="marquee5" class="test" style="-webkit-marquee: medium 10 slide normal;"></div>
-    <div id="marquee6" class="test" style="-webkit-marquee: forwards small normal;"></div>
-    <div id="marquee7" class="test" style="-webkit-marquee: forwards medium infinite;"></div>
-    <div id="marquee8" class="test" style="-webkit-marquee: slide fast;"></div>
-    <div id="marquee9" class="test" style="-webkit-marquee: forwards;"></div>
-    <div id="marquee10" class="test" style="-webkit-marquee: alternate;"></div>
-    <div id="marquee11" class="test" style="-webkit-marquee: infinite;"></div>
-    <div id="marquee12" class="test" style="-webkit-marquee: foo;"></div>
-    <div id="marquee13" class="test" style="-webkit-marquee-direction: forwards; -webkit-marquee-speed: fast;"></div>
-    <div id="marquee14" class="test" style="-webkit-marquee-repetition: infinite; -webkit-marquee-style: none;"></div>
+    <marquee id="marquee1" class="test" style="-webkit-marquee: forwards medium 10 slide normal;"></<marquee>
+    <marquee id="marquee2" class="test" style="-webkit-marquee: auto large 10 slide;"></marquee>
+    <marquee id="marquee3" class="test" style="-webkit-marquee: forwards infinite slide normal;"></marquee>
+    <marquee id="marquee4" class="test" style="-webkit-marquee: forwards medium slide normal;"></marquee>
+    <marquee id="marquee5" class="test" style="-webkit-marquee: medium 10 slide normal;"></marquee>
+    <marquee id="marquee6" class="test" style="-webkit-marquee: forwards small normal;"></marquee>
+    <marquee id="marquee7" class="test" style="-webkit-marquee: forwards medium infinite;"></marquee>
+    <marquee id="marquee8" class="test" style="-webkit-marquee: slide fast;"></marquee>
+    <marquee id="marquee9" class="test" style="-webkit-marquee: forwards;"></marquee>
+    <marquee id="marquee10" class="test" style="-webkit-marquee: alternate;"></marquee>
+    <marquee id="marquee11" class="test" style="-webkit-marquee: infinite;"></marquee>
+    <marquee id="marquee12" class="test" style="-webkit-marquee: foo;"></marquee>
+    <marquee id="marquee13" class="test" style="-webkit-marquee-direction: forwards; -webkit-marquee-speed: fast;"></marquee>
+    <marquee id="marquee14" class="test" style="-webkit-marquee-repetition: infinite; -webkit-marquee-style: none;"></marquee>
 
     <script>
       description("<a href=\"https://bugs.webkit.org/show_bug.cgi?id=110816\">Bug 110816: getPropertyValue for -webkit-marquee returns null, should compute the shorthand value</a>");
diff --git a/LayoutTests/fast/css/webkit-marquee-anonymous-node-crash-expected.txt b/LayoutTests/fast/css/webkit-marquee-anonymous-node-crash-expected.txt
deleted file mode 100644 (file)
index a5da3bb..0000000
+++ /dev/null
@@ -1 +0,0 @@
-Should not crash
diff --git a/LayoutTests/fast/css/webkit-marquee-anonymous-node-crash.html b/LayoutTests/fast/css/webkit-marquee-anonymous-node-crash.html
deleted file mode 100644 (file)
index 7e36863..0000000
+++ /dev/null
@@ -1,9 +0,0 @@
-<!DOCTYPE html>\r
-<style>p:first-letter { overflow: -webkit-marquee; float: left; }</style>\r
-<script>\r
-if (window.testRunner)\r
-    testRunner.dumpAsText();\r
-</script>\r
-<body>\r
-    <p>Should not crash</p>\r
-</body>\r
index e355616..9089e33 100644 (file)
@@ -1,3 +1,33 @@
+2016-03-15  Zalan Bujtas  <zalan@apple.com>
+
+        Remove overflow: -webkit-marquee
+        https://bugs.webkit.org/show_bug.cgi?id=155517
+        <rdar://problem/25028481>
+
+        Reviewed by Simon Fraser.
+
+        This patch is based on Blink patch from jchaffraix@chromium.org (https://src.chromium.org/viewvc/blink?revision=151756&view=revision)
+
+        * css/CSSParser.cpp:
+        (WebCore::isValidKeywordPropertyAndValue):
+        * css/CSSPrimitiveValueMappings.h:
+        (WebCore::CSSPrimitiveValue::CSSPrimitiveValue): Deleted.
+        (WebCore::CSSPrimitiveValue::operator EOverflow): Deleted.
+        * css/CSSValueKeywords.in:
+        * css/StyleResolver.cpp:
+        (WebCore::StyleResolver::adjustRenderStyle):
+        * css/html.css:
+        (marquee): Deleted.
+        * rendering/RenderBox.cpp:
+        (WebCore::RenderBox::sizesLogicalWidthToFitContent):
+        * rendering/RenderLayer.cpp:
+        (WebCore::RenderLayer::scrollTo):
+        (WebCore::RenderLayer::updateScrollInfoAfterLayout):
+        (WebCore::RenderLayer::calculateClipRects):
+        * rendering/RenderLayer.h:
+        * rendering/RenderMarquee.h:
+        * rendering/style/RenderStyleConstants.h:
+
 2016-03-15  Joanmarie Diggs  <jdiggs@igalia.com>
 
         AX: Expose pointers to SVG elements referenced by aria-labelledby
index 40b8469..27e6a39 100644 (file)
@@ -731,12 +731,12 @@ static inline bool isValidKeywordPropertyAndValue(CSSPropertyID propertyId, int
             return true;
         break;
 #endif
-    case CSSPropertyOverflowX: // visible | hidden | scroll | auto | marquee | overlay | inherit
-        if (valueID == CSSValueVisible || valueID == CSSValueHidden || valueID == CSSValueScroll || valueID == CSSValueAuto || valueID == CSSValueOverlay || valueID == CSSValueWebkitMarquee)
+    case CSSPropertyOverflowX: // visible | hidden | scroll | auto  | overlay | inherit
+        if (valueID == CSSValueVisible || valueID == CSSValueHidden || valueID == CSSValueScroll || valueID == CSSValueAuto || valueID == CSSValueOverlay)
             return true;
         break;
-    case CSSPropertyOverflowY: // visible | hidden | scroll | auto | marquee | overlay | inherit | -webkit-paged-x | -webkit-paged-y
-        if (valueID == CSSValueVisible || valueID == CSSValueHidden || valueID == CSSValueScroll || valueID == CSSValueAuto || valueID == CSSValueOverlay || valueID == CSSValueWebkitMarquee || valueID == CSSValueWebkitPagedX || valueID == CSSValueWebkitPagedY)
+    case CSSPropertyOverflowY: // visible | hidden | scroll | auto | overlay | inherit | -webkit-paged-x | -webkit-paged-y
+        if (valueID == CSSValueVisible || valueID == CSSValueHidden || valueID == CSSValueScroll || valueID == CSSValueAuto || valueID == CSSValueOverlay || valueID == CSSValueWebkitPagedX || valueID == CSSValueWebkitPagedY)
             return true;
         break;
     case CSSPropertyPageBreakAfter: // auto | always | avoid | left | right | inherit
index a9b4893..5b40d20 100644 (file)
@@ -2180,9 +2180,6 @@ template<> inline CSSPrimitiveValue::CSSPrimitiveValue(EOverflow e)
     case OAUTO:
         m_value.valueID = CSSValueAuto;
         break;
-    case OMARQUEE:
-        m_value.valueID = CSSValueWebkitMarquee;
-        break;
     case OOVERLAY:
         m_value.valueID = CSSValueOverlay;
         break;
@@ -2208,8 +2205,6 @@ template<> inline CSSPrimitiveValue::operator EOverflow() const
         return OSCROLL;
     case CSSValueAuto:
         return OAUTO;
-    case CSSValueWebkitMarquee:
-        return OMARQUEE;
     case CSSValueOverlay:
         return OOVERLAY;
     case CSSValueWebkitPagedX:
index 8ef4691..df916e8 100644 (file)
@@ -522,7 +522,6 @@ line-through
 local
 loud
 lower
--webkit-marquee
 mix
 no-close-quote
 no-open-quote
index 3d2af52..af78b7b 100644 (file)
@@ -888,12 +888,14 @@ void StyleResolver::adjustRenderStyle(RenderStyle& style, const RenderStyle& par
     else
         style.addToTextDecorationsInEffect(style.textDecoration());
 
+    // For now, <marquee> requires an overflow clip to work properly.
+    if (is<HTMLMarqueeElement>(e)) {
+        style.setOverflowX(OHIDDEN);
+        style.setOverflowY(OHIDDEN);
+    }
+
     // If either overflow value is not visible, change to auto.
-    if (style.overflowX() == OMARQUEE && style.overflowY() != OMARQUEE)
-        style.setOverflowY(OMARQUEE);
-    else if (style.overflowY() == OMARQUEE && style.overflowX() != OMARQUEE)
-        style.setOverflowX(OMARQUEE);
-    else if (style.overflowX() == OVISIBLE && style.overflowY() != OVISIBLE) {
+    if (style.overflowX() == OVISIBLE && style.overflowY() != OVISIBLE) {
         // FIXME: Once we implement pagination controls, overflow-x should default to hidden
         // if overflow-y is set to -webkit-paged-x or -webkit-page-y. For now, we'll let it
         // default to auto so we can at least scroll through the pages.
index 5e5deb7..4910964 100644 (file)
@@ -53,7 +53,6 @@ address, article, aside, div, footer, header, hgroup, layer, main, nav, section
 
 marquee {
     display: inline-block;
-    overflow: -webkit-marquee;
 }
 
 blockquote {
index 388875f..6dbe3d1 100644 (file)
@@ -2572,7 +2572,7 @@ bool RenderBox::sizesLogicalWidthToFitContent(SizeType widthType) const
     // FIXME: Think about block-flow here.  Need to find out how marquee direction relates to
     // block-flow (as well as how marquee overflow should relate to block flow).
     // https://bugs.webkit.org/show_bug.cgi?id=46472
-    if (parent()->style().overflowX() == OMARQUEE) {
+    if (parent()->isHTMLMarquee()) {
         EMarqueeDirection dir = parent()->style().marqueeDirection();
         if (dir == MAUTO || dir == MFORWARD || dir == MBACKWARD || dir == MLEFT || dir == MRIGHT)
             return true;
index fbe4aab..ba84516 100644 (file)
@@ -2347,7 +2347,7 @@ void RenderLayer::scrollTo(const ScrollPosition& position)
         return;
 
     ScrollPosition newPosition = position;
-    if (box->style().overflowX() != OMARQUEE) {
+    if (!box->isHTMLMarquee()) {
         // Ensure that the dimensions will be computed if they need to be (for overflow:hidden blocks).
         if (m_scrollDimensionsDirty)
             computeScrollDimensions();
@@ -3495,7 +3495,7 @@ void RenderLayer::updateScrollInfoAfterLayout()
     updateSnapOffsets();
 #endif
 
-    if (box->style().overflowX() != OMARQUEE && !isRubberBandInProgress()) {
+    if (!box->isHTMLMarquee() && !isRubberBandInProgress()) {
         // Layout may cause us to be at an invalid scroll position. In this case we need
         // to pull our scroll offsets back to the max (or push them up to the min).
         ScrollOffset clampedScrollOffset = clampScrollOffset(scrollOffset());
@@ -6675,7 +6675,7 @@ void RenderLayer::styleChanged(StyleDifference diff, const RenderStyle* oldStyle
         dirtyStackingContainerZOrderLists();
     }
 
-    if (renderer().style().overflowX() == OMARQUEE && renderer().style().marqueeBehavior() != MNONE && renderer().isBox()) {
+    if (renderer().isHTMLMarquee() && renderer().style().marqueeBehavior() != MNONE && renderer().isBox()) {
         if (!m_marquee)
             m_marquee = std::make_unique<RenderMarquee>(this);
         m_marquee->updateMarqueeStyle();
index 7ddb7ef..94f8e91 100644 (file)
@@ -1122,7 +1122,7 @@ private:
     
     IntPoint m_cachedOverlayScrollbarOffset;
 
-    std::unique_ptr<RenderMarquee> m_marquee; // Used by layers with overflow:marquee
+    std::unique_ptr<RenderMarquee> m_marquee; // Used for <marquee>.
     
     // Cached normal flow values for absolute positioned elements with static left/top values.
     LayoutUnit m_staticInlinePosition;
index 57bb745..a98a88e 100644 (file)
@@ -52,7 +52,7 @@ namespace WebCore {
 
 class RenderLayer;
 
-// This class handles the auto-scrolling of layers with overflow: marquee.
+// This class handles the auto-scrolling for <marquee>
 class RenderMarquee final {
     WTF_MAKE_NONCOPYABLE(RenderMarquee); WTF_MAKE_FAST_ALLOCATED;
 public:
index 85de7bd..b9b0dce 100644 (file)
@@ -191,7 +191,7 @@ enum EBoxSizing { CONTENT_BOX, BORDER_BOX };
 // Random visual rendering model attributes. Not inherited.
 
 enum EOverflow {
-    OVISIBLE, OHIDDEN, OSCROLL, OAUTO, OOVERLAY, OMARQUEE, OPAGEDX, OPAGEDY
+    OVISIBLE, OHIDDEN, OSCROLL, OAUTO, OOVERLAY, OPAGEDX, OPAGEDY
 };
 
 enum EVerticalAlign {