Snapshot. On 2009/10/21 19:03:23, jasvir wrote:
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
