http://codereview.appspot.com/135051/diff/1027/48 File features/src/main/javascript/features/caja/taming.js (right):
http://codereview.appspot.com/135051/diff/1027/48#newcode105 Line 105: var tamings___ = tamings___ || []; This works for now. Its vulnerable to a feature you don't trust resetting this array entirely to prevent it from getting exposed to a gadget but if you have a feature you don't trust, it can do anything anyways. On 2009/10/20 21:53:57, johnfargo wrote:
Not that it's a big deal in this case, but maybe it should be. This is
one of a
few use cases I've seen arise that call for a clearer representation
of the
feature dependency tree.
http://codereview.appspot.com/135051/diff/1027/46 File features/src/main/javascript/features/flash/taming.js (right): http://codereview.appspot.com/135051/diff/1027/46#newcode1 Line 1: /* On 2009/10/20 21:53:57, johnfargo wrote:
Missing a corresponding feature.xml update for flash.
Done. http://codereview.appspot.com/135051
