new snapshot
https://codereview.appspot.com/13171045/diff/1/src/com/google/caja/ses/explicit.html
File src/com/google/caja/ses/explicit.html (right):
https://codereview.appspot.com/13171045/diff/1/src/com/google/caja/ses/explicit.html#newcode73
src/com/google/caja/ses/explicit.html:73: var testSignals =
gebi('testSignals');
On 2013/08/23 17:46:08, kpreid2 wrote:
I'd like to see the Caja test glue factored out so that the separation
between
SES issues and Caja issues is clearer.
Done.
https://codereview.appspot.com/13171045/diff/1/tests/com/google/caja/plugin/BrowserTestCase.java
File tests/com/google/caja/plugin/BrowserTestCase.java (right):
https://codereview.appspot.com/13171045/diff/1/tests/com/google/caja/plugin/BrowserTestCase.java#newcode179
tests/com/google/caja/plugin/BrowserTestCase.java:179:
label.startsWith("guest-scan-") ||
On 2013/08/23 17:46:08, kpreid2 wrote:
For now, at least make this method return false and override it in
MainBrowserTest.
Done.
https://codereview.appspot.com/13171045/diff/1/tests/com/google/caja/plugin/browser-tests.json
File tests/com/google/caja/plugin/browser-tests.json (right):
https://codereview.appspot.com/13171045/diff/1/tests/com/google/caja/plugin/browser-tests.json#newcode14
tests/com/google/caja/plugin/browser-tests.json:14: "label":
"ses-explicit",
On 2013/08/23 17:46:08, kpreid2 wrote:
This should go in ses-tests.json, no?
Done.
https://codereview.appspot.com/13171045/
--
---
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.
