Updates:
        Status: Started

Comment #1 on issue 1837 by kpreid.switchb.org: Canvas does not accept rgb() colors
http://code.google.com/p/google-caja/issues/detail?id=1837

In matchesStyleFully() in the canvas taming, we're assuming false things about the whitespace in the CSS sanitizer's output. It looks like this might have been broken due to the recent CSS function parsing simplification (or it never worked; don't know for sure, as there is no test for rgb() colors in fillStyle/strokeStyle).

I have forgotten why I wrote the taming to use a predicate for colors rather than passing the sanitized value to the underlying canvas, which would seem to be the sensible thing to do given our architecture. So, I'll try switching to that.

--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--

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