Generate fast-path CssStyleDeclaration properties from browser info

tools/dom/scripts/css_code_generator.py

 #!/usr/bin/python
 #
 # Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS file
 # for details. All rights reserved. Use of this source code is governed by a
 # BSD-style license that can be found in the LICENSE file.
 
 """Generates CSSStyleDeclaration template file from css property definitions
 defined in WebKit."""
 
-import tempfile, os
+import tempfile, os, re
 
 COMMENT_LINE_PREFIX = '   * '
 # TODO(efortuna): Pull from DEPS so that we have latest css *in sync* with our
 # Dartium. Then remove the checked in CSSPropertyNames.in.
 SOURCE_PATH = 'CSSPropertyNames.in'
 #SOURCE_PATH = 'Source/WebCore/css/CSSPropertyNames.in'
 TEMPLATE_FILE = '../templates/html/impl/impl_CSSStyleDeclaration.darttemplate'
 
+# These are the properties that are supported on all Dart project supported
+# browsers as camelCased names on the CssStyleDeclaration.
+BROWSER_PATHS = [
+    'cssProperties.ie9.txt',

[Alan Knight, 2015/03/04 17:49:32]

Shouldn't we be doing ie10 and 11 rather than 9 and 10?

[sra1, 2015/03/25 05:23:35]

Yes, but to not include ie9 would break ie9 which currently mostly works.
We should only break ie9 deliberately.

[Alan Knight, 2015/03/25 17:51:05]

OK. But shouldn't there be IE11?

+    'cssProperties.ie10.txt',
+    'cssProperties.ff36.txt',
+    'cssProperties.chrome40.txt',
+    'cssProperties.safari-7.1.3.txt',
+    ]
+
 # Supported annotations for any specific CSS properties.
 annotated = {
   'transition': '''@SupportedBrowser(SupportedBrowser.CHROME)
   @SupportedBrowser(SupportedBrowser.FIREFOX)
   @SupportedBrowser(SupportedBrowser.IE, '10')
   @SupportedBrowser(SupportedBrowser.SAFARI)'''
 }
 
+class Error:
+  def __init__(self, message):
+    self.message = message
+  def __repr__(self):
+    return self.message
+
 def camelCaseName(name):
   """Convert a CSS property name to a lowerCamelCase name."""
   name = name.replace('-webkit-', '')
   words = []
   for word in name.split('-'):
     if words:
       words.append(word.title())
     else:
       words.append(word)
   return ''.join(words)
 
+def dashifyName(camelName):
+  def fix(match):
+    return '-' + match.group(0).lower()
+  return re.sub(r'[A-Z]', fix, camelName)
+
+def isCommentLine(line):
+  return line.strip() == '' or line.startswith('#') or line.startswith('//')
+
+def readCssProperties(filename):
+  data = open(filename).readlines()
+  data = sorted([d.strip() for d in set(data) if not isCommentLine(d)])
+  return data
+
 def GenerateCssTemplateFile():
   data = open(SOURCE_PATH).readlines()
 
   # filter CSSPropertyNames.in to only the properties
   # TODO(efortuna): do we also want CSSPropertyNames.in?
-  data = [d[:-1] for d in data
-          if len(d) > 1
-          and not d.startswith('#')
-          and not d.startswith('//')
+  data = [d.strip() for d in data
+          if not isCommentLine(d)
           and not '=' in d]
 
+  browser_props = [readCssProperties(file) for file in BROWSER_PATHS]
+  universal_properties = reduce(
+        lambda a, b: set(a).intersection(b), browser_props)
+  universal_properties = universal_properties.difference(['cssText'])
+  universal_properties = universal_properties.intersection(
+        map(camelCaseName, data))
+
   class_file = open(TEMPLATE_FILE, 'w')
 
   class_file.write("""
 // Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 // WARNING: DO NOT EDIT THIS TEMPLATE FILE.
 // The template file was generated by scripts/css_code_generator.py
 
 // Source of CSS properties:
 //   %s
 
 part of $LIBRARYNAME;
+""" % SOURCE_PATH)
 
-$(ANNOTATIONS)$(NATIVESPEC)$(CLASS_MODIFIERS) class $CLASSNAME $EXTENDS with
+
+  class_file.write("""
+$(ANNOTATIONS)$(NATIVESPEC)$(CLASS_MODIFIERS)class $CLASSNAME $EXTENDS with
     $(CLASSNAME)Base $IMPLEMENTS {
   factory $CLASSNAME() => new CssStyleDeclaration.css('');
 
   factory $CLASSNAME.css(String css) {
     final style = new Element.tag('div').style;
     style.cssText = css;
     return style;
   }
 
   String getPropertyValue(String propertyName) {
@@ skipping 94 matching lines @@
     }
     _setProperty(propertyName, value, priority);
   }
 
   /**
    * Checks to see if CSS Transitions are supported.
    */
   static bool get supportsTransitions => true;
 $endif
 $!MEMBERS
+$if DART2JS
+""")
+
+  for camelName in sorted(universal_properties):
+    property = dashifyName(camelName)
+    class_file.write("""
+  /** Gets the value of "%s" */
+  String get %s => JS('String', '#.%s', this);
+
+  /** Sets the value of "%s" */
+  void set %s(String value) {
+    JS('', '#.%s = #', this, value == null ? '' : value);
+  }
+    """ % (property, camelName, camelName, property, camelName, camelName))
+
+  class_file.write("""
+$endif
 }
 
 class _CssStyleDeclarationSet extends Object with CssStyleDeclarationBase {
   final Iterable<Element> _elementIterable;
   Iterable<CssStyleDeclaration> _elementCssStyleDeclarationSetIterable;
 
   _CssStyleDeclarationSet(this._elementIterable) {
     _elementCssStyleDeclarationSetIterable = new List.from(
         _elementIterable).map((e) => e.style);
   }
 
   String getPropertyValue(String propertyName) =>
       _elementCssStyleDeclarationSetIterable.first.getPropertyValue(
           propertyName);
 
   void setProperty(String propertyName, String value, [String priority]) {
     _elementCssStyleDeclarationSetIterable.forEach((e) =>
         e.setProperty(propertyName, value, priority));
   }
   // Important note: CssStyleDeclarationSet does NOT implement every method
   // available in CssStyleDeclaration. Some of the methods don't make so much
   // sense in terms of having a resonable value to return when you're
   // considering a list of Elements. You will need to manually add any of the
   // items in the MEMBERS set if you want that functionality.
 }
 
 abstract class CssStyleDeclarationBase {
   String getPropertyValue(String propertyName);
   void setProperty(String propertyName, String value, [String priority]);
-""" % SOURCE_PATH)
+""")
 
   class_lines = [];
 
   seen = set()
-  for prop in sorted(data, key=lambda p: camelCaseName(p)):
+  for prop in sorted(data, key=camelCaseName):
     camel_case_name = camelCaseName(prop)
     upper_camel_case_name = camel_case_name[0].upper() + camel_case_name[1:];
     css_name = prop.replace('-webkit-', '')
     base_css_name = prop.replace('-webkit-', '')
 
     if base_css_name in seen or base_css_name.startswith('-internal'):
       continue
     seen.add(base_css_name)
 
     comment = '  /** %s the value of "' + base_css_name + '" */'
@@ skipping 12 matching lines @@
       class_lines.append(annotated[base_css_name])
     class_lines.append("""
   void set %s(String value) {
     setProperty('%s', value, '');
   }
 """ % (camel_case_name, css_name))
 
   class_file.write(''.join(class_lines));
   class_file.write('}\n')
   class_file.close()