Updates:
        Status: Pending

Comment #6 on issue 1878 by kpreid.switchb.org: Refactor repairES5
http://code.google.com/p/google-caja/issues/detail?id=1878

https://codereview.appspot.com/54450044/

This:
* extracts the test/repair/report algorithm from repairES5.js and makes all its state non-global
* converts WeakMap.js to use it
* makes ses.ok() answer as to whether we're all done, not whether we should continue loading

It does _not_:
* add support for registering and testing a problem and then installing a repair later
* replace the nasty "_wasApplied" flags or the special extra test runs

I figured what I've done so far is enough of a substantial and coherent change to not want to add more to it.

Note that the WeakMap.js changes will conflict with https://codereview.appspot.com/54750043/ due to indentation and declaration changes only; the implementation of OurWeakMap is unaltered, but moved into a function.

--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--

--- You received this message because you are subscribed to the Google Groups "Google Caja Discuss" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to