Fixes some Eclipse complaints and 1076

Removes the cast to Object in canReadRev so that 'length' in '123'
throws a TypeError properly.

[MikeSamuel, 2009-07-22 22:11:01]

LGTM

Should we consider making all test methods final so that they can't be
unintentionally overridden?

http://codereview.appspot.com/97047/diff/1/5
File src/com/google/caja/valija-cajita.js (right):

http://codereview.appspot.com/97047/diff/1/5#newcode450
Line 450: if (name in obj) { return true; }
So this is the significant change?

[MikeSamuel, 2009-07-22 22:11:30]

On 2009/07/22 22:11:01, MikeSamuel wrote:
> LGTM
> 
> Should we consider making all test methods final so that they can't be
> unintentionally overridden?

I didn't mean to suggest that that should happen in this change.

> http://codereview.appspot.com/97047/diff/1/5
> File src/com/google/caja/valija-cajita.js (right):
> 
> http://codereview.appspot.com/97047/diff/1/5#newcode450
> Line 450: if (name in obj) { return true; }
> So this is the significant change?

[metaweta, 2009-07-22 22:13:04]

On 2009/07/22 22:11:01, MikeSamuel wrote:
\> http://codereview.appspot.com/97047/diff/1/5#newcode450
> Line 450: if (name in obj) { return true; }
> So this is the significant change?

Yes, this is the significant one.