lgtm++

http://codereview.appspot.com/5691043/diff/1/src/com/google/caja/demos/playground/client/ui/PlaygroundView.java
File src/com/google/caja/demos/playground/client/ui/PlaygroundView.java
(right):

http://codereview.appspot.com/5691043/diff/1/src/com/google/caja/demos/playground/client/ui/PlaygroundView.java#newcode499
src/com/google/caja/demos/playground/client/ui/PlaygroundView.java:499:
[email protected]::addRuntimeMessage(Ljava/lang/String;)
We should think of what the risk is, if any, if the error contains HTML,
including <script>s. Either way, I think this needs a comment explaining
the reasoning.

http://codereview.appspot.com/5691043/

Reply via email to