On 2013/08/09 20:16:28, kpreid2 wrote:

https://codereview.appspot.com/12699046/diff/1/src/com/google/caja/plugin/domado.js
File src/com/google/caja/plugin/domado.js (right):


https://codereview.appspot.com/12699046/diff/1/src/com/google/caja/plugin/domado.js#newcode4691
src/com/google/caja/plugin/domado.js:4691: if
(/\binherit\b/.test(style)) {
return ''; }
On 2013/08/09 20:10:52, MikeSamuel wrote:
> I believe the way we sanitize functions, tokens.length should be 1
because a
> color is either a hash token, or an rgb function call or a call to
rgba, so
> you could filter too if you passed a predicate that examined the
token
> list to sanitizeCssValue.

 From your description, that sounds more tightly coupled. Is there a
particular
reason to do it that way?

Probably not.  Please ignore.

https://codereview.appspot.com/12699046/

--

--- 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