snapshot
https://codereview.appspot.com/6827077/diff/18001/src/com/google/caja/ses/exportsToSES.js
File src/com/google/caja/ses/exportsToSES.js (right):
https://codereview.appspot.com/6827077/diff/18001/src/com/google/caja/ses/exportsToSES.js#newcode26
src/com/google/caja/ses/exportsToSES.js:26: ses.generate =
exports.generate;
On 2012/11/19 19:06:01, kpreid2 wrote:
Maybe stuff these on a sub-object of some sort (e.g. ses._modules =
{}), given
that these are not part of the public interface of SES or even
designed to be
cap-safe modules, but have inviting names?
I also think it is worth a comment that we are squashing together the
exports of
three modules (i.e. works-by-accident).
Done.
https://codereview.appspot.com/6827077/diff/18001/tests/com/google/caja/plugin/es53-test-domado-dom-guest.html
File tests/com/google/caja/plugin/es53-test-domado-dom-guest.html
(right):
https://codereview.appspot.com/6827077/diff/18001/tests/com/google/caja/plugin/es53-test-domado-dom-guest.html#newcode2190
tests/com/google/caja/plugin/es53-test-domado-dom-guest.html:2190: //
properly.
On 2012/11/19 19:06:01, kpreid2 wrote:
Obsolete comment.
Done.
https://codereview.appspot.com/6827077/diff/18001/tests/com/google/caja/plugin/es53-test-gotchas-guest.html
File tests/com/google/caja/plugin/es53-test-gotchas-guest.html (right):
https://codereview.appspot.com/6827077/diff/18001/tests/com/google/caja/plugin/es53-test-gotchas-guest.html#newcode30
tests/com/google/caja/plugin/es53-test-gotchas-guest.html:30: <div
class="testcontainer" id="testTypeOf"></div>
On 2012/11/19 19:06:01, kpreid2 wrote:
Move this to immediately before the testTypeOf code (splitting the
script
block).
Done.
https://codereview.appspot.com/6827077/diff/18001/tests/com/google/caja/plugin/es53-test-gotchas-guest.html#newcode32
tests/com/google/caja/plugin/es53-test-gotchas-guest.html:32: <br>
On 2012/11/19 19:06:01, kpreid2 wrote:
cruft
Done.
https://codereview.appspot.com/6827077/
