You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@shindig.apache.org by ja...@gmail.com on 2010/01/25 01:19:40 UTC

Experimental patch: Separate quick render of html from slower javascript execution (issue194049)

Reviewers: shishir.birmiwal, johnfargo,

Message:
* patch currently contains the http://codereview.appspot.com/194041/show
- I'll snapshot to remove that once its committed.  The interesting
changes are only in caja/taming.js

Description:
SanitizingGadgetRewriter neuters a gadget by removing all css, all
javascript and those html tags that aren't on its whitelist.  This is
used "to create a simple badge like display" useful when low latency and
inactive content is require - for example for a view with many gadgets.

This patch provides a lower latency alternative for caja gadgets.  It:
* immediately renders the html and inline css portions of a gadget
creating the "badge like display" SanitizingGadgetRewriter does
* adds a button to the top of the gadget which when clicked "rehydrates"
the gadget and executes all the javascript and renders remaining css
styles


Please review this at http://codereview.appspot.com/194049/show

Affected files:
   M     features/src/main/javascript/features/caja/taming.js
   M     features/src/main/javascript/features/core.util/taming.js
   M     features/src/main/javascript/features/opensocial-reference/taming.js
   M      
java/gadgets/src/test/java/org/apache/shindig/gadgets/parse/caja/CajaSocialMarkupHtmlParserTest.java
   M      
java/server/src/test/java/org/apache/shindig/server/endtoend/EndToEndTest.java
   M     java/server/src/test/resources/endtoend/cajaTest.xml
   M     java/server/src/test/resources/endtoend/failCajaTest.xml
   M     pom.xml