https://codereview.appspot.com/6827077/diff/37001/src/com/google/caja/ses/exportsToSES.js File src/com/google/caja/ses/exportsToSES.js (right):
https://codereview.appspot.com/6827077/diff/37001/src/com/google/caja/ses/exportsToSES.js#newcode33 src/com/google/caja/ses/exportsToSES.js:33: delete window.exports; At top level, the platform independent way to refer to the global is "this". "window" is browser specific, and is otherwise non-standard. https://codereview.appspot.com/6827077/diff/50001/src/com/google/caja/ses/startSES.js File src/com/google/caja/ses/startSES.js (right): https://codereview.appspot.com/6827077/diff/50001/src/com/google/caja/ses/startSES.js#newcode544 src/com/google/caja/ses/startSES.js:544: return newValue; This return is useless, since a setter's return value is ignored. (The value of the triggering assignment expression is the value of the right hand side regardless.) https://codereview.appspot.com/6827077/
