shape-outside and filter styles occur twice in the result of getComputedStyle
authorsimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 Apr 2018 23:06:26 +0000 (23:06 +0000)
committersimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 Apr 2018 23:06:26 +0000 (23:06 +0000)
https://bugs.webkit.org/show_bug.cgi?id=184931

Reviewed by Antti Koivisto.

CSSPropertyShapeOutside and CSSPropertyFilter were duplicated in the list of computedProperties[]
used by getComputedStyle. Remove the duplication.
Source/WebCore:

Tested by existing tests.

* css/CSSComputedStyleDeclaration.cpp:

LayoutTests:

* fast/css/getComputedStyle/computed-style-expected.txt:
* fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
* platform/ios/fast/css/getComputedStyle/computed-style-expected.txt:
* platform/ios/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
* platform/mac-sierra/fast/css/getComputedStyle/computed-style-expected.txt:
* platform/mac-sierra/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
* platform/mac-sierra/svg/css/getComputedStyle-basic-expected.txt:
* platform/mac/fast/css/getComputedStyle/computed-style-expected.txt:
* platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
* platform/mac/svg/css/getComputedStyle-basic-expected.txt:
* svg/css/getComputedStyle-basic-expected.txt:

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

14 files changed:
LayoutTests/ChangeLog
LayoutTests/fast/css/getComputedStyle/computed-style-expected.txt
LayoutTests/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
LayoutTests/platform/ios/fast/css/getComputedStyle/computed-style-expected.txt
LayoutTests/platform/ios/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
LayoutTests/platform/mac-sierra/fast/css/getComputedStyle/computed-style-expected.txt
LayoutTests/platform/mac-sierra/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
LayoutTests/platform/mac-sierra/svg/css/getComputedStyle-basic-expected.txt
LayoutTests/platform/mac/fast/css/getComputedStyle/computed-style-expected.txt
LayoutTests/platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
LayoutTests/platform/mac/svg/css/getComputedStyle-basic-expected.txt
LayoutTests/svg/css/getComputedStyle-basic-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/css/CSSComputedStyleDeclaration.cpp

index bfc9293..33fcebe 100644 (file)
@@ -1,3 +1,25 @@
+2018-04-24  Simon Fraser  <simon.fraser@apple.com>
+
+        shape-outside and filter styles occur twice in the result of getComputedStyle
+        https://bugs.webkit.org/show_bug.cgi?id=184931
+
+        Reviewed by Antti Koivisto.
+
+        CSSPropertyShapeOutside and CSSPropertyFilter were duplicated in the list of computedProperties[]
+        used by getComputedStyle. Remove the duplication.
+
+        * fast/css/getComputedStyle/computed-style-expected.txt:
+        * fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
+        * platform/ios/fast/css/getComputedStyle/computed-style-expected.txt:
+        * platform/ios/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
+        * platform/mac-sierra/fast/css/getComputedStyle/computed-style-expected.txt:
+        * platform/mac-sierra/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
+        * platform/mac-sierra/svg/css/getComputedStyle-basic-expected.txt:
+        * platform/mac/fast/css/getComputedStyle/computed-style-expected.txt:
+        * platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
+        * platform/mac/svg/css/getComputedStyle-basic-expected.txt:
+        * svg/css/getComputedStyle-basic-expected.txt:
+
 2018-04-24  Youenn Fablet  <youenn@apple.com>
 
         Throw in case of PeerConnection created for detached documents
index 8f964fc..f7efb40 100644 (file)
@@ -217,7 +217,6 @@ perspective: none;
 perspective-origin: 392px 288px;
 -webkit-print-color-adjust: economy;
 -webkit-rtl-ordering: logical;
-shape-outside: none;
 -webkit-text-combine: none;
 -webkit-text-decorations-in-effect: none;
 -webkit-text-emphasis-color: rgb(0, 0, 0);
@@ -241,7 +240,6 @@ clip-path: none;
 clip-rule: nonzero;
 mask: none;
 mask-type: luminance;
-filter: none;
 flood-color: rgb(0, 0, 0);
 flood-opacity: 1;
 lighting-color: rgb(255, 255, 255);
index 776bc18..55812ff 100644 (file)
@@ -216,7 +216,6 @@ perspective: none
 perspective-origin: 50% 50%
 -webkit-print-color-adjust: economy
 -webkit-rtl-ordering: logical
-shape-outside: none
 -webkit-text-combine: none
 -webkit-text-decorations-in-effect: none
 -webkit-text-emphasis-color: rgb(0, 0, 0)
@@ -240,7 +239,6 @@ clip-path: none
 clip-rule: nonzero
 mask: none
 mask-type: luminance
-filter: none
 flood-color: rgb(0, 0, 0)
 flood-opacity: 1
 lighting-color: rgb(255, 255, 255)
index 77e568a..f0a8634 100644 (file)
@@ -218,7 +218,6 @@ perspective: none;
 perspective-origin: 392px 288px;
 -webkit-print-color-adjust: economy;
 -webkit-rtl-ordering: logical;
-shape-outside: none;
 -webkit-text-combine: none;
 -webkit-text-decorations-in-effect: none;
 -webkit-text-emphasis-color: rgb(0, 0, 0);
@@ -242,7 +241,6 @@ clip-path: none;
 clip-rule: nonzero;
 mask: none;
 mask-type: luminance;
-filter: none;
 flood-color: rgb(0, 0, 0);
 flood-opacity: 1;
 lighting-color: rgb(255, 255, 255);
index 5ebca69..c7d86bc 100644 (file)
@@ -217,7 +217,6 @@ perspective: none
 perspective-origin: 50% 50%
 -webkit-print-color-adjust: economy
 -webkit-rtl-ordering: logical
-shape-outside: none
 -webkit-text-combine: none
 -webkit-text-decorations-in-effect: none
 -webkit-text-emphasis-color: rgb(0, 0, 0)
index 8f964fc..f7efb40 100644 (file)
@@ -217,7 +217,6 @@ perspective: none;
 perspective-origin: 392px 288px;
 -webkit-print-color-adjust: economy;
 -webkit-rtl-ordering: logical;
-shape-outside: none;
 -webkit-text-combine: none;
 -webkit-text-decorations-in-effect: none;
 -webkit-text-emphasis-color: rgb(0, 0, 0);
@@ -241,7 +240,6 @@ clip-path: none;
 clip-rule: nonzero;
 mask: none;
 mask-type: luminance;
-filter: none;
 flood-color: rgb(0, 0, 0);
 flood-opacity: 1;
 lighting-color: rgb(255, 255, 255);
index 776bc18..55812ff 100644 (file)
@@ -216,7 +216,6 @@ perspective: none
 perspective-origin: 50% 50%
 -webkit-print-color-adjust: economy
 -webkit-rtl-ordering: logical
-shape-outside: none
 -webkit-text-combine: none
 -webkit-text-decorations-in-effect: none
 -webkit-text-emphasis-color: rgb(0, 0, 0)
@@ -240,7 +239,6 @@ clip-path: none
 clip-rule: nonzero
 mask: none
 mask-type: luminance
-filter: none
 flood-color: rgb(0, 0, 0)
 flood-opacity: 1
 lighting-color: rgb(255, 255, 255)
index d0ee3d9..1280bb0 100644 (file)
@@ -432,8 +432,6 @@ rect: style.getPropertyValue(-webkit-print-color-adjust) : economy
 rect: style.getPropertyCSSValue(-webkit-print-color-adjust) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-rtl-ordering) : logical
 rect: style.getPropertyCSSValue(-webkit-rtl-ordering) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(shape-outside) : none
-rect: style.getPropertyCSSValue(shape-outside) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-text-combine) : none
 rect: style.getPropertyCSSValue(-webkit-text-combine) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-text-decorations-in-effect) : none
@@ -480,8 +478,6 @@ rect: style.getPropertyValue(mask) : none
 rect: style.getPropertyCSSValue(mask) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(mask-type) : luminance
 rect: style.getPropertyCSSValue(mask-type) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(filter) : none
-rect: style.getPropertyCSSValue(filter) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(flood-color) : rgb(0, 0, 0)
 rect: style.getPropertyCSSValue(flood-color) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(flood-opacity) : 1
@@ -982,8 +978,6 @@ g: style.getPropertyValue(-webkit-print-color-adjust) : economy
 g: style.getPropertyCSSValue(-webkit-print-color-adjust) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-rtl-ordering) : logical
 g: style.getPropertyCSSValue(-webkit-rtl-ordering) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(shape-outside) : none
-g: style.getPropertyCSSValue(shape-outside) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-text-combine) : none
 g: style.getPropertyCSSValue(-webkit-text-combine) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-text-decorations-in-effect) : none
@@ -1030,8 +1024,6 @@ g: style.getPropertyValue(mask) : none
 g: style.getPropertyCSSValue(mask) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(mask-type) : luminance
 g: style.getPropertyCSSValue(mask-type) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(filter) : none
-g: style.getPropertyCSSValue(filter) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(flood-color) : rgb(0, 0, 0)
 g: style.getPropertyCSSValue(flood-color) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(flood-opacity) : 1
index 7aab44d..efd6e09 100644 (file)
@@ -218,7 +218,6 @@ perspective: none;
 perspective-origin: 392px 288px;
 -webkit-print-color-adjust: economy;
 -webkit-rtl-ordering: logical;
-shape-outside: none;
 -webkit-text-combine: none;
 -webkit-text-decorations-in-effect: none;
 -webkit-text-emphasis-color: rgb(0, 0, 0);
@@ -242,7 +241,6 @@ clip-path: none;
 clip-rule: nonzero;
 mask: none;
 mask-type: luminance;
-filter: none;
 flood-color: rgb(0, 0, 0);
 flood-opacity: 1;
 lighting-color: rgb(255, 255, 255);
index 768560e..46d0ccc 100644 (file)
@@ -217,7 +217,6 @@ perspective: none
 perspective-origin: 50% 50%
 -webkit-print-color-adjust: economy
 -webkit-rtl-ordering: logical
-shape-outside: none
 -webkit-text-combine: none
 -webkit-text-decorations-in-effect: none
 -webkit-text-emphasis-color: rgb(0, 0, 0)
@@ -241,7 +240,6 @@ clip-path: none
 clip-rule: nonzero
 mask: none
 mask-type: luminance
-filter: none
 flood-color: rgb(0, 0, 0)
 flood-opacity: 1
 lighting-color: rgb(255, 255, 255)
index 5080092..4bb06e1 100644 (file)
@@ -434,8 +434,6 @@ rect: style.getPropertyValue(-webkit-print-color-adjust) : economy
 rect: style.getPropertyCSSValue(-webkit-print-color-adjust) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-rtl-ordering) : logical
 rect: style.getPropertyCSSValue(-webkit-rtl-ordering) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(shape-outside) : none
-rect: style.getPropertyCSSValue(shape-outside) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-text-combine) : none
 rect: style.getPropertyCSSValue(-webkit-text-combine) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-text-decorations-in-effect) : none
@@ -482,8 +480,6 @@ rect: style.getPropertyValue(mask) : none
 rect: style.getPropertyCSSValue(mask) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(mask-type) : luminance
 rect: style.getPropertyCSSValue(mask-type) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(filter) : none
-rect: style.getPropertyCSSValue(filter) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(flood-color) : rgb(0, 0, 0)
 rect: style.getPropertyCSSValue(flood-color) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(flood-opacity) : 1
@@ -986,8 +982,6 @@ g: style.getPropertyValue(-webkit-print-color-adjust) : economy
 g: style.getPropertyCSSValue(-webkit-print-color-adjust) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-rtl-ordering) : logical
 g: style.getPropertyCSSValue(-webkit-rtl-ordering) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(shape-outside) : none
-g: style.getPropertyCSSValue(shape-outside) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-text-combine) : none
 g: style.getPropertyCSSValue(-webkit-text-combine) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-text-decorations-in-effect) : none
@@ -1034,8 +1028,6 @@ g: style.getPropertyValue(mask) : none
 g: style.getPropertyCSSValue(mask) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(mask-type) : luminance
 g: style.getPropertyCSSValue(mask-type) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(filter) : none
-g: style.getPropertyCSSValue(filter) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(flood-color) : rgb(0, 0, 0)
 g: style.getPropertyCSSValue(flood-color) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(flood-opacity) : 1
index d0ee3d9..1280bb0 100644 (file)
@@ -432,8 +432,6 @@ rect: style.getPropertyValue(-webkit-print-color-adjust) : economy
 rect: style.getPropertyCSSValue(-webkit-print-color-adjust) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-rtl-ordering) : logical
 rect: style.getPropertyCSSValue(-webkit-rtl-ordering) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(shape-outside) : none
-rect: style.getPropertyCSSValue(shape-outside) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-text-combine) : none
 rect: style.getPropertyCSSValue(-webkit-text-combine) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-text-decorations-in-effect) : none
@@ -480,8 +478,6 @@ rect: style.getPropertyValue(mask) : none
 rect: style.getPropertyCSSValue(mask) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(mask-type) : luminance
 rect: style.getPropertyCSSValue(mask-type) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(filter) : none
-rect: style.getPropertyCSSValue(filter) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(flood-color) : rgb(0, 0, 0)
 rect: style.getPropertyCSSValue(flood-color) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(flood-opacity) : 1
@@ -982,8 +978,6 @@ g: style.getPropertyValue(-webkit-print-color-adjust) : economy
 g: style.getPropertyCSSValue(-webkit-print-color-adjust) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-rtl-ordering) : logical
 g: style.getPropertyCSSValue(-webkit-rtl-ordering) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(shape-outside) : none
-g: style.getPropertyCSSValue(shape-outside) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-text-combine) : none
 g: style.getPropertyCSSValue(-webkit-text-combine) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-text-decorations-in-effect) : none
@@ -1030,8 +1024,6 @@ g: style.getPropertyValue(mask) : none
 g: style.getPropertyCSSValue(mask) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(mask-type) : luminance
 g: style.getPropertyCSSValue(mask-type) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(filter) : none
-g: style.getPropertyCSSValue(filter) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(flood-color) : rgb(0, 0, 0)
 g: style.getPropertyCSSValue(flood-color) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(flood-opacity) : 1
index f0554ba..8d75737 100644 (file)
@@ -1,3 +1,17 @@
+2018-04-24  Simon Fraser  <simon.fraser@apple.com>
+
+        shape-outside and filter styles occur twice in the result of getComputedStyle
+        https://bugs.webkit.org/show_bug.cgi?id=184931
+
+        Reviewed by Antti Koivisto.
+
+        CSSPropertyShapeOutside and CSSPropertyFilter were duplicated in the list of computedProperties[]
+        used by getComputedStyle. Remove the duplication.
+        
+        Tested by existing tests.
+
+        * css/CSSComputedStyleDeclaration.cpp:
+
 2018-04-24  Basuke Suzuki  <Basuke.Suzuki@sony.com>
 
         [Curl] Extract proxy settings into a separate class to hold advanced information.
index 923ab54..bc4f53a 100644 (file)
@@ -397,7 +397,6 @@ static const CSSPropertyID computedProperties[] = {
 #if PLATFORM(IOS)
     CSSPropertyWebkitTouchCallout,
 #endif
-    CSSPropertyShapeOutside,
 #if ENABLE(TOUCH_EVENTS)
     CSSPropertyWebkitTapHighlightColor,
 #endif
@@ -430,7 +429,6 @@ static const CSSPropertyID computedProperties[] = {
     CSSPropertyCy,
     CSSPropertyMask,
     CSSPropertyMaskType,
-    CSSPropertyFilter,
     CSSPropertyFloodColor,
     CSSPropertyFloodOpacity,
     CSSPropertyLightingColor,