http://codereview.appspot.com/1008044/diff/1/3 File java/gadgets/src/main/java/org/apache/shindig/gadgets/render/RenderModule.java (right):
http://codereview.appspot.com/1008044/diff/1/3#newcode42 java/gadgets/src/main/java/org/apache/shindig/gadgets/render/RenderModule.java:42: protected Set<String> getAllowedTags() { as a matter of convention, I'd prefer provideFoo rather than getFoo. The latter sounds more like an accessor to me. http://codereview.appspot.com/1008044/show
