LGTM

http://codereview.appspot.com/380041/diff/8001/9003
File
java/gadgets/src/main/java/org/apache/shindig/gadgets/render/Renderer.java
(right):

http://codereview.appspot.com/380041/diff/8001/9003#newcode82
java/gadgets/src/main/java/org/apache/shindig/gadgets/render/Renderer.java:82:
if (requiresCaja(gadget)) {
looks fine, though in a followup we should use this (static-ified)
method in the other places in code that duplicate it ;)

http://codereview.appspot.com/380041/show

Reply via email to