Reviewers: MikeSamuel,

Description:
The css whitelist has an element 'dom2property' that's intended to
map a css name like 'margin-left' to a dom name like 'marginLeft'.

I'm eliminating it, because:

1. It's unnecessary. The name conversion is regular except for
float->cssFloat, and it's unlikely that any other css properties
will be irregular.

2. It's a maintenance hazard. Specifying an incorrect dom2property
will cause mysterious failures. And failing to specify a correct
dom2property may also cause mysterious failures.

There was also legacy support for style->styleFloat instead of cssFloat,
which is for IE<=8, which we're not supporting, so I'm removing the
styleFloat code.

Please review this at https://codereview.appspot.com/13060045/

Affected files:
  M     src/com/google/caja/lang/css/CssPropertyPatterns.java
  M     src/com/google/caja/lang/css/CssSchema.java
  M     src/com/google/caja/lang/css/css3-defs.json
  M     src/com/google/caja/plugin/domado.js
  M     tests/com/google/caja/lang/css/CssPropertyPatternsTest.java
  M     tests/com/google/caja/lang/css/CssSchemaTest.java
  M     tests/com/google/caja/plugin/test-domado-dom-guest.html
  M     tests/com/google/caja/plugin/test-domado-special-guest.html


--

--- You received this message because you are subscribed to the Google Groups "Google Caja Discuss" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to