LGTM
https://codereview.appspot.com/11790043/diff/1/tests/com/google/caja/plugin/BrowserTestCase.java
File tests/com/google/caja/plugin/BrowserTestCase.java (right):
https://codereview.appspot.com/11790043/diff/1/tests/com/google/caja/plugin/BrowserTestCase.java#newcode90
tests/com/google/caja/plugin/BrowserTestCase.java:90: serverHost =
ThisHostName.value();
Comment on what's going on here — why this condition goes with this
value. "If WEBDRIVER_URL is set, then we're doing remote testing, so..."
https://codereview.appspot.com/11790043/diff/1/tests/com/google/caja/plugin/Echo.java
File tests/com/google/caja/plugin/Echo.java (right):
https://codereview.appspot.com/11790043/diff/1/tests/com/google/caja/plugin/Echo.java#newcode21
tests/com/google/caja/plugin/Echo.java:21: public class Echo {
a comment talking about why this particular bundle of functions exists
would be nice.
https://codereview.appspot.com/11790043/
--
---
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.
