Title: [211884] trunk/Source/WebCore
Revision
211884
Author
[email protected]
Date
2017-02-08 10:33:46 -0800 (Wed, 08 Feb 2017)

Log Message

Simple line layout: Fix printReason for unsupported overflow values.
https://bugs.webkit.org/show_bug.cgi?id=168000

Reviewed by Simon Fraser.

Not testable.

* rendering/SimpleLineLayout.cpp:
(WebCore::SimpleLineLayout::canUseForStyle):
(WebCore::SimpleLineLayout::printReason):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (211883 => 211884)


--- trunk/Source/WebCore/ChangeLog	2017-02-08 18:02:47 UTC (rev 211883)
+++ trunk/Source/WebCore/ChangeLog	2017-02-08 18:33:46 UTC (rev 211884)
@@ -1,3 +1,16 @@
+2017-02-08  Zalan Bujtas  <[email protected]>
+
+        Simple line layout: Fix printReason for unsupported overflow values.
+        https://bugs.webkit.org/show_bug.cgi?id=168000
+
+        Reviewed by Simon Fraser.
+
+        Not testable.
+
+        * rendering/SimpleLineLayout.cpp:
+        (WebCore::SimpleLineLayout::canUseForStyle):
+        (WebCore::SimpleLineLayout::printReason):
+
 2017-02-08  Aakash Jain  <[email protected]>
 
         Remove WebCoreObjCDeallocWithWebThreadLock from header file

Modified: trunk/Source/WebCore/rendering/SimpleLineLayout.cpp (211883 => 211884)


--- trunk/Source/WebCore/rendering/SimpleLineLayout.cpp	2017-02-08 18:02:47 UTC (rev 211883)
+++ trunk/Source/WebCore/rendering/SimpleLineLayout.cpp	2017-02-08 18:33:46 UTC (rev 211884)
@@ -76,7 +76,7 @@
     FlowHasUnsupportedFloat               = 1LLU  << 10,
     FlowHasUnsupportedUnderlineDecoration = 1LLU  << 11,
     FlowHasJustifiedNonLatinText          = 1LLU  << 12,
-    FlowHasOverflowVisible                = 1LLU  << 13,
+    FlowHasOverflowNotVisible             = 1LLU  << 13,
     FlowHasWebKitNBSPMode                 = 1LLU  << 14,
     FlowIsNotLTR                          = 1LLU  << 15,
     FlowHasLineBoxContainProperty         = 1LLU  << 16,
@@ -256,7 +256,7 @@
         SET_REASON_AND_RETURN_IF_NEEDED(FlowHasUnsupportedUnderlineDecoration, reasons, includeReasons);
     // Non-visible overflow should be pretty easy to support.
     if (style.overflowX() != OVISIBLE || style.overflowY() != OVISIBLE)
-        SET_REASON_AND_RETURN_IF_NEEDED(FlowHasOverflowVisible, reasons, includeReasons);
+        SET_REASON_AND_RETURN_IF_NEEDED(FlowHasOverflowNotVisible, reasons, includeReasons);
     if (!style.isLeftToRightDirection())
         SET_REASON_AND_RETURN_IF_NEEDED(FlowIsNotLTR, reasons, includeReasons);
     if (!(style.lineBoxContain() & LineBoxContainBlock))
@@ -995,8 +995,8 @@
     case FlowHasJustifiedNonLatinText:
         stream << "text-align: justify with non-latin text";
         break;
-    case FlowHasOverflowVisible:
-        stream << "overflow: visible";
+    case FlowHasOverflowNotVisible:
+        stream << "overflow: hidden | scroll | auto";
         break;
     case FlowHasWebKitNBSPMode:
         stream << "-webkit-nbsp-mode: space";
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to