snapshot
https://codereview.appspot.com/6818060/diff/1/src/com/google/caja/plugin/ses-frame-group.js
File src/com/google/caja/plugin/ses-frame-group.js (right):
https://codereview.appspot.com/6818060/diff/1/src/com/google/caja/plugin/ses-frame-group.js#newcode390
src/com/google/caja/plugin/ses-frame-group.js:390: '(function () {' +
theContent + '\n})()'));
On 2012/10/30 21:55:50, kpreid2 wrote:
Add a comment about why the \n is necessary.
Done.
https://codereview.appspot.com/6818060/diff/1/src/com/google/caja/ses/startSES.js
File src/com/google/caja/ses/startSES.js (right):
https://codereview.appspot.com/6818060/diff/1/src/com/google/caja/ses/startSES.js#newcode703
src/com/google/caja/ses/startSES.js:703: var exprSrc = '(function() {' +
modSrc + '\n}).call(this)';
Not a regression - this was not done everywhere.
https://codereview.appspot.com/6818060/diff/1/tests/com/google/caja/plugin/es53-test-inline-script.html
File tests/com/google/caja/plugin/es53-test-inline-script.html (right):
https://codereview.appspot.com/6818060/diff/1/tests/com/google/caja/plugin/es53-test-inline-script.html#newcode18
tests/com/google/caja/plugin/es53-test-inline-script.html:18:
<script>document.write("<b><script>");//</script
On 2012/10/30 21:55:50, kpreid2 wrote:
Please add a separate test case with an appropriate name. This is too
likely to
get removed.
Since this is a guest-side test, adding another test is not expensive.
Done.
https://codereview.appspot.com/6818060/
