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

Unified Diff: chrome/browser/web_dev_style/css_checker_test.py

Issue 2345703004: web_dev_style: ignore --css-vars from alphabetical ordering requirement (Closed)
Patch Set: such tragic misspelling Created 4 years, 3 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: chrome/browser/web_dev_style/css_checker_test.py
diff --git a/chrome/browser/web_dev_style/css_checker_test.py b/chrome/browser/web_dev_style/css_checker_test.py
index 1866614ab61ee9e25d523e79e8c58f6bc85303c0..0883ff6e86f7026c0c837444f11b05c5faf0e4d1 100755
--- a/chrome/browser/web_dev_style/css_checker_test.py
+++ b/chrome/browser/web_dev_style/css_checker_test.py
@@ -133,6 +133,13 @@ div {
border-left: 5px;
border: 5px solid red;""")
+ def testAlphaIgnoresVariables(self):
+ self.VerifyContentsIsValid("""
+#id {
+ --zzyxx-xylophone: 3px;
+ --aardvark-animal: var(--zzyxz-xylophone);
dschuyler 2016/09/15 23:17:36 maybe add: --two-line: var(--second-line); -
dschuyler 2016/09/15 23:48:35 With my better understanding of the mixins being r
+}""")
+
def testCssBracesHaveSpaceBeforeAndNothingAfter(self):
self.VerifyContentsProducesOutput("""
/* Hello! */div/* Comment here*/{
« chrome/browser/web_dev_style/css_checker.py ('K') | « chrome/browser/web_dev_style/css_checker.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698