[CSSTypedOM] Computed StylePropertyMap use ComputedStyle for Lengths

third_party/WebKit/LayoutTests/typedcssom/computedstyle/computedStylePropertyMap.html

Index: third_party/WebKit/LayoutTests/typedcssom/computedstyle/computedStylePropertyMap.html
diff --git a/third_party/WebKit/LayoutTests/typedcssom/computedstyle/computedStylePropertyMap.html b/third_party/WebKit/LayoutTests/typedcssom/computedstyle/computedStylePropertyMap.html
index 5140e8bf4b26b8062d3dd3780ab723cd8bd0f511..504b5318f992280d17333294d852ea1ce08ff00d 100644
--- a/third_party/WebKit/LayoutTests/typedcssom/computedstyle/computedStylePropertyMap.html
+++ b/third_party/WebKit/LayoutTests/typedcssom/computedstyle/computedStylePropertyMap.html
@@ -21,19 +21,6 @@ test(function() {
 }, "Computed StyleMap.getProperties returns the same number of properties as ComputedStyle");
 
 test(function() {
-  var properties = computedStyleMap.getProperties();
-  for (var i = 0; i < properties.length; i++) {
-    var property = properties[i];
-    var styleValues = computedStyleMap.getAll(property);
-    if (styleValues.length == 1) {
-      // TODO(meade): Remove this if statement once all properties are supported.
-      // TODO(meade): Handle sequence types.
-      assert_equals(computedStyleMap.get(property).cssText, computedStyle[property]);
-    }
-  }
-}, 'Properties retrieved from ComputedStyleMap reflect the same values as from ComputedStyle');

[meade_UTC10, 2016/12/08 23:22:15]

Is removing this test necessary? Could you also add an expectation instead?

[meade_UTC10, 2017/01/05 03:43:45]

Only some of the properties will be different right? We can write this test
differently to account for those ones.

-
-test(function() {
   testElement.style.border = '1px solid #00ff00';
   var styleValue = computedStyleMap.get('border');
   assert_equals(styleValue.constructor, CSSStyleValue);