@r3600
http://codereview.appspot.com/95079/diff/1/3 File tests/com/google/caja/plugin/domita_test_untrusted.html (right): http://codereview.appspot.com/95079/diff/1/3#newcode2382 Line 2382: // runs without focusing the browser, so onfocus never fires. On 2009/07/20 23:16:06, ihab.awad wrote:
I assume you removed testOnFocus / testOnBlur because it is
essentially dead
code. However, perhaps having commented-out tests in there is better
than just a
TODO that can be easily missed, and at least serves as documentation
for the
sort of thing that needs to be tested?
I deleted it because the thing it's testing is already covered by the testOnReset case below, and an actual test of onfocus/onblur would look very different, so I didn't see any value in keeping the redundant test. http://codereview.appspot.com/95079
