LGTM

A headsup - when debugging a similar change I was making, I found that
the startBlock and endBlock rules never fire in the snippet that you
moved from html-emitter to sanitizecss (and as a result css blocks are
missing the surrounding braces) but I did not manage to get to the
bottom of it.  I suspect that as a result the tests that this change
adds will fail when you do wire up the tests which would be a good time
to fix that problem.

http://codereview.appspot.com/5784076/

Reply via email to