Loading...

google-caja-discuss@googlegroups.com

[Prev] Thread [Next]  |  [Prev] Date [Next]

[Caja] Re: Namespace CSS selectors in sanitizecss.js (issue 5967066) jasvir Fri Apr 06 11:02:30 2012

LGTM


http://codereview.appspot.com/5967066/diff/5003/src/com/google/caja/plugin/sanitizecss.js
File src/com/google/caja/plugin/sanitizecss.js (right):

http://codereview.appspot.com/5967066/diff/5003/src/com/google/caja/plugin/sanitizecss.js#newcode395
src/com/google/caja/plugin/sanitizecss.js:395: safeSelector =
'.vdoc-body___.' + suffix + safeSelector.substring(4);
Can you add a TODO here to erradicate .vdoc-body___ handling.  I don't
see any need for it any more.

http://codereview.appspot.com/5967066/