[WebUI] Fix multiple rule regex.

chrome/browser/resources/web_dev_style/css_checker.py

Index: chrome/browser/resources/web_dev_style/css_checker.py
diff --git a/chrome/browser/resources/web_dev_style/css_checker.py b/chrome/browser/resources/web_dev_style/css_checker.py
index db7222b3d4e9616a1ac55fb46772643a736bd99b..502196d2587ce18223da00393193c8db798cc057 100644
--- a/chrome/browser/resources/web_dev_style/css_checker.py
+++ b/chrome/browser/resources/web_dev_style/css_checker.py
@@ -65,7 +65,7 @@ class CSSChecker(object):
       return (line.find('}') >= 0 and re.search(r'[^ }]', line))
 
     def colons_have_space_after(line):
-      return re.search(r':(?!\/\/)\S(?!.*[{,]\s*$)', line)
+      return re.search(r':(?!\/\/)\S[^;]+;\s*', line)
 
     def favor_single_quotes(line):
       return line.find('"') >= 0
@@ -82,7 +82,7 @@ class CSSChecker(object):
       return re.search(small_seconds, line)
 
     def one_rule_per_line(line):
-      return re.search('(.*:(?!\/\/)){2,}(?!.*[,{]\s*$)', line)
+      return re.search('[_a-zA-Z0-9-]:(?!\/\/)[^;]+;\s*[^ }]\s*', line)

[Tyler Breisacher (Chromium), 2012/03/21 20:30:03]

You don't have to escape the slashes. So (?!\/\/) can be just (?!//). Same at
line 68. I'm also not totally following what you're doing here. AFAIK CSS only
allows /* this type */ of comments, not '// this type' so why are we checking
for '//' anyway?

[Dan Beam, 2012/03/21 20:47:29]

this is ignoring the : in protocols as in:
  background-image: url(chrome://resources/BLAH);
                              ^ had issues with this

 
     any_reg = re.compile(r':(?:-webkit-)?any\(.*?\)', re.DOTALL)
     multi_sels = re.compile(r'(?:}[\n\s]*)?([^,]+,(?=[^{}]+?{).*[,{])\s*$',