Fix incorrect feature tests in SES and Domado.

Fixes the following:

inTestFrame used for repairES5 repairs
http://code.google.com/p/google-caja/issues/detail?id=1917

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

Distinguishing between direct-proxies and old style proxies
http://code.google.com/p/google-caja/issues/detail?id=1919

@r5685

[MarkM, 2014-05-15 20:51:51]

https://codereview.appspot.com/91450047/diff/1/src/com/google/caja/ses/repair...
File src/com/google/caja/ses/repairES5.js (right):

https://codereview.appspot.com/91450047/diff/1/src/com/google/caja/ses/repair...
src/com/google/caja/ses/repairES5.js:2574: if (typeof global.DataView ===
'undefined') { return false; }
This is unnecessary. Do one of

if (global.DataView === undefined) // without the quotes, or better, void 0
if (!('DataView' in global))
if (typeof DataView === 'undefined')

https://codereview.appspot.com/91450047/diff/1/src/com/google/caja/ses/repair...
src/com/google/caja/ses/repairES5.js:2575: if (typeof global.DOMException ===
'undefined') { return false; }
Likewise

https://codereview.appspot.com/91450047/diff/1/src/com/google/caja/ses/startS...
File src/com/google/caja/ses/startSES.js (right):

https://codereview.appspot.com/91450047/diff/1/src/com/google/caja/ses/startS...
src/com/google/caja/ses/startSES.js:1146: var proxiesAvailable = typeof
global.Proxy !== 'undefined' &&
Again, if you're indirecting off of 'global.', there's no reason for the typeof.

https://codereview.appspot.com/91450047/diff/1/tests/com/google/caja/plugin/b...
File tests/com/google/caja/plugin/browser-test-case.js (right):

https://codereview.appspot.com/91450047/diff/1/tests/com/google/caja/plugin/b...
tests/com/google/caja/plugin/browser-test-case.js:550: typeof
frameGroup.iframe.Proxy !== 'undefined' &&
The combination of typeof and "." is still unneeded.

https://codereview.appspot.com/91450047/diff/1/tests/com/google/caja/plugin/t...
File tests/com/google/caja/plugin/test-scan-guest.js (right):

https://codereview.appspot.com/91450047/diff/1/tests/com/google/caja/plugin/t...
tests/com/google/caja/plugin/test-scan-guest.js:753:
argsByIdentity(window.Proxy, G.none);  // TODO abuse
What is this TODO about?

[kpreid_google, 2014-05-15 23:42:32]

Fixes the following:

inTestFrame used for repairES5 repairs
http://code.google.com/p/google-caja/issues/detail?id=1917

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

Distinguishing between direct-proxies and old style proxies
http://code.google.com/p/google-caja/issues/detail?id=1919

[kpreid_google, 2014-05-15 23:45:24]

https://codereview.appspot.com/91450047/diff/1/src/com/google/caja/ses/repair...
File src/com/google/caja/ses/repairES5.js (right):

https://codereview.appspot.com/91450047/diff/1/src/com/google/caja/ses/repair...
src/com/google/caja/ses/repairES5.js:2574: if (typeof global.DataView ===
'undefined') { return false; }
On 2014/05/15 20:51:51, MarkM wrote:
> This is unnecessary. Do one of
> 
> if (global.DataView === undefined) // without the quotes, or better, void 0

Done. I dispute that void 0 is better, on readability grounds.

https://codereview.appspot.com/91450047/diff/1/src/com/google/caja/ses/repair...
src/com/google/caja/ses/repairES5.js:2575: if (typeof global.DOMException ===
'undefined') { return false; }
On 2014/05/15 20:51:51, MarkM wrote:
> Likewise

Done.

https://codereview.appspot.com/91450047/diff/1/src/com/google/caja/ses/startS...
File src/com/google/caja/ses/startSES.js (right):

https://codereview.appspot.com/91450047/diff/1/src/com/google/caja/ses/startS...
src/com/google/caja/ses/startSES.js:1146: var proxiesAvailable = typeof
global.Proxy !== 'undefined' &&
On 2014/05/15 20:51:51, MarkM wrote:
> Again, if you're indirecting off of 'global.', there's no reason for the
typeof.

Done.

https://codereview.appspot.com/91450047/diff/1/tests/com/google/caja/plugin/b...
File tests/com/google/caja/plugin/browser-test-case.js (right):

https://codereview.appspot.com/91450047/diff/1/tests/com/google/caja/plugin/b...
tests/com/google/caja/plugin/browser-test-case.js:550: typeof
frameGroup.iframe.Proxy !== 'undefined' &&
On 2014/05/15 20:51:51, MarkM wrote:
> The combination of typeof and "." is still unneeded.

Done.

https://codereview.appspot.com/91450047/diff/1/tests/com/google/caja/plugin/t...
File tests/com/google/caja/plugin/test-scan-guest.js (right):

https://codereview.appspot.com/91450047/diff/1/tests/com/google/caja/plugin/t...
tests/com/google/caja/plugin/test-scan-guest.js:753:
argsByIdentity(window.Proxy, G.none);  // TODO abuse
On 2014/05/15 20:51:51, MarkM wrote:
> What is this TODO about?

The scanner doesn't exercise Proxy behavior, and it should.

Referring to this as "abuse" is because the scanner's job is to look for things
like edge cases from funny arguments, most of which are expected to fail.

[MarkM, 2014-05-16 03:00:44]

LGTM