Reviewers: ihab.awad,

Description:
for some reason, a CSS selector like "input:-moz-placeholder" is
a fatal error in es53, and it aborts processing.

I don't see a reason for this to be a fatal eror, so I'm downgrading
it to a warning. It's safe to do that because the warning still
marks that css rule as invalid, and it gets removed later.

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

Affected files:
  M     src/com/google/caja/plugin/PluginMessageType.java


Index: src/com/google/caja/plugin/PluginMessageType.java
===================================================================
--- src/com/google/caja/plugin/PluginMessageType.java   (revision 5451)
+++ src/com/google/caja/plugin/PluginMessageType.java   (working copy)
@@ -56,8 +56,8 @@
       "%s: css property %s with value %s not in range [%s, %s]",
       MessageLevel.WARNING),
   UNSAFE_CSS_IDENTIFIER(
-      "%s: css identifier '%s' contains characters that may not work"
-      + " on all browsers", MessageLevel.FATAL_ERROR),
+      "%s: potentially unsafe css identifier '%s'",
+      MessageLevel.WARNING),
   UNSAFE_CSS_PROPERTY("%s: unsafe css property %s", MessageLevel.ERROR),
   UNSAFE_CSS_PSEUDO_SELECTOR(
       "%s: unsafe css pseudo-selector %s", MessageLevel.ERROR),


--

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