https://codereview.appspot.com/10648043/diff/5001/src/com/google/caja/plugin/guest-manager.js
File src/com/google/caja/plugin/guest-manager.js (right):
https://codereview.appspot.com/10648043/diff/5001/src/com/google/caja/plugin/guest-manager.js#newcode68
src/com/google/caja/plugin/guest-manager.js:68: if (o.hasOwnProperty(k))
{
On 2013/06/27 20:33:26, kpreid2 wrote:
Use Object.prototype.hasOwnProperty.call just for robustness.
Done.
https://codereview.appspot.com/10648043/diff/5001/tests/com/google/caja/plugin/test-index.js
File tests/com/google/caja/plugin/test-index.js (right):
https://codereview.appspot.com/10648043/diff/5001/tests/com/google/caja/plugin/test-index.js#newcode116
tests/com/google/caja/plugin/test-index.js:116: loadCatalog('tests',
'apitaming-tests.json');
On 2013/06/27 20:33:26, kpreid2 wrote:
Please add
// TODO(kpreid): Refactor to avoid unnecessary latency and
dependencies, perhaps
using Q.
Done.
https://codereview.appspot.com/10648043/
--
---
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.
