fix onfocus/onblur, and make DomitaTest more reliable

http://code.google.com/p/google-caja/issues/detail?id=1075

1. bridal only attaches to real events if the event name
   corresponds to an html whitelisted attribute '*:onfoo'.
   focus/blur is whitelisted for specific tags, not for '*',
   so the whitelist check fails, and no handler gets attached.
   this change fixes that.

2. DomitaTest is prone to false positives because
   the string 'all tests passed' gets added to the title
   before all tests finish running.
   this change fixes that.

[ihab.awad, 2009-07-16 18:38:50]

LGTM++

See comments.

http://codereview.appspot.com/91117/diff/1/5
File tests/com/google/caja/plugin/DomitaTest.java (right):

http://codereview.appspot.com/91117/diff/1/5#newcode128
Line 128: assertTrue("Too many wait rounds.", waitRounds < waitRoundLimit);
I would suggest formatting the "clicking" and "waiting" loops so they have
identical line spacing, etc., to highlight their similarity.

http://codereview.appspot.com/91117/diff/1/4
File tests/com/google/caja/plugin/domita_test_untrusted.html (right):

http://codereview.appspot.com/91117/diff/1/4#newcode2387
Line 2387: false);
Please indent above lines 2 more spaces. Same elsewhere.

http://codereview.appspot.com/91117/diff/1/3
File tests/com/google/caja/plugin/jsunit.js (right):

http://codereview.appspot.com/91117/diff/1/3#newcode131
Line 131: console.log('FAIL: ' + testName);
Why not also updateStatus() on failure, since you updateStatus() on passing?

[felix8a, 2009-07-16 19:28:29]

On 2009/07/16 18:38:50, ihab.awad wrote:
> LGTM++
> 
> See comments.
> 
> http://codereview.appspot.com/91117/diff/1/5
> File tests/com/google/caja/plugin/DomitaTest.java (right):
> 
> http://codereview.appspot.com/91117/diff/1/5#newcode128
> Line 128: assertTrue("Too many wait rounds.", waitRounds < waitRoundLimit);
> I would suggest formatting the "clicking" and "waiting" loops so they have
> identical line spacing, etc., to highlight their similarity.

ok

> http://codereview.appspot.com/91117/diff/1/4
> File tests/com/google/caja/plugin/domita_test_untrusted.html (right):
> 
> http://codereview.appspot.com/91117/diff/1/4#newcode2387
> Line 2387: false);
> Please indent above lines 2 more spaces. Same elsewhere.

ok

> http://codereview.appspot.com/91117/diff/1/3
> File tests/com/google/caja/plugin/jsunit.js (right):
> 
> http://codereview.appspot.com/91117/diff/1/3#newcode131
> Line 131: console.log('FAIL: ' + testName);
> Why not also updateStatus() on failure, since you updateStatus() on passing?

ah, right.  forgot to do that.