LGTM beyond this comment.

http://codereview.appspot.com/1240043/diff/9001/10001
File
java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultIframeUriManager.java
(right):

http://codereview.appspot.com/1240043/diff/9001/10001#newcode172
java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultIframeUriManager.java:172:
protected String generateSecurityToken(Gadget gadget) {
didn't enjoy the wantsSecurityToken(gadget) idea?

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

Reply via email to