Chromium Code Reviews Chromium Code Reviews
Issue 376803002:
[DevTools] Color values should be case insensitive while suggestions should be case aware (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| Index: LayoutTests/inspector/elements/styles/mixed-case-color-aware-properties.html |
| diff --git a/LayoutTests/inspector/elements/styles/mixed-case-color-aware-properties.html b/LayoutTests/inspector/elements/styles/mixed-case-color-aware-properties.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..366080a18b6a1e46fc1a6797dacd07605604ca45 |
| --- /dev/null |
| +++ b/LayoutTests/inspector/elements/styles/mixed-case-color-aware-properties.html |
| @@ -0,0 +1,29 @@ |
| +<html> |
| +<head> |
| +<script src="../../../http/tests/inspector/inspector-test.js"></script> |
| +<script src="../../../http/tests/inspector/elements-test.js"></script> |
| +<script> |
| + |
| +function test() |
| +{ |
| + var colorAwareProperties = ["bAckground-ColoR", "COloR", "Border-coLoR", "border-right-color", "BOX-SHADOW"]; |
| + for (var i = 0; i < colorAwareProperties.length; ++i) { |
| + if (WebInspector.CSSMetadata.isColorAwareProperty(colorAwareProperties[i])) |
|
apavlov
2014/07/14 12:23:42
braces presence must be consistent across if-arms
apavlov
2014/07/14 12:23:42
We should rely on test expectations rather than so
vivekg
2014/07/14 17:57:51
Done.
vivekg
2014/07/14 17:57:51
Done.
|
| + result = "PASS"; |
| + else { |
| + result = "FAIL: " + colorAwareProperties[i] + " is not listed as color aware property."; |
| + break; |
| + } |
| + } |
| + InspectorTest.addResult("Result: " + result); |
| + InspectorTest.completeTest(); |
| +} |
| +</script> |
| +</head> |
| + |
| +<body onload="runTest()"> |
| +<p> |
| +Tests that color related mixed-cased CSS properties are really color aware. |
|
apavlov
2014/07/14 12:23:42
color related -> color-related
mixed-cased -> mix-
vivekg
2014/07/14 17:57:51
Done.
|
| +</p> |
| +</body> |
| +</html> |
