Revision: 5332
Author: [email protected]
Date: Thu Mar 28 16:17:01 2013
Log: Really fix whitelisting of cajaVM.es5ProblemReports.
https://codereview.appspot.com/8064045
r5331 was slightly wrong (yes, this is a fix of a fix) in that it added
es5ProblemReports, let it be deleted by the whitelist, and then added it
again. This is largely harmless, but confusing and not warning-clean (it
causes the extra early copy to be reported as deleted).
[email protected]
http://code.google.com/p/google-caja/source/detail?r=5332
Modified:
/trunk/src/com/google/caja/ses/startSES.js
=======================================
--- /trunk/src/com/google/caja/ses/startSES.js Thu Mar 28 15:47:59 2013
+++ /trunk/src/com/google/caja/ses/startSES.js Thu Mar 28 16:17:01 2013
@@ -1140,9 +1140,11 @@
makeImports: constFunc(makeImports),
copyToImports: constFunc(copyToImports),
- makeArrayLike: constFunc(makeArrayLike),
+ makeArrayLike: constFunc(makeArrayLike)
- es5ProblemReports: ses.es5ProblemReports
+ // Not defined here because it cannot be whitelisted; see assignment
and
+ // comments below.
+ //es5ProblemReports: ses.es5ProblemReports
};
var extensionsRecord = extensions();
gopn(extensionsRecord).forEach(function (p) {
--
---
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.