Update dart2js test status

Update dart2js test status

Committed: https://code.google.com/p/dart/source/detail?r=12086

[ahe, 2012-09-08 08:37:22]

Submitted TBR

https://chromiumcodereview.appspot.com/10911167/diff/1/dart/tests/language/la...
File dart/tests/language/language_dart2js.status (right):

https://chromiumcodereview.appspot.com/10911167/diff/1/dart/tests/language/la...
dart/tests/language/language_dart2js.status:47: metadata_test: Fail # Metadata
on type parameters not supported.
$host_unchecked is when dart2js is running on the VM's production mode.

See http://code.google.com/p/dart/wiki/TestingDart2js

https://chromiumcodereview.appspot.com/10911167/diff/1/dart/tests/language/la...
dart/tests/language/language_dart2js.status:50: metadata_test: Crash # Metadata
on type parameters not supported.
$host_checked is when dart2js is running on the VM's developer mode.

So what is happening here is that there is a dynamic type error in the
implementation of dart2js. We report that as a crash. This is not a VM crash.

[ricow1, 2012-09-08 19:26:08]

LGTM

https://chromiumcodereview.appspot.com/10911167/diff/1/dart/tests/co19/co19-d...
File dart/tests/co19/co19-dart2js.status (right):

https://chromiumcodereview.appspot.com/10911167/diff/1/dart/tests/co19/co19-d...
dart/tests/co19/co19-dart2js.status:548: 
I know this is probably a lost battle, but could we file a bug and ask the
people that made these 3 tests if they really need to take this long (if they do
- fine - otherwise this is wasting hours of cycle time a day)

https://chromiumcodereview.appspot.com/10911167/diff/1/dart/tests/language/la...
File dart/tests/language/language_dart2js.status (right):

https://chromiumcodereview.appspot.com/10911167/diff/1/dart/tests/language/la...
dart/tests/language/language_dart2js.status:50: metadata_test: Crash # Metadata
on type parameters not supported.
On 2012/09/08 08:37:22, ahe wrote:
> $host_checked is when dart2js is running on the VM's developer mode.
> 
> So what is happening here is that there is a dynamic type error in the
> implementation of dart2js. We report that as a crash. This is not a VM crash.
I think that makes perfekt sense (since this is a crash of the compiler), but
you might want to make this explicit in a comment here (I have not seen  Crash
marks on dart2js a lot, but it might very well confuse people)

[ahe, 2012-09-08 19:54:30]

https://chromiumcodereview.appspot.com/10911167/diff/1/dart/tests/co19/co19-d...
File dart/tests/co19/co19-dart2js.status (right):

https://chromiumcodereview.appspot.com/10911167/diff/1/dart/tests/co19/co19-d...
dart/tests/co19/co19-dart2js.status:548: 
On 2012/09/08 19:26:08, ricow1 wrote:
> I know this is probably a lost battle, but could we file a bug and ask the
> people that made these 3 tests if they really need to take this long (if they
do
> - fine - otherwise this is wasting hours of cycle time a day)

On my laptop, this test completes in 1 second in unchecked mode. In checked
mode, it takes 19 seconds.

For what its worth, the sort test completes in a quarter of a second. But it
takes 15 seconds to compile.

In my opinion, the problem is not the tests. The problem is dart2js.

[hausner, 2012-09-10 15:35:42]

lgtm