Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(161)

Unified Diff: LayoutTests/inspector/elements/styles/mixed-case-color-aware-properties.html

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
Patch Set: Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..b7aada2e7fe2c497443647d8b1f3dd08988050d7
--- /dev/null
+++ b/LayoutTests/inspector/elements/styles/mixed-case-color-aware-properties.html
@@ -0,0 +1,47 @@
+<html>
+<head>
+<script src="../../../http/tests/inspector/inspector-test.js"></script>
+<script src="../../../http/tests/inspector/elements-test.js"></script>
+<script>
+
+function test()
+{
+ function getRandomInteger(min, max) {
+ return min + Math.floor(Math.random() * (max - min + 1));
+ }
+
+ function randomizeStringCase(str)
apavlov 2014/07/14 11:25:53 All tests must be deterministic, so this is not an
+ {
+ var charIndicesForUpperCase = [];
+ for (var i = 0; i < str.length / 2; ++i) {
+ charIndicesForUpperCase.push(getRandomInteger(0, str.length));
+ }
+ str = str.split("");
+ for(i = 0; i < charIndicesForUpperCase.length; i++){
+ if (charIndicesForUpperCase[i] < str.length)
+ str[charIndicesForUpperCase[i]] = str[charIndicesForUpperCase[i]].toUpperCase();
+ }
+ return str.join('');
+ }
+
+ var colorAwareProperties = Object.keys(WebInspector.CSSMetadata._colorAwareProperties);
+ for (var i = 0; i < colorAwareProperties.length; ++i) {
+ if (WebInspector.CSSMetadata.isColorAwareProperty(randomizeStringCase(colorAwareProperties[i])))
+ result = "PASS";
+ else {
+ result = "FAIL";
+ 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.
+</p>
+</body>
+</html>

Powered by Google App Engine
This is Rietveld 408576698