Removed the 'we don't run multitests on browsers'-hack

Removed the 'we don't run multitests on browsers'-hack

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

[ahe, 2013-01-16 10:27:36]

Did you test this on Dartium? It still wraps tests as far as I know, so it will
most likely break.

[kustermann, 2013-01-16 10:29:53]

On 2013/01/16 10:27:36, ahe wrote:
> Did you test this on Dartium? It still wraps tests as far as I know, so it
will
> most likely break.

I tested it on ff. But thanks for the hint, I'll test it on drt as well.

[ahe, 2013-01-16 10:37:47]

On 2013/01/16 10:29:53, kustermann wrote:
> On 2013/01/16 10:27:36, ahe wrote:
> > Did you test this on Dartium? It still wraps tests as far as I know, so it
> will
> > most likely break.
> 
> I tested it on ff. But thanks for the hint, I'll test it on drt as well.

test.py -mrelease -rdrt -cnone ...

[kustermann, 2013-01-16 17:54:55]

I did run the following tests:

./tools/test.py -mrelease -cnone -rvm --report --time
./tools/test.py -mrelease -cnone -rdrt --report --time
./tools/test.py -mrelease -cdart2js -rdrt --report --time
xvfb-run -a ./tools/test.py -mrelease -cdart2js -rff --report --time

and all of them pass successfully on my local machine.

https://codereview.chromium.org/11886064/diff/2002/tests/language/language.st...
File tests/language/language.status (right):

https://codereview.chromium.org/11886064/diff/2002/tests/language/language.st...
tests/language/language.status:288:
call_constructor_on_unresolvable_class_test/01: Fail # Issue 2264
The wrapping of this test doesn't work because this test imports a library but
doesn't have a library declaration itself. See http://dartbug.com/2264.

https://codereview.chromium.org/11886064/diff/2002/tools/testing/dart/test_su...
File tools/testing/dart/test_suite.dart (right):

https://codereview.chromium.org/11886064/diff/2002/tools/testing/dart/test_su...
tools/testing/dart/test_suite.dart:1311: new RegExp("^(#?source|#?import|part)[
\t]+[\('\"]", multiLine: true);
This was broken before. After correcting this we'll get sometimes warnings:
 "Warning for somefile.dart: Browser tests require #library in any file that
uses #import, #source, or #resource"

The reason for that is that the wrapping of dart tests doesn't work in all
situations (see http://dartbug.com/2264).

[ahe, 2013-01-16 17:57:25]

Awesome, thank you!

LGTM

https://codereview.chromium.org/11886064/diff/2002/tools/testing/dart/test_su...
File tools/testing/dart/test_suite.dart (right):

https://codereview.chromium.org/11886064/diff/2002/tools/testing/dart/test_su...
tools/testing/dart/test_suite.dart:1311: new RegExp("^(#?source|#?import|part)[
\t]+[\('\"]", multiLine: true);
On 2013/01/16 17:54:55, kustermann wrote:
> This was broken before. After correcting this we'll get sometimes warnings:
>  "Warning for somefile.dart: Browser tests require #library in any file that
> uses #import, #source, or #resource"
> 
> The reason for that is that the wrapping of dart tests doesn't work in all
> situations (see http://dartbug.com/2264).

We should reword the warning to inform the user that the test should be marked
as failing with a reference to bug 2264.