lgtm++
Note remarks.
https://codereview.appspot.com/14441072/diff/1/src/com/google/caja/demos/playground/client/ui/PlaygroundView.java
File src/com/google/caja/demos/playground/client/ui/PlaygroundView.java
(right):
https://codereview.appspot.com/14441072/diff/1/src/com/google/caja/demos/playground/client/ui/PlaygroundView.java#newcode328
src/com/google/caja/demos/playground/client/ui/PlaygroundView.java:328:
// think it's an unknown exploit).
Sorry that didn't parse ... that what is an unknown exploit?
https://codereview.appspot.com/14441072/diff/1/src/com/google/caja/demos/playground/client/ui/PlaygroundView.java#newcode419
src/com/google/caja/demos/playground/client/ui/PlaygroundView.java:419:
frame = frame.code(baseUrl, "text/html", html)
Any way we could squirrel in a semicolon here?
https://codereview.appspot.com/14441072/diff/1/src/com/google/caja/plugin/Config.java
File src/com/google/caja/plugin/Config.java (right):
https://codereview.appspot.com/14441072/diff/1/src/com/google/caja/plugin/Config.java#newcode124
src/com/google/caja/plugin/Config.java:124: private final Option
ID_CLASS = defineOption(
Interesting that this stuffery remains. :) What uses it, if we're no
longer doing Java cajoling?
https://codereview.appspot.com/14441072/
--
---
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.
