Title: [240310] trunk/Source/WebCore
Revision
240310
Author
mcatanz...@igalia.com
Date
2019-01-22 16:37:30 -0800 (Tue, 22 Jan 2019)

Log Message

Unreviewed, fix -Wsign-compare warning
https://bugs.webkit.org/show_bug.cgi?id=188697
<rdar://problem/46105624>

* css/StyleProperties.cpp:
(WebCore::StyleProperties::asText const):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (240309 => 240310)


--- trunk/Source/WebCore/ChangeLog	2019-01-23 00:32:59 UTC (rev 240309)
+++ trunk/Source/WebCore/ChangeLog	2019-01-23 00:37:30 UTC (rev 240310)
@@ -1,3 +1,12 @@
+2019-01-22  Michael Catanzaro  <mcatanz...@igalia.com>
+
+        Unreviewed, fix -Wsign-compare warning
+        https://bugs.webkit.org/show_bug.cgi?id=188697
+        <rdar://problem/46105624>
+
+        * css/StyleProperties.cpp:
+        (WebCore::StyleProperties::asText const):
+
 2019-01-22  Devin Rousso  <drou...@apple.com>
 
         Web Inspector: Audit: provide a way to get related Accessibility properties for a given node

Modified: trunk/Source/WebCore/css/StyleProperties.cpp (240309 => 240310)


--- trunk/Source/WebCore/css/StyleProperties.cpp	2019-01-23 00:32:59 UTC (rev 240309)
+++ trunk/Source/WebCore/css/StyleProperties.cpp	2019-01-23 00:37:30 UTC (rev 240310)
@@ -979,7 +979,7 @@
         String value;
         auto serializeBorderShorthand = [&] (const CSSPropertyID borderProperty, const CSSPropertyID fallbackProperty) {
             // FIXME: Deal with cases where only some of border sides are specified.
-            ASSERT(borderProperty - firstCSSProperty < shorthandPropertyAppeared.size());
+            ASSERT(borderProperty - firstCSSProperty < static_cast<CSSPropertyID>(shorthandPropertyAppeared.size()));
             if (!shorthandPropertyAppeared[borderProperty - firstCSSProperty] && isEnabledCSSProperty(borderProperty)) {
                 value = getPropertyValue(borderProperty);
                 if (value.isNull())
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to